[libcamera-devel] [PATCH v2 04/10] ipa: ipu3: Introduce a prepare() call to Algorithm
Kieran Bingham
kieran.bingham at ideasonboard.com
Fri Aug 13 10:48:37 CEST 2021
On 12/08/2021 17:52, Jean-Michel Hautbois wrote:
> When a new ipu3_uapi_params buffer is ready, IPAIPU3 receives a
> EventFillParams. When this is called, the algorithms should fill every
> field in the parameter structure they need to update.
>
> Demonstrate it by adding a loop on the prepare() call in IPAIPU3 and let
I don't think we need to demonstrate it, we 'implement' it ;-)
> the Grid algorithm update the grid field.
> This leads to removing this same update from the Awb algorithm.
>
> Signed-off-by: Jean-Michel Hautbois <jeanmichel.hautbois at ideasonboard.com>
> ---
> src/ipa/ipu3/algorithms/algorithm.h | 2 ++
> src/ipa/ipu3/algorithms/grid.cpp | 6 ++++++
> src/ipa/ipu3/algorithms/grid.h | 1 +
> src/ipa/ipu3/ipu3.cpp | 3 +++
> src/ipa/ipu3/ipu3_awb.cpp | 1 -
> 5 files changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/src/ipa/ipu3/algorithms/algorithm.h b/src/ipa/ipu3/algorithms/algorithm.h
> index c1b37276..b571f55a 100644
> --- a/src/ipa/ipu3/algorithms/algorithm.h
> +++ b/src/ipa/ipu3/algorithms/algorithm.h
> @@ -26,6 +26,8 @@ public:
> {
> return 0;
> }
> +
> + virtual void prepare([[maybe_unused]] IPAContext &context, [[maybe_unused]] ipu3_uapi_params ¶ms) {}
> };
>
> } /* namespace ipa::ipu3 */
> diff --git a/src/ipa/ipu3/algorithms/grid.cpp b/src/ipa/ipu3/algorithms/grid.cpp
> index 3578f41b..e42a760d 100644
> --- a/src/ipa/ipu3/algorithms/grid.cpp
> +++ b/src/ipa/ipu3/algorithms/grid.cpp
> @@ -78,6 +78,12 @@ int Grid::configure(IPAContext &context, const IPAConfigInfo &configInfo)
> return 0;
> }
>
> +void Grid::prepare(IPAContext &context, ipu3_uapi_params ¶ms)
> +{
> + /* Update the IPU3 parameters with new calculated grid */
I don't think it's a new calculated grid, as the grid is only calculated
during the configure phase.
I would say:
Update the IPU3 parameters with the configured grid
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> + params.acc_param.awb.config.grid = context.configuration.grid.bdsGrid;
> +}
> +
> } /* namespace ipa::ipu3::algorithms */
>
> } /* namespace libcamera */
> diff --git a/src/ipa/ipu3/algorithms/grid.h b/src/ipa/ipu3/algorithms/grid.h
> index b4a51b42..89399bd2 100644
> --- a/src/ipa/ipu3/algorithms/grid.h
> +++ b/src/ipa/ipu3/algorithms/grid.h
> @@ -19,6 +19,7 @@ public:
> ~Grid() = default;
>
> int configure(IPAContext &context, const IPAConfigInfo &configInfo) override;
> + void prepare(IPAContext &context, ipu3_uapi_params ¶ms) override;
> };
>
> } /* namespace ipa::ipu3::algorithms */
> diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp
> index ef7fec86..394a7a45 100644
> --- a/src/ipa/ipu3/ipu3.cpp
> +++ b/src/ipa/ipu3/ipu3.cpp
> @@ -305,6 +305,9 @@ void IPAIPU3::processControls([[maybe_unused]] unsigned int frame,
>
> void IPAIPU3::fillParams(unsigned int frame, ipu3_uapi_params *params)
> {
> + for (auto const &algo : algorithms_)
> + algo->prepare(context_, params_);
> +
> if (agcAlgo_->updateControls())
> awbAlgo_->updateWbParameters(params_, agcAlgo_->gamma());
>
> diff --git a/src/ipa/ipu3/ipu3_awb.cpp b/src/ipa/ipu3/ipu3_awb.cpp
> index 4ee5ee6f..911760b3 100644
> --- a/src/ipa/ipu3/ipu3_awb.cpp
> +++ b/src/ipa/ipu3/ipu3_awb.cpp
> @@ -159,7 +159,6 @@ void IPU3Awb::initialise(ipu3_uapi_params ¶ms, const Size &bdsOutputSize, st
> params.acc_param.awb.config.rgbs_thr_r = 8191;
> params.acc_param.awb.config.rgbs_thr_gb = 8191;
> params.acc_param.awb.config.rgbs_thr_b = 8191 | IPU3_UAPI_AWB_RGBS_THR_B_EN | IPU3_UAPI_AWB_RGBS_THR_B_INCL_SAT;
> - params.acc_param.awb.config.grid = bdsGrid;
> awbGrid_ = bdsGrid;
>
> params.use.acc_bnr = 1;
>
More information about the libcamera-devel
mailing list