@@ -97,16 +97,16 @@ static int vimc_sen_get_fmt(struct v4l2_subdev *sd,
static void vimc_sen_tpg_s_format(struct vimc_sen_device *vsen)
{
u32 pixelformat = vsen->ved.stream->producer_pixfmt;
- const struct v4l2_format_info *pix_info;
-
- pix_info = v4l2_format_info(pixelformat);
+ unsigned int i;
tpg_reset_source(&vsen->tpg, vsen->mbus_format.width,
vsen->mbus_format.height, vsen->mbus_format.field);
- tpg_s_bytesperline(&vsen->tpg, 0,
- vsen->mbus_format.width * pix_info->bpp[0]);
tpg_s_buf_height(&vsen->tpg, vsen->mbus_format.height);
tpg_s_fourcc(&vsen->tpg, pixelformat);
+
+ for (i = 0; i < tpg_g_planes(&vsen->tpg); i++)
+ tpg_s_bytesperline(&vsen->tpg, i, vsen->frame.bytesperline[i]);
+
/* TODO: add support for V4L2_FIELD_ALTERNATE */
tpg_s_field(&vsen->tpg, vsen->mbus_format.field, false);
tpg_s_colorspace(&vsen->tpg, vsen->mbus_format.colorspace);
@@ -182,8 +182,12 @@ static struct vimc_frame *vimc_sen_process_frame(struct vimc_ent_device *ved,
{
struct vimc_sen_device *vsen = container_of(ved, struct vimc_sen_device,
ved);
+ unsigned int i;
+
+ for (i = 0; i < tpg_g_planes(&vsen->tpg); i++)
+ tpg_fill_plane_buffer(&vsen->tpg, 0, i,
+ vsen->frame.plane_addr[i]);
- tpg_fill_plane_buffer(&vsen->tpg, 0, 0, vsen->frame.plane_addr[0]);
return &vsen->frame;
}
@@ -191,36 +195,39 @@ static int vimc_sen_s_stream(struct v4l2_subdev *sd, int enable)
{
struct vimc_sen_device *vsen =
container_of(sd, struct vimc_sen_device, sd);
+ unsigned int i, ret = 0;
if (enable) {
- u32 pixelformat = vsen->ved.stream->producer_pixfmt;
- const struct v4l2_format_info *pix_info;
- unsigned int frame_size;
-
/* Calculate the frame size */
- pix_info = v4l2_format_info(pixelformat);
- frame_size = vsen->mbus_format.width * pix_info->bpp[0] *
- vsen->mbus_format.height;
-
+ vimc_fill_frame(&vsen->frame, vsen->ved.stream->producer_pixfmt,
+ vsen->mbus_format.width,
+ vsen->mbus_format.height,
+ vsen->ved.stream->multiplanar);
/*
* Allocate the frame buffer. Use vmalloc to be able to
* allocate a large amount of memory
*/
- vsen->frame.plane_addr[0] = vmalloc(frame_size);
- if (!vsen->frame.plane_addr[0])
- return -ENOMEM;
+ for (i = 0; i < vsen->frame.num_planes; i++) {
+ vsen->frame.plane_addr[i] =
+ vmalloc(vsen->frame.sizeimage[i]);
+ if (!vsen->frame.plane_addr[i]) {
+ ret = -ENOMEM;
+ goto free_planes;
+ }
+ }
/* configure the test pattern generator */
vimc_sen_tpg_s_format(vsen);
} else {
-
- vfree(vsen->frame.plane_addr[0]);
- vsen->frame.plane_addr[0] = NULL;
- return 0;
+free_planes:
+ for (i = 0; i < vsen->frame.num_planes; i++) {
+ vfree(vsen->frame.plane_addr[i]);
+ vsen->frame.plane_addr[i] = NULL;
+ }
}
- return 0;
+ return ret;
}
static const struct v4l2_subdev_core_ops vimc_sen_core_ops = {
This commit adapts vimc-sensor to handle multiplanar pixel formats, adapting the memory allocation and TPG configuration. Signed-off-by: André Almeida <andrealmeid@collabora.com> --- Changes in v2: - Fix alignment issues - Remove unnecessary variable assignment - Reuse and share the code to free the memory of planes with a goto drivers/media/platform/vimc/vimc-sensor.c | 51 +++++++++++++---------- 1 file changed, 29 insertions(+), 22 deletions(-)