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

  • 1470 Replies
  • 166280 Views
*

Offline reyalp

  • ******
  • 9858
  • Publish
    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

    • ******
    • 3118
  • Publish
    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 »

  • Publish
    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 ?

    *

    Offline ADamb

    • **
    • 65
    • sx200is
  • Publish
    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

    • ******
    • 9858
  • Publish
    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.

  • Publish
    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.

    *

    Offline msl

    • *****
    • 1173
    • A720 IS, SX220 HS 1.01a
      • CHDK inside
  • Publish
    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
    German CHDK pages:  CHDK forum | CHDK inside | CHDK Twitter News by msl | Download CHDK-DE (Autobuild)
    Note: SDM violates the GPL rules!

  • Publish
    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.


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


    *

    Offline reyalp

    • ******
    • 9858
  • Publish
    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