[libcamera-devel,libcamera-devel,4/5] ipa: rkisp1: Add ov5640 configuration file
diff mbox series

Message ID 20220523092435.475510-5-fsylvestre@baylibre.com
State Superseded
Headers show
Series
  • Add configuration file support for rkisp1 blc algo
Related show

Commit Message

Florian Sylvestre May 23, 2022, 9:24 a.m. UTC 
Add the ov5640 tuning file containing default values for 'black level
correction' algorithm.

Signed-off-by: Florian Sylvestre <fsylvestre@baylibre.com>
---
 src/ipa/rkisp1/data/meson.build | 8 ++++++++
 src/ipa/rkisp1/data/ov5640.yaml | 8 ++++++++
 src/ipa/rkisp1/meson.build      | 1 +
 3 files changed, 17 insertions(+)
 create mode 100644 src/ipa/rkisp1/data/meson.build
 create mode 100644 src/ipa/rkisp1/data/ov5640.yaml

Comments

Laurent Pinchart May 25, 2022, 11:25 p.m. UTC | #1 
Hi Florian,

Thank you for the patch.

"tuning file" in the subject line.

On Mon, May 23, 2022 at 11:24:34AM +0200, Florian Sylvestre via libcamera-devel wrote:
> Add the ov5640 tuning file containing default values for 'black level
> correction' algorithm.
> 
> Signed-off-by: Florian Sylvestre <fsylvestre@baylibre.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  src/ipa/rkisp1/data/meson.build | 8 ++++++++
>  src/ipa/rkisp1/data/ov5640.yaml | 8 ++++++++
>  src/ipa/rkisp1/meson.build      | 1 +
>  3 files changed, 17 insertions(+)
>  create mode 100644 src/ipa/rkisp1/data/meson.build
>  create mode 100644 src/ipa/rkisp1/data/ov5640.yaml
> 
> diff --git a/src/ipa/rkisp1/data/meson.build b/src/ipa/rkisp1/data/meson.build
> new file mode 100644
> index 00000000..2656bb2e
> --- /dev/null
> +++ b/src/ipa/rkisp1/data/meson.build
> @@ -0,0 +1,8 @@
> +# SPDX-License-Identifier: CC0-1.0
> +
> +conf_files = files([
> +    'ov5640.yaml',
> +])
> +
> +install_data(conf_files,
> +             install_dir : ipa_data_dir / 'rkisp1')
> diff --git a/src/ipa/rkisp1/data/ov5640.yaml b/src/ipa/rkisp1/data/ov5640.yaml
> new file mode 100644
> index 00000000..d8e56463
> --- /dev/null
> +++ b/src/ipa/rkisp1/data/ov5640.yaml
> @@ -0,0 +1,8 @@
> +# SPDX-License-Identifier: CC0-1.0
> +---
> +BlackLevelCorrection:
> +  R:  256
> +  Gr: 256
> +  Gb: 256
> +  B:  256
> +...
> diff --git a/src/ipa/rkisp1/meson.build b/src/ipa/rkisp1/meson.build
> index 8c822fbb..ccb84b27 100644
> --- a/src/ipa/rkisp1/meson.build
> +++ b/src/ipa/rkisp1/meson.build
> @@ -1,6 +1,7 @@
>  # SPDX-License-Identifier: CC0-1.0
>  
>  subdir('algorithms')
> +subdir('data')
>  
>  ipa_name = 'ipa_rkisp1'
>
Nicolas Dufresne via libcamera-devel May 27, 2022, 8:27 a.m. UTC | #2 
Hi Florian,

On Mon, May 23, 2022 at 11:24:34AM +0200, Florian Sylvestre via libcamera-devel wrote:
> Add the ov5640 tuning file containing default values for 'black level
> correction' algorithm.
> 
> Signed-off-by: Florian Sylvestre <fsylvestre@baylibre.com>

Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>

> ---
>  src/ipa/rkisp1/data/meson.build | 8 ++++++++
>  src/ipa/rkisp1/data/ov5640.yaml | 8 ++++++++
>  src/ipa/rkisp1/meson.build      | 1 +
>  3 files changed, 17 insertions(+)
>  create mode 100644 src/ipa/rkisp1/data/meson.build
>  create mode 100644 src/ipa/rkisp1/data/ov5640.yaml
> 
> diff --git a/src/ipa/rkisp1/data/meson.build b/src/ipa/rkisp1/data/meson.build
> new file mode 100644
> index 00000000..2656bb2e
> --- /dev/null
> +++ b/src/ipa/rkisp1/data/meson.build
> @@ -0,0 +1,8 @@
> +# SPDX-License-Identifier: CC0-1.0
> +
> +conf_files = files([
> +    'ov5640.yaml',
> +])
> +
> +install_data(conf_files,
> +             install_dir : ipa_data_dir / 'rkisp1')
> diff --git a/src/ipa/rkisp1/data/ov5640.yaml b/src/ipa/rkisp1/data/ov5640.yaml
> new file mode 100644
> index 00000000..d8e56463
> --- /dev/null
> +++ b/src/ipa/rkisp1/data/ov5640.yaml
> @@ -0,0 +1,8 @@
> +# SPDX-License-Identifier: CC0-1.0
> +---
> +BlackLevelCorrection:
> +  R:  256
> +  Gr: 256
> +  Gb: 256
> +  B:  256
> +...
> diff --git a/src/ipa/rkisp1/meson.build b/src/ipa/rkisp1/meson.build
> index 8c822fbb..ccb84b27 100644
> --- a/src/ipa/rkisp1/meson.build
> +++ b/src/ipa/rkisp1/meson.build
> @@ -1,6 +1,7 @@
>  # SPDX-License-Identifier: CC0-1.0
>  
>  subdir('algorithms')
> +subdir('data')
>  
>  ipa_name = 'ipa_rkisp1'
>  
> -- 
> 2.34.1
>

Patch
diff mbox series 

diff --git a/src/ipa/rkisp1/data/meson.build b/src/ipa/rkisp1/data/meson.build
new file mode 100644
index 00000000..2656bb2e
--- /dev/null
+++ b/src/ipa/rkisp1/data/meson.build
@@ -0,0 +1,8 @@ 
+# SPDX-License-Identifier: CC0-1.0
+
+conf_files = files([
+    'ov5640.yaml',
+])
+
+install_data(conf_files,
+             install_dir : ipa_data_dir / 'rkisp1')
diff --git a/src/ipa/rkisp1/data/ov5640.yaml b/src/ipa/rkisp1/data/ov5640.yaml
new file mode 100644
index 00000000..d8e56463
--- /dev/null
+++ b/src/ipa/rkisp1/data/ov5640.yaml
@@ -0,0 +1,8 @@ 
+# SPDX-License-Identifier: CC0-1.0
+---
+BlackLevelCorrection:
+  R:  256
+  Gr: 256
+  Gb: 256
+  B:  256
+...
diff --git a/src/ipa/rkisp1/meson.build b/src/ipa/rkisp1/meson.build
index 8c822fbb..ccb84b27 100644
--- a/src/ipa/rkisp1/meson.build
+++ b/src/ipa/rkisp1/meson.build
@@ -1,6 +1,7 @@ 
 # SPDX-License-Identifier: CC0-1.0
 
 subdir('algorithms')
+subdir('data')
 
 ipa_name = 'ipa_rkisp1'