[v2,3/4] ipa: simple: data: Add OV2740 tuning file
diff mbox series

Message ID 20260304232539.C3F581EA006C@mailuser.phl.internal
State Superseded
Headers show
Series
  • Untitled series #5821
Related show

Commit Message

Javier Tia March 4, 2026, 11:14 p.m. UTC
Add a tuning file for the OmniVision OV2740 sensor for the Simple/SoftISP
pipeline, tested on a ThinkPad X1 Carbon Gen 10 (Alder Lake, Intel IPU6).

No CCM is needed: AWB grey-world convergence gives R/G ~ 0.98 and
B/G ~ 0.99 under 6500 K lighting after the statistics normalization fix.

Signed-off-by: Javier Tia <floss@jetm.me>
---
 src/ipa/simple/data/meson.build |  1 +
 src/ipa/simple/data/ov2740.yaml | 13 +++++++++++++
 2 files changed, 14 insertions(+)
 create mode 100644 src/ipa/simple/data/ov2740.yaml

Comments

Milan Zamazal March 5, 2026, 7:14 p.m. UTC | #1
Hi Javier,

Javier Tia <floss@jetm.me> writes:

> Add a tuning file for the OmniVision OV2740 sensor for the Simple/SoftISP
> pipeline, tested on a ThinkPad X1 Carbon Gen 10 (Alder Lake, Intel IPU6).
>
> No CCM is needed: AWB grey-world convergence gives R/G ~ 0.98 and
> B/G ~ 0.99 under 6500 K lighting after the statistics normalization fix.

Does it make any sense to add a basically empty tuning file, equal to
uncalibrated.yaml?  I suppose we still need CCMs for the sensor to get
correct colours (in addition to having correct white balance).

> Signed-off-by: Javier Tia <floss@jetm.me>
> ---
>  src/ipa/simple/data/meson.build |  1 +
>  src/ipa/simple/data/ov2740.yaml | 13 +++++++++++++
>  2 files changed, 14 insertions(+)
>  create mode 100644 src/ipa/simple/data/ov2740.yaml
>
> diff --git a/src/ipa/simple/data/meson.build b/src/ipa/simple/data/meson.build
> index 92795ee4..e3e4de74 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([
> +    'ov2740.yaml',
>      'uncalibrated.yaml',
>  ])
>  
> diff --git a/src/ipa/simple/data/ov2740.yaml b/src/ipa/simple/data/ov2740.yaml
> new file mode 100644
> index 00000000..35cce1c1
> --- /dev/null
> +++ b/src/ipa/simple/data/ov2740.yaml
> @@ -0,0 +1,13 @@
> +# SPDX-License-Identifier: CC0-1.0
> +#
> +# OmniVision OV2740 tuning for libcamera Simple/SoftISP pipeline.
> +#
> +# Tested on ThinkPad X1 Carbon Gen 10 (Alder Lake) with Intel IPU6 ISYS.
> +%YAML 1.1
> +---
> +version: 1
> +algorithms:
> +  - BlackLevel:
> +  - Awb:
> +  - Adjust:
> +  - Agc:

Patch
diff mbox series

diff --git a/src/ipa/simple/data/meson.build b/src/ipa/simple/data/meson.build
index 92795ee4..e3e4de74 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([
+    'ov2740.yaml',
     'uncalibrated.yaml',
 ])
 
diff --git a/src/ipa/simple/data/ov2740.yaml b/src/ipa/simple/data/ov2740.yaml
new file mode 100644
index 00000000..35cce1c1
--- /dev/null
+++ b/src/ipa/simple/data/ov2740.yaml
@@ -0,0 +1,13 @@ 
+# SPDX-License-Identifier: CC0-1.0
+#
+# OmniVision OV2740 tuning for libcamera Simple/SoftISP pipeline.
+#
+# Tested on ThinkPad X1 Carbon Gen 10 (Alder Lake) with Intel IPU6 ISYS.
+%YAML 1.1
+---
+version: 1
+algorithms:
+  - BlackLevel:
+  - Awb:
+  - Adjust:
+  - Agc: