supplierdeeply

SX410 IS porting thread

  • 86 Replies
  • 42951 Views
*

Offline blackhole

  • *****
  • 642
  • A590IS 101b
    • Planetary astrophotography
Re: SX410 IS porting thread
« Reply #20 on: 11 / September / 2016, 21:27:40 »
Advertisements
Quote
A added a link for the shutter problem and mention of iris in notes.txt
Thanks.

- the LED addresses in lib.c do not match the sigfinder's suggestion AND the code uses the legacy switching method which is unsuitable for 0xc022dXXX GPIOs
In addition to the things srsa_4c mentioned, I noticed that the included movie_rec.c does match what is generated by codegen, because of the lines
Code: [Select]
//"    STMFD   SP!, {R4,R5} \n"
//"    LDR     R5, =0xFFC0CC8C \n"

There is also a confrontation between CHDK shortcut for zebra and native Canon key combinations for AFL.
Will be fixed with the next patch.

Quote
Have the propcases been verified? platform_camera.h has PROPSET 6, but that's quite old compared to the camera release date.
Checked, the camera uses PROPSET 6.

Quote
Regarding the button problems
Quote
Other keys I could not check because the value of the PS1 and PS2 are constantly changing.
Probably only some unrelated bits are changing. You can read the values with rmem and convert to binary to see if any additional bits really change in response to the buttons. e.g.
Code: [Select]
rmem -i32 0x12224 3
I'll see,but I think that is OK.

Quote
If the shutter button is truly being detected outside of the normal physw_task code, the camera log might provide some insight into what code is doing it. E.g.

Code: [Select]
!require'extras/devutil'.init_cli()
dlstart
...  half press shutter in alt mode
dlgetcam
Look through the messages in dbg.log for anything that looks related to the half press, if you find them, look for the strings in rom code etc.
I'll see next days, when I catch enough free time.

Thank you both for your advice!

*

Offline blackhole

  • *****
  • 642
  • A590IS 101b
    • Planetary astrophotography
Re: SX410 IS porting thread
« Reply #21 on: 20 / September / 2016, 19:13:38 »
I found the cause of the problem with the keyboard. I forgot to 'zoom in' and  'zoom out' keys.
Now everything works well and the scripts are executed without problems.
Attached is a patch for all the forgotten stuff.

*

Offline reyalp

  • ******
  • 11591
Re: SX410 IS porting thread
« Reply #22 on: 24 / September / 2016, 15:31:30 »
I found the cause of the problem with the keyboard. I forgot to 'zoom in' and  'zoom out' keys.
Now everything works well and the scripts are executed without problems.
Attached is a patch for all the forgotten stuff.
Nice work :)
Added in trunk 4693. Is this port ready to be added to the autobuild?
Have you gone through http://chdk.wikia.com/wiki/Testing?
Don't forget what the H stands for.

*

Offline blackhole

  • *****
  • 642
  • A590IS 101b
    • Planetary astrophotography
Re: SX410 IS porting thread
« Reply #23 on: 24 / September / 2016, 18:15:41 »
Quote
Added in trunk 4693. Is this port ready to be added to the autobuild?
Have you gone through http://chdk.wikia.com/wiki/Testing?
OK,thanks.
The camera passes all the test scripts without any problems.
Free memory is always above 2.5MB.
The camera all the time works really stable and I do not see any problem.


*

Offline reyalp

  • ******
  • 11591
Re: SX410 IS porting thread
« Reply #24 on: 24 / September / 2016, 21:12:43 »
The camera all the time works really stable and I do not see any problem.
I've added it to the trunk autobuild as "alpha". Congratulations :D

You can add the "auto" key to the keys CHDK knows about. This would allow it to be pressed and detected from script, and it could also be allowed as an alt button.
In addition to the keymap, it should go in:
include/keyboard.h
modules/script_key_funcs.c
and in the platform key mask if it isn't already.


If play/rec switching works from PTP, the "TODO" comment should be removed from platform_camera.h CAM_USB_EVENTID
Don't forget what the H stands for.

*

Offline blackhole

  • *****
  • 642
  • A590IS 101b
    • Planetary astrophotography
Re: SX410 IS porting thread
« Reply #25 on: 30 / September / 2016, 14:46:07 »
Added "AUTO" key and support for 'filewritetask'.
EDIT:
The default shortcut for the Histogram was changed in Half Shutter + Display,due to conflicts with Canon's key combination for AEL.
« Last Edit: 30 / September / 2016, 15:20:48 by blackhole »

*

Offline reyalp

  • ******
  • 11591
Re: SX410 IS porting thread
« Reply #26 on: 30 / September / 2016, 23:36:19 »
Added "AUTO" key and support for 'filewritetask'.
EDIT:
The default shortcut for the Histogram was changed in Half Shutter + Display,due to conflicts with Canon's key combination for AEL.
Thanks. Added in trunk 4696

For filewrite, have you tested whether the camera crashes if you remote shoot and then switch to play?

Also, have you tested remoteshoot with continuous mode, e.g. enable continuous mode in the canon ui, and then do something like rs -cont=5 ?

If either of these fail, it should be mentioned in the notes.txt.

See https://chdk.setepontos.com/index.php?topic=4338.msg111230#msg111230 / http://chdk.wikia.com/wiki/User:Nafraf/RemoteShootIssues
Don't forget what the H stands for.

*

Offline blackhole

  • *****
  • 642
  • A590IS 101b
    • Planetary astrophotography
Re: SX410 IS porting thread
« Reply #27 on: 01 / October / 2016, 13:38:19 »
Quote
For filewrite, have you tested whether the camera crashes if you remote shoot and then switch to play?
After switching back to play mode, camera works well, the only thing I see is a question mark and the message "Unidentified Image".
Quote
Also, have you tested remoteshoot with continuous mode, e.g. enable continuous mode in the canon ui, and then do something like rs -cont=5 ?
In continuous mode all looks good, all images are stored and undamaged.

I do not see any difference in the work of the camera, when I compile chdk with or without filewrite_hdr.c file.
Why we really need that file?


*

Offline reyalp

  • ******
  • 11591
Re: SX410 IS porting thread
« Reply #28 on: 01 / October / 2016, 13:43:51 »
I do not see any difference in the work of the camera, when I compile chdk with or without filewrite_hdr.c file.
Why we really need that file?
It's used in to generate filewrite.c when you run code-gen. This isn't done in the normal build, code-gen only needs to be run if the generated assembly files need to be changed (i.e. codgen.txt changed, or a new sub added, or the _hdr file changed.)

Not adding it to svn was an accident on my part, sometimes tortoise doesn't "add" files from a patch and I missed this one. This is now fixed.
Don't forget what the H stands for.

*

Offline blackhole

  • *****
  • 642
  • A590IS 101b
    • Planetary astrophotography
Re: SX410 IS porting thread
« Reply #29 on: 10 / October / 2016, 16:16:53 »
Correction for 'Zebra'.
With the old DEF for "recreview_hold", with zebra was shown and some garbage in the lower and upper part of the display.
The same problem is likely also present on sx400is.

 

Related Topics