openwrt/target/linux/bcm27xx/patches-5.10/950-0491-media-i2c-imx477-Remove-internal-v4l2_mbus_framefmt-.patch
Álvaro Fernández Rojas 8299d1f057 bcm27xx: add kernel 5.10 support
Rebased RPi foundation patches on linux 5.10.59, removed applied and reverted
patches, wireless patches and defconfig patches.

bcm2708: boot tested on RPi B+ v1.2
bcm2709: boot tested on RPi 4B v1.1 4G
bcm2711: boot tested on RPi 4B v1.1 4G

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
2021-08-21 19:07:07 +02:00

71 lines
2.4 KiB
Diff

From 013bd52fb83d145e90b8678c004ae1b32854d7f7 Mon Sep 17 00:00:00 2001
From: Naushir Patuck <naush@raspberrypi.com>
Date: Fri, 19 Feb 2021 10:30:49 +0000
Subject: [PATCH] media: i2c: imx477: Remove internal
v4l2_mbus_framefmt from the state
The only field in this struct that is used is the format code, so
replace the struct with this single field.
Save the format code in imx477_set_pad_format() when setting up a new
mode so that imx477_get_pad_format() performs the right lookup.
Otherwise, this caused a bug where the mode lookup occurred on the
12-bit table rather than the 10-bit table.
Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
---
drivers/media/i2c/imx477.c | 19 ++++---------------
1 file changed, 4 insertions(+), 15 deletions(-)
--- a/drivers/media/i2c/imx477.c
+++ b/drivers/media/i2c/imx477.c
@@ -1073,7 +1073,7 @@ struct imx477 {
struct v4l2_subdev sd;
struct media_pad pad[NUM_PADS];
- struct v4l2_mbus_framefmt fmt;
+ unsigned int fmt_code;
struct clk *xclk;
u32 xclk_freq;
@@ -1235,21 +1235,9 @@ static u32 imx477_get_format_code(struct
static void imx477_set_default_format(struct imx477 *imx477)
{
- struct v4l2_mbus_framefmt *fmt = &imx477->fmt;
-
/* Set default mode to max resolution */
imx477->mode = &supported_modes_12bit[0];
-
- fmt->code = MEDIA_BUS_FMT_SRGGB12_1X12;
- fmt->colorspace = V4L2_COLORSPACE_SRGB;
- fmt->ycbcr_enc = V4L2_MAP_YCBCR_ENC_DEFAULT(fmt->colorspace);
- fmt->quantization = V4L2_MAP_QUANTIZATION_DEFAULT(true,
- fmt->colorspace,
- fmt->ycbcr_enc);
- fmt->xfer_func = V4L2_MAP_XFER_FUNC_DEFAULT(fmt->colorspace);
- fmt->width = imx477->mode->width;
- fmt->height = imx477->mode->height;
- fmt->field = V4L2_FIELD_NONE;
+ imx477->fmt_code = MEDIA_BUS_FMT_SRGGB12_1X12;
}
static int imx477_open(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh)
@@ -1520,7 +1508,7 @@ static int imx477_get_pad_format(struct
imx477_update_image_pad_format(imx477, imx477->mode,
fmt);
fmt->format.code =
- imx477_get_format_code(imx477, imx477->fmt.code);
+ imx477_get_format_code(imx477, imx477->fmt_code);
} else {
imx477_update_metadata_pad_format(fmt);
}
@@ -1611,6 +1599,7 @@ static int imx477_set_pad_format(struct
*framefmt = fmt->format;
} else {
imx477->mode = mode;
+ imx477->fmt_code = fmt->format.code;
imx477_set_framing_limits(imx477);
}
} else {