Mirror/libcamera
1
0
Fork 0
mirror of https://git.libcamera.org/libcamera/libcamera.git synced 2025-07-24 09:05:06 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

libcamera: ipu3: Always import buffers for ImgU sinks

Browse source 
When the IPU3 pipeline was first developed sinks of the ImgU that where
not active still needed to have buffers allocated to allow streaming to
start. This is no longer true, it's enough that the sinks have imported
buffers to allow streaming to start. As we already need to import
buffers for stream that are active we can align the two cases and always
import buffers.

With this there is no longer a reason to store the allocated
FrameBuffers to keep them alive and the vector tracking them can be
removed.

Signed-off-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
This commit is contained in:
Niklas Söderlund 2020-06-27 00:31:40 +02:00
parent dcb3e7e2df
commit 72e36e8d8d
1 changed files with 7 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

26
src/libcamera/pipeline/ipu3/ipu3.cpp
View file

@ -49,7 +49,6 @@ public:
V4L2VideoDevice *dev;
unsigned int pad;
std::string name;
std::vector<std::unique_ptr<FrameBuffer>> buffers;
};
ImgUDevice()
@ -1124,9 +1123,6 @@ int ImgUDevice::configureOutput(ImgUOutput *output,
*/
int ImgUDevice::allocateBuffers(IPU3CameraData *data, unsigned int bufferCount)
{
IPU3Stream *outStream = &data->outStream_;
IPU3Stream *vfStream = &data->vfStream_;
/* Share buffers between CIO2 output and ImgU input. */
int ret = input_->importBuffers(bufferCount);
if (ret) {
@ -1148,27 +1144,19 @@ int ImgUDevice::allocateBuffers(IPU3CameraData *data, unsigned int bufferCount)
}
/*
* Allocate buffers for both outputs. If an output is active, prepare
* for buffer import, otherwise allocate internal buffers. Use the same
* number of buffers in either case.
* Import buffers for all outputs, regardless of whether the
* corresponding stream is active or inactive, as the driver needs
* buffers to be requested on the V4L2 devices in order to operate.
*/
if (outStream->active_)
ret = output_.dev->importBuffers(bufferCount);
else
ret = output_.dev->allocateBuffers(bufferCount,
&output_.buffers);
ret = output_.dev->importBuffers(bufferCount);
if (ret < 0) {
LOG(IPU3, Error) << "Failed to allocate ImgU output buffers";
LOG(IPU3, Error) << "Failed to import ImgU output buffers";
goto error;
}
if (vfStream->active_)
ret = viewfinder_.dev->importBuffers(bufferCount);
else
ret = viewfinder_.dev->allocateBuffers(bufferCount,
&viewfinder_.buffers);
ret = viewfinder_.dev->importBuffers(bufferCount);
if (ret < 0) {
LOG(IPU3, Error) << "Failed to allocate ImgU viewfinder buffers";
LOG(IPU3, Error) << "Failed to import ImgU viewfinder buffers";
goto error;
}