[{"id":35564,"web_url":"https://patchwork.libcamera.org/comment/35564/","msgid":"<CAEmqJPohMQU8UB5ZcOJ_rh0hbg0bWA4xyn=ze7D10kseN_g6qA@mail.gmail.com>","date":"2025-08-26T07:13:40","subject":"Re: [PATCH 4/6] ipa: rpi: vc4: Minor tuning changes","submitter":{"id":34,"url":"https://patchwork.libcamera.org/api/people/34/","name":"Naushir Patuck","email":"naush@raspberrypi.com"},"content":"Hi David,\n\nOn Fri, 22 Aug 2025 at 13:37, David Plowman\n<david.plowman@raspberrypi.com> wrote:\n>\n> Sharpening is reduced slightly for official Raspberry Pi cameras, and\n> exposure profiles made a bit more consistent.\n>\n> Denoise is reduced for the imx708 where it appears too strong.\n>\n> Signed-off-by: David Plowman <david.plowman@raspberrypi.com>\n\nReviewed-by: Naushir Patuck <naush@raspberrypi.com>\n\n> ---\n>  src/ipa/rpi/vc4/data/imx219.json            | 15 ++++++++++-----\n>  src/ipa/rpi/vc4/data/imx219_noir.json       | 15 ++++++++++-----\n>  src/ipa/rpi/vc4/data/imx296.json            |  8 ++++----\n>  src/ipa/rpi/vc4/data/imx296_mono.json       | 10 +++++-----\n>  src/ipa/rpi/vc4/data/imx477.json            | 15 ++++++++++-----\n>  src/ipa/rpi/vc4/data/imx477_noir.json       | 15 ++++++++++-----\n>  src/ipa/rpi/vc4/data/imx477_scientific.json | 15 ++++++++++-----\n>  src/ipa/rpi/vc4/data/imx708.json            | 21 +++++++++++++--------\n>  src/ipa/rpi/vc4/data/imx708_noir.json       | 21 +++++++++++++--------\n>  src/ipa/rpi/vc4/data/imx708_wide.json       | 21 +++++++++++++--------\n>  src/ipa/rpi/vc4/data/imx708_wide_noir.json  | 21 +++++++++++++--------\n>  src/ipa/rpi/vc4/data/ov5647.json            | 11 ++++++++---\n>  src/ipa/rpi/vc4/data/ov5647_noir.json       | 11 ++++++++---\n>  13 files changed, 127 insertions(+), 72 deletions(-)\n>\n> diff --git a/src/ipa/rpi/vc4/data/imx219.json b/src/ipa/rpi/vc4/data/imx219.json\n> index a020b12f..8098889e 100644\n> --- a/src/ipa/rpi/vc4/data/imx219.json\n> +++ b/src/ipa/rpi/vc4/data/imx219.json\n> @@ -162,17 +162,17 @@\n>                              \"normal\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 66666 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 10.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 10.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 10.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -675,7 +675,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.hdr\":\n> diff --git a/src/ipa/rpi/vc4/data/imx219_noir.json b/src/ipa/rpi/vc4/data/imx219_noir.json\n> index d8bc9639..c8c59da0 100644\n> --- a/src/ipa/rpi/vc4/data/imx219_noir.json\n> +++ b/src/ipa/rpi/vc4/data/imx219_noir.json\n> @@ -78,17 +78,17 @@\n>                              \"normal\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 66666 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 10.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 10.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 10.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -609,7 +609,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.hdr\":\n> diff --git a/src/ipa/rpi/vc4/data/imx296.json b/src/ipa/rpi/vc4/data/imx296.json\n> index 8f24ce5b..7c8589b2 100644\n> --- a/src/ipa/rpi/vc4/data/imx296.json\n> +++ b/src/ipa/rpi/vc4/data/imx296.json\n> @@ -159,18 +159,18 @@\n>                  {\n>                      \"normal\":\n>                      {\n> -                        \"shutter\": [ 100, 30000, 45000, 60000, 120000 ],\n> -                        \"gain\": [ 1.0, 1.0, 2.0, 4.0, 12.0 ]\n> +                        \"shutter\": [ 100, 30000, 45000, 60000, 66666 ],\n> +                        \"gain\": [ 1.0, 1.0, 2.0, 4.0, 16.0 ]\n>                      },\n>                      \"short\":\n>                      {\n> -                        \"shutter\": [ 100, 5000, 10000, 20000, 30000 ],\n> +                        \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n>                          \"gain\": [ 1.0, 2.0, 4.0, 8.0, 16.0 ]\n>                      },\n>                      \"long\":\n>                      {\n>                          \"shutter\": [ 1000, 30000, 60000, 90000, 120000 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      }\n>                  },\n>                  \"constraint_modes\":\n> diff --git a/src/ipa/rpi/vc4/data/imx296_mono.json b/src/ipa/rpi/vc4/data/imx296_mono.json\n> index fe331569..ea1ed768 100644\n> --- a/src/ipa/rpi/vc4/data/imx296_mono.json\n> +++ b/src/ipa/rpi/vc4/data/imx296_mono.json\n> @@ -62,18 +62,18 @@\n>                  {\n>                      \"normal\":\n>                      {\n> -                        \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 6.0 ]\n> +                        \"shutter\": [ 100, 10000, 30000, 60000, 66666 ],\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      },\n>                      \"short\":\n>                      {\n> -                        \"shutter\": [ 100, 5000, 10000, 20000, 120000 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 6.0 ]\n> +                        \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      },\n>                      \"long\":\n>                      {\n>                          \"shutter\": [ 1000, 30000, 60000, 90000, 120000 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      }\n>                  },\n>                  \"constraint_modes\":\n> diff --git a/src/ipa/rpi/vc4/data/imx477.json b/src/ipa/rpi/vc4/data/imx477.json\n> index fa25ee86..7d922d68 100644\n> --- a/src/ipa/rpi/vc4/data/imx477.json\n> +++ b/src/ipa/rpi/vc4/data/imx477.json\n> @@ -167,17 +167,17 @@\n>                              \"normal\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 66666 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -680,7 +680,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.hdr\":\n> diff --git a/src/ipa/rpi/vc4/data/imx477_noir.json b/src/ipa/rpi/vc4/data/imx477_noir.json\n> index 472f33fe..9de6ac4e 100644\n> --- a/src/ipa/rpi/vc4/data/imx477_noir.json\n> +++ b/src/ipa/rpi/vc4/data/imx477_noir.json\n> @@ -78,17 +78,17 @@\n>                              \"normal\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 66666 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -636,7 +636,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.hdr\":\n> diff --git a/src/ipa/rpi/vc4/data/imx477_scientific.json b/src/ipa/rpi/vc4/data/imx477_scientific.json\n> index 9dc32eb1..1e5d8433 100644\n> --- a/src/ipa/rpi/vc4/data/imx477_scientific.json\n> +++ b/src/ipa/rpi/vc4/data/imx477_scientific.json\n> @@ -173,17 +173,17 @@\n>                      \"normal\":\n>                      {\n>                          \"shutter\": [ 100, 10000, 30000, 60000, 66666 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      },\n>                      \"short\":\n>                      {\n> -                        \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n> +                        \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      },\n>                      \"long\":\n>                      {\n>                          \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                      }\n>                  },\n>                  \"constraint_modes\":\n> @@ -482,7 +482,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          }\n>      ]\n>  }\n> \\ No newline at end of file\n> diff --git a/src/ipa/rpi/vc4/data/imx708.json b/src/ipa/rpi/vc4/data/imx708.json\n> index e54ceff4..ec17243d 100644\n> --- a/src/ipa/rpi/vc4/data/imx708.json\n> +++ b/src/ipa/rpi/vc4/data/imx708.json\n> @@ -24,8 +24,8 @@\n>          {\n>              \"rpi.noise\":\n>              {\n> -                \"reference_constant\": 16.0,\n> -                \"reference_slope\": 4.0\n> +                \"reference_constant\": 0.0,\n> +                \"reference_slope\": 2.5\n>              }\n>          },\n>          {\n> @@ -169,18 +169,18 @@\n>                          {\n>                              \"normal\":\n>                              {\n> -                                \"shutter\": [ 100, 15000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 6.0 ]\n> +                                \"shutter\": [ 100, 15000, 30000, 60000, 66666 ],\n> +                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 16.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 6.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 1000, 30000, 60000, 90000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -611,7 +611,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.af\":\n> diff --git a/src/ipa/rpi/vc4/data/imx708_noir.json b/src/ipa/rpi/vc4/data/imx708_noir.json\n> index f351a180..b03906fb 100644\n> --- a/src/ipa/rpi/vc4/data/imx708_noir.json\n> +++ b/src/ipa/rpi/vc4/data/imx708_noir.json\n> @@ -24,8 +24,8 @@\n>          {\n>              \"rpi.noise\":\n>              {\n> -                \"reference_constant\": 16.0,\n> -                \"reference_slope\": 4.0\n> +                \"reference_constant\": 0.0,\n> +                \"reference_slope\": 2.5\n>              }\n>          },\n>          {\n> @@ -169,18 +169,18 @@\n>                          {\n>                              \"normal\":\n>                              {\n> -                                \"shutter\": [ 100, 15000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 6.0 ]\n> +                                \"shutter\": [ 100, 15000, 30000, 60000, 66666 ],\n> +                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 16.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 6.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 1000, 30000, 60000, 90000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -710,7 +710,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.af\":\n> diff --git a/src/ipa/rpi/vc4/data/imx708_wide.json b/src/ipa/rpi/vc4/data/imx708_wide.json\n> index bf1b122c..e94f2441 100644\n> --- a/src/ipa/rpi/vc4/data/imx708_wide.json\n> +++ b/src/ipa/rpi/vc4/data/imx708_wide.json\n> @@ -24,8 +24,8 @@\n>          {\n>              \"rpi.noise\":\n>              {\n> -                \"reference_constant\": 16.0,\n> -                \"reference_slope\": 4.0\n> +                \"reference_constant\": 0.0,\n> +                \"reference_slope\": 2.5\n>              }\n>          },\n>          {\n> @@ -159,18 +159,18 @@\n>                          {\n>                              \"normal\":\n>                              {\n> -                                \"shutter\": [ 100, 15000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 6.0 ]\n> +                                \"shutter\": [ 100, 15000, 30000, 60000, 66666 ],\n> +                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 16.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 6.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 1000, 30000, 60000, 90000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -610,7 +610,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.af\":\n> diff --git a/src/ipa/rpi/vc4/data/imx708_wide_noir.json b/src/ipa/rpi/vc4/data/imx708_wide_noir.json\n> index ea1c8c69..791e05bc 100644\n> --- a/src/ipa/rpi/vc4/data/imx708_wide_noir.json\n> +++ b/src/ipa/rpi/vc4/data/imx708_wide_noir.json\n> @@ -24,8 +24,8 @@\n>          {\n>              \"rpi.noise\":\n>              {\n> -                \"reference_constant\": 16.0,\n> -                \"reference_slope\": 4.0\n> +                \"reference_constant\": 0.0,\n> +                \"reference_slope\": 2.5\n>              }\n>          },\n>          {\n> @@ -159,18 +159,18 @@\n>                          {\n>                              \"normal\":\n>                              {\n> -                                \"shutter\": [ 100, 15000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 6.0 ]\n> +                                \"shutter\": [ 100, 15000, 30000, 60000, 66666 ],\n> +                                \"gain\": [ 1.0, 1.0, 2.0, 4.0, 16.0 ]\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 6.0 ]\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 1000, 30000, 60000, 90000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 16.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -601,7 +601,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.af\":\n> diff --git a/src/ipa/rpi/vc4/data/ov5647.json b/src/ipa/rpi/vc4/data/ov5647.json\n> index 40c6059c..ca561e59 100644\n> --- a/src/ipa/rpi/vc4/data/ov5647.json\n> +++ b/src/ipa/rpi/vc4/data/ov5647.json\n> @@ -166,13 +166,13 @@\n>                              },\n>                              \"short\":\n>                              {\n> -                                \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> +                                \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n>                                  \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n>                              },\n>                              \"long\":\n>                              {\n>                                  \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                                \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n>                              }\n>                          },\n>                          \"constraint_modes\":\n> @@ -676,7 +676,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          },\n>          {\n>              \"rpi.hdr\":\n> diff --git a/src/ipa/rpi/vc4/data/ov5647_noir.json b/src/ipa/rpi/vc4/data/ov5647_noir.json\n> index 488b7119..ae2eb71e 100644\n> --- a/src/ipa/rpi/vc4/data/ov5647_noir.json\n> +++ b/src/ipa/rpi/vc4/data/ov5647_noir.json\n> @@ -80,13 +80,13 @@\n>                      },\n>                      \"short\":\n>                      {\n> -                        \"shutter\": [ 100, 5000, 10000, 20000, 33333 ],\n> +                        \"shutter\": [ 100, 5000, 10000, 20000, 66666 ],\n>                          \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n>                      },\n>                      \"long\":\n>                      {\n>                          \"shutter\": [ 100, 10000, 30000, 60000, 120000 ],\n> -                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 12.0 ]\n> +                        \"gain\": [ 1.0, 2.0, 4.0, 6.0, 8.0 ]\n>                      }\n>                  },\n>                  \"constraint_modes\":\n> @@ -406,7 +406,12 @@\n>              }\n>          },\n>          {\n> -            \"rpi.sharpen\": { }\n> +            \"rpi.sharpen\":\n> +            {\n> +                \"threshold\": 0.75,\n> +                \"limit\": 0.5,\n> +                \"strength\": 1.0\n> +            }\n>          }\n>      ]\n>  }\n> \\ No newline at end of file\n> --\n> 2.39.5\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 254F6BEFBE\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 26 Aug 2025 07:14:19 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id C4ADB692EB;\n\tTue, 26 Aug 2025 09:14:18 +0200 (CEST)","from mail-vk1-xa2a.google.com (mail-vk1-xa2a.google.com\n\t[IPv6:2607:f8b0:4864:20::a2a])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8397769257\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 26 Aug 2025 09:14:17 +0200 (CEST)","by mail-vk1-xa2a.google.com with SMTP id\n\t71dfb90a1353d-54015704c15so117987e0c.0\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 26 Aug 2025 00:14:17 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=raspberrypi.com header.i=@raspberrypi.com\n\theader.b=\"Yl86NeQ/\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google; t=1756192456; x=1756797256;\n\tdarn=lists.libcamera.org; \n\th=cc:to:subject:message-id:date:from:in-reply-to:references\n\t:mime-version:from:to:cc:subject:date:message-id:reply-to;\n\tbh=a2ps/02Vt4qwnExJJcabX+EHgrWsuAWKKm2xkB8GmhU=;\n\tb=Yl86NeQ/Vkj4Iag3gy9WEh+r8NC+zWtcE+SkMiMX0N543+iUfO7N67kO0OJLU4Gt/u\n\tzKfiyWYad/ocX9Y5GSQxPv6D0JGlbb2jHld6g5pMqKNKAF4FdfuvgZ0il/PeGx0zrokX\n\tF5b7H/GB4XL2lZmhRZatuKe4l1G3fviwxm3yNzJzJ6x2EMuY+IUV2x9FtCUe0EFI0x2D\n\t10ACpTH9DEATZkOPt8A9fe0uTC8kfr9s49/xUYGJ5kfOp8BqnpRz2p1D5sCC3hhrCW1v\n\t6RbEU2GS/S3zBVkhNWbm9XmUlKGIVKgLdx10EZJW61IYKz+68Ns1x0ZPbcp1b5M7jtNh\n\tUGqA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1756192456; x=1756797256;\n\th=cc:to:subject:message-id:date:from:in-reply-to:references\n\t:mime-version:x-gm-message-state:from:to:cc:subject:date:message-id\n\t:reply-to;\n\tbh=a2ps/02Vt4qwnExJJcabX+EHgrWsuAWKKm2xkB8GmhU=;\n\tb=UMhdI0gYT+xDkLiBfpVqhA2jParPfyY1h6JyrAwp+6BKH5wFmOolBG22z1H1JM94pC\n\tzMKwpMkT2pgqi2SL7cn/nIpW94UpcWvO4a2dkO7UIP+700Gl+YOBjq7RDUgVkdKsLI+0\n\t/sLq3aSvQ/6Di35XELmvFgYfnr2+l3XfiJJzwE57TMTrHhe0D5UuDKQoTblHooSG7iC/\n\tPKeENDV3oq4CPj4b8Tb31L11llvfFw9b8MACxhPvL53kqk0mVboOw5/+knaVaUhv2N7V\n\tUkza6/ghMAWOA2skVBiX5BiVowgccTq5JyVe6u+QO2vCFCKQv59ymMHXnfkeKqZxoh1Z\n\t3xSQ==","X-Gm-Message-State":"AOJu0YxKW8hTLfbtlV87gVbRtInuSbs90dvuSSeqt0ocGCxwqVhJpGJ/\n\tjvu1u+sOBIT4BIPJNx7o4UC2GFu+46rnDNVCY3gxroWczlehWC/b1htGWaiHDr44jV2FHszsjSw\n\tvWti9ANjCszOVeywwH8mua73eav+e7Mf1MzS8RdkfPA==","X-Gm-Gg":"ASbGncvhOWIpyJwzCt/YkkiMLxZA2LNnCmpJqpDd6nc+Sda1gJtNJ9Z0UOQVkWlRToQ\n\tGmh7Jl3brdO0+RX4Mh4+JjGuh34JzZyTBkoHRLz5sOYuuXdholbo4SN4vgmFZjB4WEM9iHDMfHh\n\tzea8mdpzmTHbfBf/gplo/k82bFXhckyhjEDJON+FTM5XNtpKp/v4C2IYmHqY96JiDzo6UlYWhGU\n\taGVNknaxrw77FMbAvZO9D1SitzcJnjur8SwhT0=","X-Google-Smtp-Source":"AGHT+IGXG0zxoO5kViuXf2CW+at6RyikcdxgF9e5yBU37oAFEW4blQcMwMt3Vu4wZX6+9LAmS88mhhJhtEQi6V4kPzo=","X-Received":"by 2002:a05:6102:511f:b0:522:57ef:c403 with SMTP id\n\tada2fe7eead31-52257ff0c15mr725821137.6.1756192456232; Tue, 26 Aug 2025\n\t00:14:16 -0700 (PDT)","MIME-Version":"1.0","References":"<20250822123709.16966-1-david.plowman@raspberrypi.com>\n\t<20250822123709.16966-5-david.plowman@raspberrypi.com>","In-Reply-To":"<20250822123709.16966-5-david.plowman@raspberrypi.com>","From":"Naushir Patuck <naush@raspberrypi.com>","Date":"Tue, 26 Aug 2025 08:13:40 +0100","X-Gm-Features":"Ac12FXydZPubBS46rlvFyqUB799LbXzq9JA571v491MlHH51JSi9lydRjd6GVBA","Message-ID":"<CAEmqJPohMQU8UB5ZcOJ_rh0hbg0bWA4xyn=ze7D10kseN_g6qA@mail.gmail.com>","Subject":"Re: [PATCH 4/6] ipa: rpi: vc4: Minor tuning changes","To":"David Plowman <david.plowman@raspberrypi.com>","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"UTF-8\"","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]