diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2022-09-30 04:08:47 +0300 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2022-10-01 00:01:56 +0300 |
commit | 79f0fc937d95cbf1bd39f04dfd8b83206bda5098 (patch) | |
tree | d964865780525e85a6286504cb9bf2662fe93afa /src/ipa/rkisp1/algorithms/dpf.cpp | |
parent | c97cf467ea79ed886dc549bff6837900427ca11e (diff) |
ipa: rkisp1: Remove initialized_ flags from algorithms
Multiple algorithms have an initialized_ flag that they set to true at
the end of the init() function, and check at the beginning of prepare()
to skip preparation. This serves no real purpose, as the flag can only
be false if init() fails, in which case the IPA module initialization as
a whole will fail.
Drop the initialized_ flags.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'src/ipa/rkisp1/algorithms/dpf.cpp')
-rw-r--r-- | src/ipa/rkisp1/algorithms/dpf.cpp | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/ipa/rkisp1/algorithms/dpf.cpp b/src/ipa/rkisp1/algorithms/dpf.cpp index f7bc371d..82d4f7fe 100644 --- a/src/ipa/rkisp1/algorithms/dpf.cpp +++ b/src/ipa/rkisp1/algorithms/dpf.cpp @@ -35,7 +35,7 @@ namespace ipa::rkisp1::algorithms { LOG_DEFINE_CATEGORY(RkISP1Dpf) Dpf::Dpf() - : initialized_(false), config_({}), strengthConfig_({}) + : config_({}), strengthConfig_({}) { } @@ -166,8 +166,6 @@ int Dpf::init([[maybe_unused]] IPAContext &context, strengthConfig_.g = fSObject["g"].get<uint16_t>(64); strengthConfig_.b = fSObject["b"].get<uint16_t>(64); - initialized_ = true; - return 0; } @@ -219,9 +217,6 @@ void Dpf::queueRequest(IPAContext &context, void Dpf::prepare(IPAContext &context, const uint32_t frame, IPAFrameContext &frameContext, rkisp1_params_cfg *params) { - if (!initialized_) - return; - if (frame == 0) { params->others.dpf_config = config_; params->others.dpf_strength_config = strengthConfig_; |