NekoCWD/libcamera
NekoCWD/libcamera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 7 Wiki Activity Actions

libcamera: rkisp1: Prepare for additional camera controls

Browse source 
Currently the rkisp1 pipeline handler only registers controls that are
related to the IPA. This patch prepares the rkisp1 pipeline-handler to
register camera controls which are not related to the IPA.

Hence, introduce an additional ControlInfoMap for IPA controls. These
controls will be merged together with the controls in the pipeline
handler (introduced subsequently) as part of updateControls() and
together will be registered during the registration of the camera.
This is similar to what IPU3 pipeline handler handles its controls.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Stefan Klug <stefan.klug@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
This commit is contained in:
Umang Jain 2024-09-26 15:06:22 +05:30
parent 1b0b90a332
commit 485f92ac06
1 changed files with 39 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

42
src/libcamera/pipeline/rkisp1/rkisp1.cpp
View file

@ -108,6 +108,8 @@ public:
std::unique_ptr<ipa::rkisp1::IPAProxyRkISP1> ipa_; std::unique_ptr<ipa::rkisp1::IPAProxyRkISP1> ipa_;
ControlInfoMap ipaControls_;
private: private:
void paramFilled(unsigned int frame, unsigned int bytesused); void paramFilled(unsigned int frame, unsigned int bytesused);
void setSensorControls(unsigned int frame, void setSensorControls(unsigned int frame,
@ -183,6 +185,8 @@ private:
int allocateBuffers(Camera *camera); int allocateBuffers(Camera *camera);
int freeBuffers(Camera *camera); int freeBuffers(Camera *camera);
int updateControls(RkISP1CameraData *data);
MediaDevice *media_; MediaDevice *media_;
std::unique_ptr<V4L2Subdevice> isp_; std::unique_ptr<V4L2Subdevice> isp_;
std::unique_ptr<V4L2VideoDevice> param_; std::unique_ptr<V4L2VideoDevice> param_;
@ -364,7 +368,7 @@ int RkISP1CameraData::loadIPA(unsigned int hwRevision)
} }
ret = ipa_->init({ ipaTuningFile, sensor_->model() }, hwRevision, ret = ipa_->init({ ipaTuningFile, sensor_->model() }, hwRevision,
sensorInfo, sensor_->controls(), &controlInfo_); sensorInfo, sensor_->controls(), &ipaControls_);
if (ret < 0) { if (ret < 0) {
LOG(RkISP1, Error) << "IPA initialization failure"; LOG(RkISP1, Error) << "IPA initialization failure";
return ret; return ret;
@ -842,12 +846,13 @@ int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
ipaConfig.sensorControls = data->sensor_->controls(); ipaConfig.sensorControls = data->sensor_->controls();
ipaConfig.paramFormat = paramFormat.fourcc; ipaConfig.paramFormat = paramFormat.fourcc;
ret = data->ipa_->configure(ipaConfig, streamConfig, &data->controlInfo_); ret = data->ipa_->configure(ipaConfig, streamConfig, &data->ipaControls_);
if (ret) { if (ret) {
LOG(RkISP1, Error) << "failed configuring IPA (" << ret << ")"; LOG(RkISP1, Error) << "failed configuring IPA (" << ret << ")";
return ret; return ret;
} }
return 0;
return updateControls(data);
} }
int PipelineHandlerRkISP1::exportFrameBuffers([[maybe_unused]] Camera *camera, Stream *stream, int PipelineHandlerRkISP1::exportFrameBuffers([[maybe_unused]] Camera *camera, Stream *stream,
@ -1114,6 +1119,35 @@ int PipelineHandlerRkISP1::initLinks(Camera *camera,
return 0; return 0;
} }
/**
* \brief Update the camera controls
* \param[in] data The camera data
*
* Compute the camera controls by calculating controls which the pipeline
* is reponsible for and merge them with the controls computed by the IPA.
*
* This function needs data->ipaControls_ to be refreshed when a new
* configuration is applied to the camera by the IPA configure() function.
*
* Always call this function after IPA configure() to make sure to have a
* properly refreshed IPA controls list.
*
* \return 0 on success or a negative error code otherwise
*/
int PipelineHandlerRkISP1::updateControls(RkISP1CameraData *data)
{
ControlInfoMap::Map controls;
/* Add the IPA registered controls to list of camera controls. */
for (const auto &ipaControl : data->ipaControls_)
controls[ipaControl.first] = ipaControl.second;
data->controlInfo_ = ControlInfoMap(std::move(controls),
controls::controls);
return 0;
}
int PipelineHandlerRkISP1::createCamera(MediaEntity *sensor) int PipelineHandlerRkISP1::createCamera(MediaEntity *sensor)
{ {
int ret; int ret;
@ -1150,6 +1184,8 @@ int PipelineHandlerRkISP1::createCamera(MediaEntity *sensor)
if (ret) if (ret)
return ret; return ret;
updateControls(data.get());
std::set<Stream *> streams{ std::set<Stream *> streams{
&data->mainPathStream_, &data->mainPathStream_,
&data->selfPathStream_, &data->selfPathStream_,