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

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

  • 1685 Replies
  • 827180 Views
*

Offline srsa_4c

  • ******
  • 4451
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1630 on: 15 / December / 2019, 11:32:40 »
Advertisements
Your fix looks OK to me.
Thanks, committed. Fortunately, it was a rather benign bug - as long as malloc succeeded.

*

Offline blackhole

  • *****
  • 942
  • A590IS 101b
    • Planetary astrophotography
Source code for ixus275_elph350, firmware version 100a.

*

Offline reyalp

  • ******
  • 14111
Source code for ixus275_elph350, firmware version 100a.
Thanks, added in trunk r5435
Don't forget what the H stands for.

*

Offline blackhole

  • *****
  • 942
  • A590IS 101b
    • Planetary astrophotography
Source code for sx620hs, firmware version 100b.


*

Offline reyalp

  • ******
  • 14111
Source code for sx620hs, firmware version 100b.
Thanks. Added in trunk 5448
Don't forget what the H stands for.

*

Offline srsa_4c

  • ******
  • 4451
Two propset 12 related changes from my m100 patch that are not in trunk yet.
Code: [Select]
Index: core/shooting.c
===================================================================
--- core/shooting.c (revision 5449)
+++ core/shooting.c (working copy)
@@ -135,7 +135,7 @@
 int shooting_get_digital_zoom_mode(void)
 {
     int x=shooting_get_prop(PROPCASE_DIGITAL_ZOOM_MODE);
-#if CAM_PROPSET == 7 || CAM_PROPSET == 9 || CAM_PROPSET == 10|| CAM_PROPSET == 11
+#if CAM_PROPSET == 7 || CAM_PROPSET == 9 || CAM_PROPSET == 10 || CAM_PROPSET == 11 || CAM_PROPSET == 12
     if(x==1) {
         return 0;
     }
Index: platform/generic/wrappers.c
===================================================================
--- platform/generic/wrappers.c (revision 5449)
+++ platform/generic/wrappers.c (working copy)
@@ -95,7 +95,7 @@
 long set_property_case(long id, void *buf, long bufsize)
 {
     // ignore set on fake prop
-#if CAM_PROPSET == 7 || CAM_PROPSET == 9 || CAM_PROPSET == 10 || CAM_PROPSET == 11
+#if CAM_PROPSET == 7 || CAM_PROPSET == 9 || CAM_PROPSET == 10 || CAM_PROPSET == 11 || CAM_PROPSET == 12
     if(id==PROPCASE_SHOOTING) {
         return 0;
     }
The second one is clearly needed, not sure about the first one (no digital zoom here).

*

Offline reyalp

  • ******
  • 14111
Two propset 12 related changes from my m100 patch that are not in trunk yet.
The second one is clearly needed, not sure about the first one (no digital zoom here).
Thanks for catching those, checked in. I wondered how I missed the wrappers one, since sx730 is tested OK, but it turns out I only missed the set_ call  :-[
Don't forget what the H stands for.

Updated patch for G16  simple_movie_status variable.

https://chdk.setepontos.com/index.php?topic=14035
« Last Edit: 18 / April / 2020, 20:30:21 by waterwingz »
Ported :   A1200    SD940   G10    Powershot N    G16


*

Offline reyalp

  • ******
  • 14111
Updated patch for G16  simple_movie_status variable.

https://chdk.setepontos.com/index.php?topic=14035
Thanks. Added in r5469
Don't forget what the H stands for.

*

Offline blackhole

  • *****
  • 942
  • A590IS 101b
    • Planetary astrophotography
Source code for ixus265_elph340, firmware version 100c.

 

Related Topics


SimplePortal 2.3.6 © 2008-2014, SimplePortal