Adding feature support "Extended TV Range" (ETR): 30 Cams || 0 to go :-)

koshy

1096

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #130 on: 04 / August / 2014, 19:01:15 »

Advertisements

Quote from: srsa_4c on 04 / August / 2014, 17:08:57

If you'd like to investigate further

Sure, I'm a curious person and while I had intended not to get down to the manual code editing level as I just don't have all of the skills needed here your instructions were just to precise to ignore

Also commenting out six lines isn't really "code editing".

Anyway, for IXUS 50:

No USB cable pluged in:
0xc0220204: 0x0404099b

Upon USB cable plug-in
0xc0220204 becomes 0x040409db

Both in shooting and play mode.

« Last Edit: 04 / August / 2014, 19:02:50 by koshy »

Logged

Koshy had a little ELPH which wasn't white as snow but everywhere that Koshy went the ELPH was sure to go. (actually an SD, but that detail ruins the rhyme...)

koshy

1096

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #131 on: 04 / August / 2014, 19:02:10 »

And for IXUS 700:

No USB cable pluged in:
0xc0220204: 0x24042f9f

Upon USB cable plug-in
0xc0220204 becomes 0x24042fdf

Both in shooting and play mode.

Logged

Koshy had a little ELPH which wasn't white as snow but everywhere that Koshy went the ELPH was sure to go. (actually an SD, but that detail ruins the rhyme...)

nafraf

1308

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #132 on: 04 / August / 2014, 20:34:02 »

@srsa_4c. Thanks for the help.

@koshy. Using srsa_4c's patch for ixus30/40 as reference, and using the values you posted, here are the patches for ixus50 and ixus700. Please test if it works. Thanks.

« Last Edit: 05 / October / 2014, 00:51:28 by nafraf »

Logged

nafraf

1308

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #133 on: 04 / August / 2014, 21:06:29 »

ixus900_sd900 100c

Quote from: koshy on 04 / August / 2014, 14:21:46

32772 is Manual mode (whatever is on current mode map seems to be wrong, if I choose Manual in CHDKPTP the cam crashes)

Camera crashes because Manual mode is mapped to 33772 (a invalid mode). I'm trying to fix this in new version.

Quote

32774 sets exposure time to 1'' so I guess it should be called long shutter or something. This prop does not get set by choosing long shutter from the camera menu though.

EDIT: as we found for a previous cam choosing "long shutter" from the camera menu doesn't alter the prop.

This is another bug in the port. CAM_PROPSET is set as 1, but ixus900 has a Digic 3 processor, then probably CAM_PROPSET is 2. Try to check the value of property 49.

Properties are documented here: http://chdk.wikia.com/wiki/PropertyCase

« Last Edit: 05 / October / 2014, 00:51:48 by nafraf »

Logged

reyalp

14034

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #134 on: 04 / August / 2014, 21:44:32 »

Quote from: nafraf on 04 / August / 2014, 21:06:29

This is another bug in the port. CAM_PROPSET is set as 1, but ixus900 has a Digic 3 processor, then probably CAM_PROPSET is 2. Try to check the value of property 49.

There were a few crossovers, as noted on the wiki

Quote

Furthermore, some Digic III cameras (G7, SD800IS, SD900) use the Digic II property set.

I'd be a bit surprised if a port got that far with the wrong propset. Propset 1 was quite different from the later propsets, so it would be really broken.

Logged

Don't forget what the H stands for.

nafraf

1308

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #135 on: 04 / August / 2014, 22:14:49 »

Quote from: reyalp on 04 / August / 2014, 21:44:32

There were a few crossovers, as noted on the wiki
Quote
Furthermore, some Digic III cameras (G7, SD800IS, SD900) use the Digic II property set.

I'd be a bit surprised if a port got that far with the wrong propset. Propset 1 was quite different from the later propsets, so it would be really broken.

Sorry, I did not see the note. I got confused with the note in the platform/ixus900_sd900/platform_camera.h

Code: [Select]

    #define CAM_PROPSET                     1       // ToDo: test if correct

Logged

waterwingz

12138

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #136 on: 04 / August / 2014, 23:32:46 »

Quote from: reyalp on 04 / August / 2014, 21:44:32

Quote from: nafraf on 04 / August / 2014, 21:06:29
This is another bug in the port. CAM_PROPSET is set as 1, but ixus900 has a Digic 3 processor, then probably CAM_PROPSET is 2. Try to check the value of property 49.
There were a few crossovers, as noted on the wiki
Quote
Furthermore, some Digic III cameras (G7, SD800IS, SD900) use the Digic II property set.

I'd be a bit surprised if a port got that far with the wrong propset. Propset 1 was quite different from the later propsets, so it would be really broken.

I think we sometimes lose sight of the fact that propsets are a CHDK invention. Each propset is just a grouping the various CHDK devs have noticed of how Canon developers are using those variables over some random development period. It's quite possible that some cameras could fall in between two propsets - sort of like a propset 1A. The Canon devs tend to not rev their variable map for every camera and that's given us what we now think of as propsets.

Logged

Ported : A1200 SD940 G10 Powershot N G16

nafraf

1308

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #137 on: 05 / August / 2014, 02:56:09 »

Test version for ixus980_sd990 100e

ixus980_sd990-100e-1.3.0-3568-full.zip (678.38 kB - downloaded 13 times.)

Logged

koshy

1096

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #138 on: 05 / August / 2014, 06:41:14 »

Quote from: nafraf on 04 / August / 2014, 21:06:29

ixus900_sd900 100c
Quote from: koshy on 04 / August / 2014, 14:21:46
32772 is Manual mode (whatever is on current mode map seems to be wrong, if I choose Manual in CHDKPTP the cam crashes)
Camera crashes because Manual mode is mapped to 33772 (a invalid mode). I'm trying to fix this in new version.

Setting Manual mode via CHDKPTP works now. Looks like it was a typo in the number.
I re-tested everything else and it worked. So I'm marking IXUS 900 as done.

Logged

Koshy had a little ELPH which wasn't white as snow but everywhere that Koshy went the ELPH was sure to go. (actually an SD, but that detail ruins the rhyme...)

koshy

1096

Re: Adding feature support "Extended TV Range" (ETR): 30 Cams || 6 to go :-)

« Reply #139 on: 05 / August / 2014, 07:08:09 »

Quote from: nafraf on 04 / August / 2014, 20:34:02

@koshy. Using srsa_4c's patch for ixus30/40 as reference, and using the values you posted, here are the patches for ixus50 and ixus700. Please test if it works. Thanks.

Thanks. I built the according CHDK versions with your patch and on both IXUS 50 and IXUS 700 they worked beautifully. USB Remote now works on all of the cameras we completed looking at.

Logged

Koshy had a little ELPH which wasn't white as snow but everywhere that Koshy went the ELPH was sure to go. (actually an SD, but that detail ruins the rhyme...)