« Reply #45 on: 31 / January / 2014, 20:09:11 »
Works on my G1X (r49), I can open the door with the camera running.
It requires the startup delay to boot with the door open.
Thanks for confirming, can I commit that patch, or do the #defines need a different name?
Looks ok to me, I think the names are fine.
Is this something that should be compiled in by default or enabled with a build option only for people who really need it?
IMHO it doesn't need to be in the autobuilds. If somebody needs it, it can be enabled in their build.
Should probably add a new OPT_xxx setting for buildconf.inc, and makefile changes that people can use to conditionally enable the code.
Phil.
In revision 3344 (1.3 only) I have added a new build option - OPT_RUN_WITH_BATT_COVER_OPEN
This is implemented for the G12, G1X and SX40.
The idea is that cameras that have the code for running with the battery door open can use this to only include the code when the build option is selected, rather than have it in the autobuild version for all users.
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)