| Message ID | 20251024085130.995967-35-stefan.klug@ideasonboard.com |
|---|---|
| State | New |
| Headers | show |
| Series |
|
| Related | show |
Quoting Stefan Klug (2025-10-24 17:50:58) > Now that the rkisp1 is well synchronized, we can safely increase the > regulation speed and reduce the number of startup frames. With current > settings, the results based on the stats for frame 0 are active on frame > 5. Setting the startup frames to 7 leaves room for one additional lost > frame. > > Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com> Looks good to me. Reviewed-by: Paul Elder <paul.elder@ideasonboard.com> > --- > src/ipa/rkisp1/algorithms/agc.cpp | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/ipa/rkisp1/algorithms/agc.cpp b/src/ipa/rkisp1/algorithms/agc.cpp > index 71ad3a01d90c..e8f2324797d4 100644 > --- a/src/ipa/rkisp1/algorithms/agc.cpp > +++ b/src/ipa/rkisp1/algorithms/agc.cpp > @@ -122,6 +122,9 @@ uint8_t Agc::computeHistogramPredivider(const Size &size, > Agc::Agc() > { > supportsRaw_ = true; > + /* rkisp1 is well synchronized, increase the speed. */ > + regulationSpeed_ = 0.6; > + numStartupFrames_ = 7; > } > > /** > -- > 2.48.1 >
diff --git a/src/ipa/rkisp1/algorithms/agc.cpp b/src/ipa/rkisp1/algorithms/agc.cpp index 71ad3a01d90c..e8f2324797d4 100644 --- a/src/ipa/rkisp1/algorithms/agc.cpp +++ b/src/ipa/rkisp1/algorithms/agc.cpp @@ -122,6 +122,9 @@ uint8_t Agc::computeHistogramPredivider(const Size &size, Agc::Agc() { supportsRaw_ = true; + /* rkisp1 is well synchronized, increase the speed. */ + regulationSpeed_ = 0.6; + numStartupFrames_ = 7; } /**
Now that the rkisp1 is well synchronized, we can safely increase the regulation speed and reduce the number of startup frames. With current settings, the results based on the stats for frame 0 are active on frame 5. Setting the startup frames to 7 leaves room for one additional lost frame. Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com> --- src/ipa/rkisp1/algorithms/agc.cpp | 3 +++ 1 file changed, 3 insertions(+)