Adding new cameras, applying patches into trunk (with source code prepared) - page 80 - General Discussion and Assistance - CHDK Forum supplierdeeply
« previous next »
Pages: 1 ... 75 76 77 78 79 [80] 81 82 83 84 85 ... 167   Go Down

Adding new cameras, applying patches into trunk (with source code prepared)

  • 1666 Replies
  • 697191 Views
*

Offline reyalp

  • ******
  • 13847
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #790 on: 30 / July / 2012, 23:06:49 »
Advertisements
Quote from: srsa_4c on 30 / July / 2012, 17:44:44
The following patch hasn't been committed yet (as it doesn't affect any other model, I think it could go to both 1.1 and trunk. The patch still applies cleanly to 1.1 and almost cleanly to current trunk. The A2200 port is planned to also use this new code path.
Quote from: srsa_4c on 03 / July / 2012, 10:36:02
Here's my updated patch to prevent the possibility of a SD override related crash on the Ixus850/SD800. I think this code will need some cleanup in the future - but that would require tests on cameras none of the current developers have...
Thanks for the reminder, I'll add this when I get a chance.
Quote
Another issue: I noticed changeset 1988, which made me do a little research.
CAM_FIRMWARE_MEMINFO is only activated for the following cameras:
a2100, a540, a590, d10, g10, g12, g1x, ixus310_elph500hs, s100, s95, sx130is, sx20, sx30, sx40hs
With this enabled, my usual DIGIC II test cameras showed a big performance improvement during multi file upload over CHDK-PTP (for small files: about 1-2 second/file without CAM_FIRMWARE_MEMINFO, at least five times faster with it).
That's useful info, I knew the old way was bad, but never tested how bad. This probably affects live view performance too.
Quote
Don't know how this situation should be handled:
a) enable for all DryOS (i.e. trust the sigfinder)
b) enable for all
c) enable only individually, after tests
If it's c), I have tested it on 3 cameras, patch attached.
I think turning it on for all dryos should be fine. I meant to do this before the least release but didn't get to it. In this post http://chdk.setepontos.com/index.php?topic=2509.msg66350#msg66350 I wanted to get a few more models tested, but I expect it's OK.

For vxworks, we need to find sys_mempart_id, which I haven't finished doing for all cameras. Baring typos, I think to should be safe to turn on for all that have been found.


edit:
Added sd override fix in trunk changeset 2029, meminfo in  changeset 2030, merged both to stable in changeset 2031
« Last Edit: 31 / July / 2012, 02:43:55 by reyalp »
Logged
Don't forget what the H stands for.
*

Offline srsa_4c

  • ******
  • 4450
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #791 on: 31 / July / 2012, 23:35:49 »
Thx.
Quote from: reyalp on 30 / July / 2012, 23:06:49
I think turning it on for all dryos should be fine. I meant to do this before the least release but didn't get to it. In this post http://chdk.setepontos.com/index.php?topic=2509.msg66350#msg66350 I wanted to get a few more models tested, but I expect it's OK.

For vxworks, we need to find sys_mempart_id, which I haven't finished doing for all cameras. Baring typos, I think to should be safe to turn on for all that have been found.
From a quick look, sys_mempart_id could be found by looking at an eventproc named "memShow" which is a wrapper for memPartShow. The latter needs sys_mempart_id as its first param. Haven't looked at a lot of dumps, but memShow seems to be present in all of them (except in Ixus30/40).
The PTP transfer speed (for small files or liveview) on the Ixus65 (DIGIC II, USB 2.0) is nearly 4 times as fast as without CAM_FIRMWARE_MEMINFO.
« Last Edit: 31 / July / 2012, 23:39:38 by srsa_4c »
Logged
*

Offline waterwingz

  • ******
  • 12130
    • Waterwingz
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #792 on: 01 / August / 2012, 17:48:56 »
As small patch to change the default <ALT> key for the ixus115 and ixus120 to be the Playback key (both cameras already implement user adjustable <ALT> keys).   

