In platform_camera.h are #define CAM_MULTIPART (you removed multipartition part)
You're right, I forgot to remove that.
and #define CAM_BRACKETING (camera has no native bracketing mode) unnecessary,
True, I'll remove that too, it's a leftover from the reference port I used.
maybe also #define CAM_CAN_UNLOCK_OPTICAL_ZOOM_IN_VIDEO
This one calls _UnsetZoomForMovie. I think it's not needed because the camera natively supports zooming in video.
and #define CAM_AF_SCAN_DURING_VIDEO_RECORD.
I removed that because the camera crashes when zooming after calling _MakeAfScan. I found that _DoAflock acts in a similar way but also crashes when I set the camera to manual focus and try to zoom. To avoid trouble I just removed it. Maybe we can find a solution for this later.
The ALT mode switching with DISP+SET is a litte pain. I use the alternative ALT mode button 'playback'. But why is this button defined as 'video'? Maybe it was better using the playback button as default ALT button.
Yes, I have plans to change that and move it as the REC button and add a new PLAYBACK button to keyboard.h. I was waiting to see if the patch for shoot_full_only will be applied first to not screw up that work.
Two capture modes (high speed burst & best image selection) use the continuous capturing. These modes should be excluded for 'bracketing in continuous mode' function. Maybe this is not only a SX220 problem. Then a generic solution would be better.
I like the bracketing on high speed burst. I find it useful for ultra fast HDR bracketing. I don't think that can cause problems.
I agree for the best image selection, should be removed.
