Adding new cameras, applying patches into trunk (with source code prepared) - page 33 - General Discussion and Assistance - CHDK Forum

Adding new cameras, applying patches into trunk (with source code prepared)

  • 1679 Replies
  • 785725 Views
*

Offline philmoz

  • *****
  • 3450
    • Photos
Advertisements
SX30 firmware version 1.00m.

There are no code changes between 1.00l and 1.00m in the firmware so this just updates the makefile to copy and rename the 1.00l firmware builds. I hope this is the correct way to do this.

Phil.
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)

*

Offline reyalp

  • ******
  • 14080
SX30 firmware version 1.00m.
Added, changeset 1210  chdkde changeset 689
Quote
I hope this is the correct way to do this.
Correct ? Probably not, but it's how we do it ;) You did have an extra -full in the batch-zip, which I took out.

At some point it would be nice to convert those monster blocks of copy/paste into a loop or something, with all the cameras/subs and beta status defined in one place. It seems like it could come from the directory tree, but you'd need a way of suppressing ones you didn't want in the autobuild.
« Last Edit: 18 / June / 2011, 19:59:28 by reyalp »
Don't forget what the H stands for.

Beta release of IXUS200_SD980 firmware 1.01c

Original port for 1.00c by RaduP.  Port from there to 1.01c by philmoz & waterwingz.  Testing by simon96 and titbb.



Ported :   A1200    SD940   G10    Powershot N    G16

*

Offline philmoz

  • *****
  • 3450
    • Photos
G12 & SX30 minor patch
- stubs cleanup and address fixes.
- removed redundant entries from platform_camera.h
- changed CAM_BLACK_LEVEL in camera.h to a calculated value based on bits-per-pixel (CAM_BLACK_LEVEL and CAM_WHITE_LEVEL overrides can be removed from most platform_camera.h files)
- removed definition of O_RDONLY from generic/wrappers.c since stdlib.h is now included

Phil.
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)


*

Offline philmoz

  • *****
  • 3450
    • Photos
Code cleanup in ubasic.c and script.c

Phil.

Edit: patch file removed. Updated patch in subsequent post.
« Last Edit: 25 / June / 2011, 23:59:27 by philmoz »
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)

*

Offline funnel

  • ****
  • 349
@philmoz

I took a look at your changes and I'd like to know why did you remove the refresh_physical_screen from stubs_min.S. Thanks.

EDIT: I see now that you're not using anywhere in code.  :)
« Last Edit: 25 / June / 2011, 04:28:16 by funnel »

*

Offline philmoz

  • *****
  • 3450
    • Photos
@philmoz

I took a look at your changes and I'd like to know why did you remove the refresh_physical_screen from stubs_min.S. Thanks.

Do you mean 'enabled_refresh_physical_screen'?
It's not used on the G12 and SX30.
The screen refresh method that uses 'enabled_refresh_physical_screen' doesn't work on the G12.
I use ScreenLock and ScreenUnlock instead (see platform/g12/lib.c).
I found that the ScreenLock and ScreenUnlock also worked on the SX30 so I made them both the same.

Phil.
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)

S95 firmware 1.00k release.

No negative feedback after 29 downloads of Beta version so I guess we are "good to go" with beta status.
Ported :   A1200    SD940   G10    Powershot N    G16


*

Offline reyalp

  • ******
  • 14080
Beta release of IXUS200_SD980 firmware 1.01c

Original port for 1.00c by RaduP.  Port from there to 1.01c by philmoz & waterwingz.  Testing by simon96 and titbb.
Added, chdk changeset 1212
G12 & SX30 minor patch
- stubs cleanup and address fixes.
- removed redundant entries from platform_camera.h
- changed CAM_BLACK_LEVEL in camera.h to a calculated value based on bits-per-pixel (CAM_BLACK_LEVEL and CAM_WHITE_LEVEL overrides can be removed from most platform_camera.h files)
- removed definition of O_RDONLY from generic/wrappers.c since stdlib.h is now included
Added, chdk changeset 1213

S95 firmware 1.00k release.
Added, chdk changeset 1215
« Last Edit: 25 / June / 2011, 21:03:23 by reyalp »
Don't forget what the H stands for.

*

Offline reyalp

  • ******
  • 14080
Code cleanup in ubasic.c and script.c
This looks like a nice memory saving (>1k in my build.)

How thoroughly has this been tested ? The short/int conversion concerns me slightly: for signed values the difference between casting and simply passing an int to a function that expects a short will be significant, even though they both use a full register. Most of the APEX96 values can legitimately be negative.
Don't forget what the H stands for.

 

Related Topics