ipa: simple: data: Add OV01A10 tuning file
diff mbox series

Message ID 20260501191400.985920-3-devve.3@gmail.com
State New
Headers show
Series
  • ipa: simple: data: Add OV01A10 tuning file
Related show

Commit Message

devve May 1, 2026, 7:13 p.m. UTC 
Add tuning data for the OmniVision OV01A10 sensor with black level,
colour correction matrix, and AGC parameters tuned to suppress
exposure oscillation on this sensor.

Signed-off-by: d3vv3 <devve.3@gmail.com>
---
 src/ipa/simple/data/meson.build  |  1 +
 src/ipa/simple/data/ov01a10.yaml | 25 +++++++++++++++++++++++++
 2 files changed, 26 insertions(+)
 create mode 100644 src/ipa/simple/data/ov01a10.yaml

Comments

Milan Zamazal May 5, 2026, 12:30 p.m. UTC | #1 
d3vv3 <devve.3@gmail.com> writes:

> Add tuning data for the OmniVision OV01A10 sensor with black level,
> colour correction matrix, and AGC parameters tuned to suppress
> exposure oscillation on this sensor.
>
> Signed-off-by: d3vv3 <devve.3@gmail.com>
> ---
>  src/ipa/simple/data/meson.build  |  1 +
>  src/ipa/simple/data/ov01a10.yaml | 25 +++++++++++++++++++++++++
>  2 files changed, 26 insertions(+)
>  create mode 100644 src/ipa/simple/data/ov01a10.yaml
>
> diff --git a/src/ipa/simple/data/meson.build b/src/ipa/simple/data/meson.build
> index 92795ee4..e6110320 100644
> --- a/src/ipa/simple/data/meson.build
> +++ b/src/ipa/simple/data/meson.build
> @@ -1,6 +1,7 @@
>  # SPDX-License-Identifier: CC0-1.0
>  
>  conf_files = files([
> +    'ov01a10.yaml',
>      'uncalibrated.yaml',
>  ])
>  
> diff --git a/src/ipa/simple/data/ov01a10.yaml b/src/ipa/simple/data/ov01a10.yaml
> new file mode 100644
> index 00000000..9b3b8007
> --- /dev/null
> +++ b/src/ipa/simple/data/ov01a10.yaml
> @@ -0,0 +1,25 @@
> +# SPDX-License-Identifier: CC0-1.0
> +%YAML 1.1
> +---
> +version: 1
> +algorithms:
> +  - BlackLevel:

Is the black level value known?

> +  - Awb:
> +      maxGainR: 2.0
> +      maxGainB: 2.0
> +      speed: 0.25
> +  - Ccm:
> +      ccms:
> +        - ct: 6500
> +          ccm: [  1.5000, -0.8000,  0.3500,
> +                 -0.1000,  0.9500,  0.0000,
> +                 -0.3000, -0.1000,  1.4500 ]

Where do these values (Awb, Ccm) come from?  It would be good to mention
this in the commit message.

And if I'm not mistaken, the sum of each of the matrix rows should be
1.0.

> +  - Adjust:
> +      gamma: 2.2
> +      contrast: 1.0
> +      saturation: 1.0
> +  - Agc:
> +      exposureTarget: 1.4
> +      hysteresis: 0.5
> +      stepDenominator: 20
> +...

Patch
diff mbox series 

diff --git a/src/ipa/simple/data/meson.build b/src/ipa/simple/data/meson.build
index 92795ee4..e6110320 100644
--- a/src/ipa/simple/data/meson.build
+++ b/src/ipa/simple/data/meson.build
@@ -1,6 +1,7 @@ 
 # SPDX-License-Identifier: CC0-1.0
 
 conf_files = files([
+    'ov01a10.yaml',
     'uncalibrated.yaml',
 ])
 
diff --git a/src/ipa/simple/data/ov01a10.yaml b/src/ipa/simple/data/ov01a10.yaml
new file mode 100644
index 00000000..9b3b8007
--- /dev/null
+++ b/src/ipa/simple/data/ov01a10.yaml
@@ -0,0 +1,25 @@ 
+# SPDX-License-Identifier: CC0-1.0
+%YAML 1.1
+---
+version: 1
+algorithms:
+  - BlackLevel:
+  - Awb:
+      maxGainR: 2.0
+      maxGainB: 2.0
+      speed: 0.25
+  - Ccm:
+      ccms:
+        - ct: 6500
+          ccm: [  1.5000, -0.8000,  0.3500,
+                 -0.1000,  0.9500,  0.0000,
+                 -0.3000, -0.1000,  1.4500 ]
+  - Adjust:
+      gamma: 2.2
+      contrast: 1.0
+      saturation: 1.0
+  - Agc:
+      exposureTarget: 1.4
+      hysteresis: 0.5
+      stepDenominator: 20
+...