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

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

  • 1645 Replies
  • 543943 Views
*

Offline srsa_4c

  • ******
  • 4302
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1270 on: 01 / December / 2013, 14:41:48 »
Advertisements
Two patches, I'm not completely confident to check these in.

sx210:
[OK] keymap corrections, makes shoot (the script command) work
[??] I have changed two firmware variable addresses (simply enabled the automatically found ones), the test builds contained this change, but I didn't ask for confirmation
[??] I have enabled alternative ALT buttons, it's probably OK.

If the tester's words are not convincing enough, perhaps I / someone should ask him again?

sx230:
Enabled ARAM instead of EXMEM.
Here's an early reaction (not my patch), here's another (using a build with the v2 aram patch with sx230 added). edit: another report.

I didn't touch anything else in the patched files to make the patches "noiseless", some of them look really ugly.
« Last Edit: 03 / December / 2013, 12:36:27 by srsa_4c »

*

Offline reyalp

  • ******
  • 12793
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1271 on: 03 / December / 2013, 17:10:11 »
Sorry I haven't looked at the actual patches carefully yet.

SX230 in aram seems like it should be fine. It would be good to know how much canon heap is available, if it is really low exmem might still be required.

For SX210, keymap and adjustable alt should be fine. For the firmware variables, it would be good to either sanity check in the disassembly (look for equivalent code on a known working camera) or get a direct report from the test. recreview_hold being wrong in the original port would not be a surprise at all.
Don't forget what the H stands for.

*

Offline srsa_4c

  • ******
  • 4302
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1272 on: 06 / December / 2013, 11:59:26 »
SX230 in aram seems like it should be fine. It would be good to know how much canon heap is available, if it is really low exmem might still be required.
I have posted a request for malloc heap details. Enabling aram beside exmem would not bring much visible improvements, so I hope there's enough Canon heap...
Quote
For SX210, keymap and adjustable alt should be fine.
Committed.
Quote
For the firmware variables, it would be good to either sanity check in the disassembly (look for equivalent code on a known working camera) or get a direct report from the test. recreview_hold being wrong in the original port would not be a surprise at all.
Problem is, these may not be too easy to test (unless you have an idea), and I don't know whether the other r43 ports are a reliable source. I'll try to trace things in the disassembly...

*

Offline blackhole

  • *****
  • 800
  • A590IS 101b
    • Planetary astrophotography
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1273 on: 14 / December / 2013, 04:10:13 »
a530
PTP display stuff
Menu colors and digital zoom corrected.


*

Offline reyalp

  • ******
  • 12793
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1274 on: 14 / December / 2013, 21:28:47 »
a530
PTP display stuff
Menu colors and digital zoom corrected.
Add in trunk changeset 3272 release changeset 3273
Don't forget what the H stands for.

Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1275 on: 30 / December / 2013, 00:33:01 »
Propcase left out of propset3 - PROPCASE_AF_LOCK.

Tested on ixus120_sd940.
Ported :   A1200    SD940   G10    Powershot N    G16

*

Offline reyalp

  • ******
  • 12793
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1276 on: 30 / December / 2013, 00:51:37 »
Propcase left out of propset3 - PROPCASE_AF_LOCK.

Tested on ixus120_sd940.
Added,  trunk changeset 3286 release changeset 3287
Don't forget what the H stands for.

Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1277 on: 30 / December / 2013, 08:42:35 »
Found the other two AF props for propset3 as well.
Ported :   A1200    SD940   G10    Powershot N    G16


*

Offline reyalp

  • ******
  • 12793
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1278 on: 30 / December / 2013, 15:12:12 »
Found the other two AF props for propset3 as well.
Added, trunk changeset 3288 release changeset 3289
Don't forget what the H stands for.

*

Offline nafraf

  • *****
  • 1308
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1279 on: 31 / December / 2013, 09:06:52 »
Patch to include more keys to ubtest.bas: zoom in, zoom out, mode, help, video, playback

 

Related Topics