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

philmoz

3450

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

« Reply #1430 on: 14 / February / 2015, 20:08:41 »

Advertisements

Quote from: waterwingz on 14 / February / 2015, 18:52:26

Patch file (1.4.0) to cleanup USB remote code per the discussion here : http://chdk.setepontos.com/index.php?topic=7127.msg120508#msg120508

Patch simplifies the code by removing unnecessary interlocks, fixes some issues with debug mode, and adds a usb_sync_wait() function to Lua and uBASIC for scripting use with multi-camera setups.

Updates to the usb_force_active() function per this discussion : http://chdk.setepontos.com/index.php?topic=7127.msg120539#msg120539 deferred pending reyalp updates to kbd.c files.

For consistency the 'usb_sync_wait_flag' entry in module_exportlist.c should be '&usb_sync_wait_flag' - this generates a variable reference instead of a function reference in module_hashlist.c.

(Both generate the same code with current compilers.)

Phil.

Logged

CHDK ports:
sx30is (1.00c, 1.00h, 1.00l, 1.00n & 1.00p)
g12 (1.00c, 1.00e, 1.00f & 1.00g)
sx130is (1.01d & 1.01f)
ixus310hs (1.00a & 1.01a)
sx40hs (1.00d, 1.00g & 1.00i)
g1x (1.00e, 1.00f & 1.00g)
g5x (1.00c, 1.01a, 1.01b)
g7x2 (1.01a, 1.01b, 1.10b)

waterwingz

12138

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

« Reply #1431 on: 14 / February / 2015, 20:25:28 »

Quote from: philmoz on 14 / February / 2015, 20:08:41

For consistency the 'usb_sync_wait_flag' entry in module_exportlist.c should be '&usb_sync_wait_flag' - this generates a variable reference instead of a function reference in module_hashlist.c.

Patch

usb_remote_update_v9a.patch (0.37 kB - downloaded 25 times.)

Logged

Ported : A1200 SD940 G10 Powershot N G16

reyalp

14079

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

« Reply #1432 on: 14 / February / 2015, 20:41:30 »

done in 4003

Logged

Don't forget what the H stands for.

waterwingz

12138

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

« Reply #1433 on: 20 / February / 2015, 20:01:34 »

Patch for S100 ND filter as discussed here : http://chdk.setepontos.com/index.php?topic=7887.msg120563#msg120563

Tested on S100.

s100_nd_filter_1-3-0.patch (0.55 kB - downloaded 26 times.)

s100_nd_filter_1-4-0.patch (0.55 kB - downloaded 27 times.)

Logged

Ported : A1200 SD940 G10 Powershot N G16

waterwingz

12138

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

« Reply #1434 on: 20 / February / 2015, 23:38:30 »

Patch to re-enable the creation of a ps.fi2 file for the S100. Tested on the 1.02A and thus assumed to likely work on the other versions.

s100_psfi2.patch (1.86 kB - downloaded 29 times.)

Logged

Ported : A1200 SD940 G10 Powershot N G16

reyalp

14079

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

« Reply #1435 on: 21 / February / 2015, 00:18:44 »

Quote from: waterwingz on 20 / February / 2015, 20:01:34

Patch for S100 ND filter as discussed here :

Added in trunk 3028, stable 4029

Quote

Patch to re-enable the creation of a ps.fi2 file for the S100. Tested on the 1.02A and thus assumed to likely work on the other versions.

Added in trunk 4030. As you suggested in IRC, I'll leave it there for people to try other firmware versions before enabling in 1.3

Logged

Don't forget what the H stands for.

TobiMarg

81

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

« Reply #1436 on: 23 / February / 2015, 10:02:22 »

Patch to show gps errors (no signal, no gps info in image, ...) using the message box instead of writing directly to the screen and wait/block for a few seconds. Because of the limited string length in the message box I had to insert some newlines (only fixed english and german).

gps_mbox.diff (4.66 kB - downloaded 30 times.)

Logged

waterwingz

12138

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

« Reply #1437 on: 28 / February / 2015, 14:48:19 »

Quote from: TobiMarg on 23 / February / 2015, 10:02:22

Patch to show gps errors (no signal, no gps info in image, ...) using the message box instead of writing directly to the screen and wait/block for a few seconds. Because of the limited string length in the message box I had to insert some newlines (only fixed english and german).

This was definitely on my list of things to clean up but I'm currently traveling with almost no internet access so have not had a chance to test it. Perhaps it would have been better to post to the forum thread for the GPS code first? I try to make it a point to submit my patches that way in case there is any discussion.

Having said that, rudi also sent me a list of things he wanted to do but can't currently find the time for. When I get back next week I'll get a combined list to the forum to discuss?

Logged

Ported : A1200 SD940 G10 Powershot N G16

waterwingz

12138

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

« Reply #1438 on: 14 / March / 2015, 15:23:54 »

Patch file to add codegen for the G10. Includes the addition of filewrite.c

Tested on 1.02a and by using arm-elf-objdump to compare the new boot.o, capt_seq.o, movie_rec.o files to the originals (after tweaking chdk_disasm.c to remove some of the optimization for nullsub and double jump removal for the validation only).

Would be good to add only to unstable until the 1.03b & 1.04a are tried.

g10_code_gen_r14.patch (501.62 kB - downloaded 26 times.)

Logged

Ported : A1200 SD940 G10 Powershot N G16

reyalp

14079

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

« Reply #1439 on: 14 / March / 2015, 16:45:48 »

Quote from: waterwingz on 14 / March / 2015, 15:23:54

Patch file to add codegen for the G10. Includes the addition of filewrite.c

Added, trunk 4066. Did you check remote capture, including -cont?

Quote

Would be good to add only to unstable until the 1.03b & 1.04a are tried.

Added to testing needed page. That said, diffing the files between platforms and verifying that only offsets change is probably good enough.

Logged

Don't forget what the H stands for.