Adding new cameras, applying patches into trunk (with source code prepared) - page 80 - General Discussion and Assistance - CHDK Forum

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

  • 1685 Replies
  • 839415 Views
*

Offline reyalp

  • ******
  • 14121
Advertisements
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.
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 »
Don't forget what the H stands for.

*

Offline srsa_4c

  • ******
  • 4451
Thx.
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 »

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 ?
Ported :   A1200    SD940   G10    Powershot N    G16

*

Offline ADamb

  • **
  • 65
  • sx200is
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.
SX200 IS-100C


*

Offline reyalp

  • ******
  • 14121
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.
Don't forget what the H stands for.

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.
Ported :   A1200    SD940   G10    Powershot N    G16

*

Offline msl

  • *****
  • 1280
  • A720 IS, SX220 HS 1.01a
    • CHDK-DE links
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
CHDK-DE:  CHDK-DE links

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.
Ported :   A1200    SD940   G10    Powershot N    G16


One more fix for the IXUS115 user defined ALT function.   Hopefully the last - its a pain not having the camera to test with.

Ported :   A1200    SD940   G10    Powershot N    G16

*

Offline reyalp

  • ******
  • 14121
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
Don't forget what the H stands for.

 

Related Topics


SimplePortal 2.3.6 © 2008-2014, SimplePortal