| Message ID | 20220303234956.1463551-8-djrscally@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series |
|
| Related | show |
Hi ! Thanks for the patch ! On 04/03/2022 00:49, Daniel Scally wrote: > Now that we have added lens controls, rename the existiing s/existiing/existing > member of the class to clarify that it relates to the sensor's > controls. > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Signed-off-by: Daniel Scally <djrscally@gmail.com> With the small typo corrected (can be done when applying): Reviewed-by: Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com> > --- > Changes in v5: > > - None > > Changes since v4: > > - None > > Changes in v3: > > - New patch > > src/ipa/ipu3/ipu3.cpp | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp > index 008f93a7..e539d875 100644 > --- a/src/ipa/ipu3/ipu3.cpp > +++ b/src/ipa/ipu3/ipu3.cpp > @@ -162,7 +162,7 @@ private: > > std::map<unsigned int, MappedFrameBuffer> buffers_; > > - ControlInfoMap ctrls_; > + ControlInfoMap sensorCtrls_; > ControlInfoMap lensCtrls_; > > IPACameraSensorInfo sensorInfo_; > @@ -457,22 +457,22 @@ int IPAIPU3::configure(const IPAConfigInfo &configInfo, > * Compute the sensor V4L2 controls to be used by the algorithms and > * to be set on the sensor. > */ > - ctrls_ = configInfo.sensorControls; > + sensorCtrls_ = configInfo.sensorControls; > > - const auto itExp = ctrls_.find(V4L2_CID_EXPOSURE); > - if (itExp == ctrls_.end()) { > + const auto itExp = sensorCtrls_.find(V4L2_CID_EXPOSURE); > + if (itExp == sensorCtrls_.end()) { > LOG(IPAIPU3, Error) << "Can't find exposure control"; > return -EINVAL; > } > > - const auto itGain = ctrls_.find(V4L2_CID_ANALOGUE_GAIN); > - if (itGain == ctrls_.end()) { > + const auto itGain = sensorCtrls_.find(V4L2_CID_ANALOGUE_GAIN); > + if (itGain == sensorCtrls_.end()) { > LOG(IPAIPU3, Error) << "Can't find gain control"; > return -EINVAL; > } > > - const auto itVBlank = ctrls_.find(V4L2_CID_VBLANK); > - if (itVBlank == ctrls_.end()) { > + const auto itVBlank = sensorCtrls_.find(V4L2_CID_VBLANK); > + if (itVBlank == sensorCtrls_.end()) { > LOG(IPAIPU3, Error) << "Can't find VBLANK control"; > return -EINVAL; > } > @@ -497,10 +497,10 @@ int IPAIPU3::configure(const IPAConfigInfo &configInfo, > initAfGrid(configInfo.bdsOutputSize); > > /* Update the camera controls using the new sensor settings. */ > - updateControls(sensorInfo_, ctrls_, ipaControls); > + updateControls(sensorInfo_, sensorCtrls_, ipaControls); > > /* Update the IPASessionConfiguration using the sensor settings. */ > - updateSessionConfiguration(ctrls_); > + updateSessionConfiguration(sensorCtrls_); > > for (auto const &algo : algorithms_) { > int ret = algo->configure(context_, configInfo); > @@ -699,7 +699,7 @@ void IPAIPU3::setControls(unsigned int frame) > exposure_ = context_.frameContext.agc.exposure; > gain_ = camHelper_->gainCode(context_.frameContext.agc.gain); > > - ControlList ctrls(ctrls_); > + ControlList ctrls(sensorCtrls_); > ctrls.set(V4L2_CID_EXPOSURE, static_cast<int32_t>(exposure_)); > ctrls.set(V4L2_CID_ANALOGUE_GAIN, static_cast<int32_t>(gain_)); > op.sensorControls = ctrls;
diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp index 008f93a7..e539d875 100644 --- a/src/ipa/ipu3/ipu3.cpp +++ b/src/ipa/ipu3/ipu3.cpp @@ -162,7 +162,7 @@ private: std::map<unsigned int, MappedFrameBuffer> buffers_; - ControlInfoMap ctrls_; + ControlInfoMap sensorCtrls_; ControlInfoMap lensCtrls_; IPACameraSensorInfo sensorInfo_; @@ -457,22 +457,22 @@ int IPAIPU3::configure(const IPAConfigInfo &configInfo, * Compute the sensor V4L2 controls to be used by the algorithms and * to be set on the sensor. */ - ctrls_ = configInfo.sensorControls; + sensorCtrls_ = configInfo.sensorControls; - const auto itExp = ctrls_.find(V4L2_CID_EXPOSURE); - if (itExp == ctrls_.end()) { + const auto itExp = sensorCtrls_.find(V4L2_CID_EXPOSURE); + if (itExp == sensorCtrls_.end()) { LOG(IPAIPU3, Error) << "Can't find exposure control"; return -EINVAL; } - const auto itGain = ctrls_.find(V4L2_CID_ANALOGUE_GAIN); - if (itGain == ctrls_.end()) { + const auto itGain = sensorCtrls_.find(V4L2_CID_ANALOGUE_GAIN); + if (itGain == sensorCtrls_.end()) { LOG(IPAIPU3, Error) << "Can't find gain control"; return -EINVAL; } - const auto itVBlank = ctrls_.find(V4L2_CID_VBLANK); - if (itVBlank == ctrls_.end()) { + const auto itVBlank = sensorCtrls_.find(V4L2_CID_VBLANK); + if (itVBlank == sensorCtrls_.end()) { LOG(IPAIPU3, Error) << "Can't find VBLANK control"; return -EINVAL; } @@ -497,10 +497,10 @@ int IPAIPU3::configure(const IPAConfigInfo &configInfo, initAfGrid(configInfo.bdsOutputSize); /* Update the camera controls using the new sensor settings. */ - updateControls(sensorInfo_, ctrls_, ipaControls); + updateControls(sensorInfo_, sensorCtrls_, ipaControls); /* Update the IPASessionConfiguration using the sensor settings. */ - updateSessionConfiguration(ctrls_); + updateSessionConfiguration(sensorCtrls_); for (auto const &algo : algorithms_) { int ret = algo->configure(context_, configInfo); @@ -699,7 +699,7 @@ void IPAIPU3::setControls(unsigned int frame) exposure_ = context_.frameContext.agc.exposure; gain_ = camHelper_->gainCode(context_.frameContext.agc.gain); - ControlList ctrls(ctrls_); + ControlList ctrls(sensorCtrls_); ctrls.set(V4L2_CID_EXPOSURE, static_cast<int32_t>(exposure_)); ctrls.set(V4L2_CID_ANALOGUE_GAIN, static_cast<int32_t>(gain_)); op.sensorControls = ctrls;