Adding feature support "File Write Task" (FWT): 52 Cams || 41 done || 11 to go - page 20 - General Discussion and Assistance - CHDK Forum supplierdeeply

Adding feature support "File Write Task" (FWT): 52 Cams || 41 done || 11 to go

  • 258 Replies
  • 93172 Views
*

Offline koshy

  • *****
  • 1096
Advertisements
Please test patch for a450.
Thanks, I did. There is some problem with it.
A small S sized JPEG got transferred, then the cam crashed and CHDKPTP said:

Code: [Select]
ERROR: C:\Users\web\Downloads\chdkptp-r599-win32\lua\cli.lua:1903: attempt to concatenate local 'err' (a boolean value)
stack traceback:
[C]: in function 'xpcall'
C:\Users\web\Downloads\chdkptp-r599-win32\lua\cli.lua:244: in function 'execute'
C:\Users\web\Downloads\chdkptp-r599-win32\lua\gui_user.lua:74: in function <C:\Users\web\Downloads\chdkptp-r599-win32\lua\gui_user.lua:72>
(...tail calls...)
[C]: in function 'MainLoop'
C:\Users\web\Downloads\chdkptp-r599-win32\lua\gui.lua:749: in function <C:\Users\web\Downloads\chdkptp-r599-win32\lua\gui.lua:721>
(...tail calls...)
C:\Users\web\Downloads\chdkptp-r599-win32\lua\main.lua:266: in main chunk
[C]: in function 'require'
[string "require('main')"]:1: in main chunk

A high quality full resolution JPEG didn't even get transferred before the camera crashed and CHDKPTP put out the above. I'm attaching a ROMLOG of the crash and the build logs.
« Last Edit: 09 / June / 2015, 19:12:28 by koshy »
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...)

*

Offline srsa_4c

  • ******
  • 4451
@nafraf
Since you have no VxWorks cameras to test this on, I'll take a look at the a450 fwt issue (on my a460).

*

Offline koshy

  • *****
  • 1096
@nafraf
Since you have no VxWorks cameras to test this on, I'll take a look at the a450 fwt issue (on my a460).
It never ceases to amaze me what many of you do without the camera on your desk. Chapeau.
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...)

*

Offline nafraf

  • *****
  • 1308
@srsa_4c. Thanks. After porting a460, it should be easy to replicate on a450 and others.


*

Offline srsa_4c

  • ******
  • 4451
After porting a460, it should be easy to replicate on a450 and others.
Turns out, your patch was actually correct. In one of the copied functions, FW $ copied one line less from the firmware routine (that's a bug in code_gen). I fixed it by adding the missing instruction with FW 1. We could probably just use ->FW instead of FW $ there, as the routine doesn't loop back to its modified first part.

*

Offline koshy

  • *****
  • 1096
After porting a460, it should be easy to replicate on a450 and others.
Turns out, your patch was actually correct. In one of the copied functions, FW $ copied one line less from the firmware routine (that's a bug in code_gen). I fixed it by adding the missing instruction with FW 1. We could probably just use ->FW instead of FW $ there, as the routine doesn't loop back to its modified first part.
Thank you for following up on this. @nafraf Well done as many times before.

I tested both your A450 and A460 FWT patches and both worked fine in single mode including switching to play and back. I found no continuous mode on either so I guess they also have none... UI overlay and CHDKPTP palette looked fine, too. So, we're down to 18 cams it seems.
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...)

*

Offline nafraf

  • *****
  • 1308
Turns out, your patch was actually correct. In one of the copied functions, FW $ copied one line less from the firmware routine (that's a bug in code_gen). I fixed it by adding the missing instruction with FW 1. We could probably just use ->FW instead of FW $ there, as the routine doesn't loop back to its modified first part.
Thanks!

I think that code_gen tool is fine. I found a bug in code_gen.txt, the length of function was forced (FUNC ref=1 length=39). A new patch in attachment. If you agree, I can submit the patch.

*

Offline srsa_4c

  • ******
  • 4451
I think that code_gen tool is fine. I found a bug in code_gen.txt, the length of function was forced (FUNC ref=1 length=39).
Thanks for noticing. I'm obviously not as experienced when it comes to code_gen.
Quote
A new patch in attachment. If you agree, I can submit the patch.
Looks okay as the generated code is correct.
I found no continuous mode on either so I guess they also have none...
They do have continuous mode, you can find it in the shooting menu (the one you can display by pressing SET in shooting mode).


*

Offline koshy

  • *****
  • 1096
I found no continuous mode on either so I guess they also have none...
They do have continuous mode, you can find it in the shooting menu (the one you can display by pressing SET in shooting mode).
How silly of me... You have to be in "P" or manual mode opposed to Auto for it to show up.  :haha

I'll re-test the three cams in question in that regard.
« Last Edit: 11 / June / 2015, 12:35:04 by koshy »
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...)

*

Offline koshy

  • *****
  • 1096
I tested A420, A450 and A460 in continuous mode with rs -jpg -cont=5 and all was fine including switching to play and back.
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...)

 

Related Topics


SimplePortal 2.3.6 © 2008-2014, SimplePortal