Changes: Quality definitions to standard jpeg quality

Author: cnadler86

README.md

@@ -110,13 +110,15 @@ make USER_C_MODULES=../../../../micropython-camera-API/src/micropython.cmake BOA
 ```
 
 If you experience problems, visit [MicroPython external C modules](https://docs.micropython.org/en/latest/develop/cmodules.html).
+
 ## Notes
 
 - The OV5640 pinout is compatible with boards designed for the OV2640 but the voltage supply is too high for the internal 1.5V regulator, so the camera overheats unless a heat sink is applied. For recording purposes the OV5640 should only be used with an ESP32S3 board. Frame sizes above FHD framesize should only be used for still images due to memory limitations.
 - If your target board is a ESP32, I recommend using IDF >= 5.2, since older versions may lead to IRAM overflow during build. Alternatively you can modify your sdkconfig-file (see [issue #1](https://github.com/cnadler86/micropython-camera-API/issues/1)).
 
 ## Plans for the future
-- [ ] imolrment structure in repo to include other boards like xiao sense
+
+- [ ] implement structure in repo to include other boards like xiao sense
 - [ ] harmonize properties to standard ones at API level, e.g. jpeg quality to the range 100=very good, 1/0= very bad
 - [ ] edge case: enable usage of pins such as i2c for other applications
-- [ ] provide examples in binary image with lfs-merge
+- [ ] provide examples in binary image with lfs-merge

src/modcamera.c

@@ -91,7 +91,7 @@ void mp_camera_hal_construct(
         // configure camera based on arguments
         self->camera_config.pixel_format = pixel_format;
         self->camera_config.frame_size = frame_size;        
-        self->camera_config.jpeg_quality = jpeg_quality;    //0-63 lower number means higher quality. TODO: Harmonization in API and Validation
+        self->camera_config.jpeg_quality = (int8_t)map(jpeg_quality,0,100,63,0);    //0-63 lower number means higher quality. TODO: Harmonization in API and Validation
         self->camera_config.pin_d0 = data_pins[0];
         self->camera_config.pin_d1 = data_pins[1];
         self->camera_config.pin_d2 = data_pins[2];
@@ -364,7 +364,7 @@ SENSOR_STATUS_GETSET_IN_RANGE(int, saturation, saturation, set_saturation, -2, 2
 SENSOR_STATUS_GETSET_IN_RANGE(int, sharpness, sharpness, set_sharpness, -2, 2);
 SENSOR_STATUS_GETSET(int, denoise, denoise, set_denoise);
 SENSOR_STATUS_GETSET(mp_camera_gainceiling_t, gainceiling, gainceiling, set_gainceiling);
-SENSOR_STATUS_GETSET(int, quality, quality, set_quality); //in_Range not needed since driver limits value
+SENSOR_STATUS_GETSET(int, quality_raw, quality, set_quality); //in_Range not needed since driver limits value. Raw to be used in wrapper funtion
 SENSOR_STATUS_GETSET(bool, colorbar, colorbar, set_colorbar);
 SENSOR_STATUS_GETSET(bool, whitebal, awb, set_whitebal);
 SENSOR_STATUS_GETSET(bool, gain_ctrl, agc, set_gain_ctrl);
@@ -384,6 +384,14 @@ SENSOR_STATUS_GETSET(bool, wpc, wpc, set_wpc);
 SENSOR_STATUS_GETSET(bool, raw_gma, raw_gma, set_raw_gma);
 SENSOR_STATUS_GETSET(bool, lenc, lenc, set_lenc);
 
+int mp_camera_hal_get_quality(mp_camera_obj_t *self){
+    return map(mp_camera_hal_get_quality_raw(self),63,0,0,100);
+}
+
+void mp_camera_hal_set_quality(mp_camera_obj_t *self,int quality){
+    mp_camera_hal_set_quality_raw(self,map(quality,0,100,63,0));
+}
+
 mp_camera_pixformat_t mp_camera_hal_get_pixel_format(mp_camera_obj_t *self) {
     return self->camera_config.pixel_format;
 }