Question : should we consider this for every camera that does not have a Print key ?
Logged
Ported :   A1200    SD940   G10    Powershot N    G16
*

Offline ADamb

  • **
  • 65
  • sx200is
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #793 on: 02 / August / 2012, 13:56:00 »
Patches for sx200is 100c, revision 2034.
1. Add transparent grey to use for menu background, use custom palette in video playback mode.
2. PTP live support, CAM_FIRMWARE_MEMINFO.
Logged
SX200 IS-100C
*

Offline reyalp

  • ******
  • 13847
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #794 on: 03 / August / 2012, 00:43:20 »
Quote from: waterwingz on 01 / August / 2012, 17:48:56
As small patch to change the default <ALT> key for the ixus115 and ixus120 to be the Playback key (both cameras already implement user adjustable <ALT> keys).   
Added trunk changeset 2039  release changeset 2040
Quote
Question : should we consider this for every camera that does not have a Print key ?
I would like to standardize on play as much as possible, but I don't think it is urgent. If there are any left that use combos, I'd like to move away from that.
Logged
Don't forget what the H stands for.
*

Offline waterwingz

  • ******
  • 12130
    • Waterwingz
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #795 on: 03 / August / 2012, 18:01:33 »
Quote from: reyalp on 03 / August / 2012, 00:43:20
I would like to standardize on play as much as possible, but I don't think it is urgent. If there are any left that use combos, I'd like to move away from that.
A quick grep of the current trunk for KEY_PRINT does not show any combos remaining.
Logged
Ported :   A1200    SD940   G10    Powershot N    G16
*

Offline msl

  • *****
  • 1280
  • A720 IS, SX220 HS 1.01a
    • CHDK-DE links
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #796 on: 04 / August / 2012, 04:34:31 »
Quote from: waterwingz on 03 / August / 2012, 18:01:33
A quick grep of the current trunk for KEY_PRINT does not show any combos remaining.

ixus300/sd4000 KEY_UP + KEY_LEFT

http://trac.assembla.com/chdk/browser/trunk/platform/ixus300_sd4000/kbd.c#L292
Logged
CHDK-DE:  CHDK-DE links
*

Offline waterwingz

  • ******
  • 12130
    • Waterwingz
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #797 on: 04 / August / 2012, 07:58:04 »
Quote from: msl on 04 / August / 2012, 04:34:31
Quote from: waterwingz on 03 / August / 2012, 18:01:33
A quick grep of the current trunk for KEY_PRINT does not show any combos remaining.
ixus300/sd4000 KEY_UP + KEY_LEFT
Looks like my "quick grep" was too quick. I see that one now too.  That camera seems to set the record for having a minimum set of buttons!  And the current method of getting into <ALT> mode looks really ugly to me.  Unfortunately,  the kbd.c file for that camera does not have the key code for the Playback key.
Logged
Ported :   A1200    SD940   G10    Powershot N    G16
*

Offline waterwingz

  • ******
  • 12130
    • Waterwingz
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #798 on: 04 / August / 2012, 08:01:22 »
One more fix for the IXUS115 user defined ALT function.   Hopefully the last - its a pain not having the camera to test with.

Logged
Ported :   A1200    SD940   G10    Powershot N    G16
*

Offline reyalp

  • ******
  • 13847
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #799 on: 04 / August / 2012, 17:41:08 »
Quote from: ADamb on 02 / August / 2012, 13:56:00
Patches for sx200is 100c, revision 2034.
1. Add transparent grey to use for menu background, use custom palette in video playback mode.
Added, trunk changeset 2045 release changeset 2046. Note I modified this slightly, removing COLOR_GREY_BG which isn't defined/used anywhere else.

Quote
2. PTP live support, CAM_FIRMWARE_MEMINFO.
Added, trunk changeset 2047 release changeset 2048
Logged
Don't forget what the H stands for.
Pages: 1 ... 75 76 77 78 79 [80] 81 82 83 84 85 ... 167   Go Up
« previous next »
 

Related Topics