diff options
Diffstat (limited to 'target/linux/bcm27xx/patches-4.19/950-0736-media-bcm2835-unicam-Fix-one-to-many-mapping-for-YUY.patch')
-rw-r--r-- | target/linux/bcm27xx/patches-4.19/950-0736-media-bcm2835-unicam-Fix-one-to-many-mapping-for-YUY.patch | 144 |
1 files changed, 0 insertions, 144 deletions
diff --git a/target/linux/bcm27xx/patches-4.19/950-0736-media-bcm2835-unicam-Fix-one-to-many-mapping-for-YUY.patch b/target/linux/bcm27xx/patches-4.19/950-0736-media-bcm2835-unicam-Fix-one-to-many-mapping-for-YUY.patch deleted file mode 100644 index 3c881a1bfc..0000000000 --- a/target/linux/bcm27xx/patches-4.19/950-0736-media-bcm2835-unicam-Fix-one-to-many-mapping-for-YUY.patch +++ /dev/null @@ -1,144 +0,0 @@ -From 94d77466473f3abcf799ac44b8038766ab32f27c Mon Sep 17 00:00:00 2001 -From: Dave Stevenson <dave.stevenson@raspberrypi.org> -Date: Wed, 16 Oct 2019 18:53:06 +0100 -Subject: [PATCH] media: bcm2835-unicam: Fix one-to-many mapping for - YUYV formats - -V4L2 format V4L2_PIX_FMT_YUYV maps to both MEDIA_BUS_FMT_YUYV8_2X8 -and MEDIA_BUS_FMT_YUYV8_1X16. The change to not cache the active -formats also meant that we only ever found the first variant of -the mediabus format when trying to setup the device. - -Flag the formats that have other representations, and ensure -that the format conversion checks whether the found format -matches one supported by the sensor before returning it. - -Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.org> ---- - .../media/platform/bcm2835/bcm2835-unicam.c | 45 ++++++++++++++++--- - 1 file changed, 39 insertions(+), 6 deletions(-) - ---- a/drivers/media/platform/bcm2835/bcm2835-unicam.c -+++ b/drivers/media/platform/bcm2835/bcm2835-unicam.c -@@ -136,6 +136,8 @@ MODULE_PARM_DESC(debug, "Debug level 0-3 - * @code: V4L2 media bus format code. - * @depth: Bits per pixel as delivered from the source. - * @csi_dt: CSI data type. -+ * @check_variants: Flag to denote that there are multiple mediabus formats -+ * still in the list that could match this V4L2 format. - */ - struct unicam_fmt { - u32 fourcc; -@@ -143,6 +145,7 @@ struct unicam_fmt { - u32 code; - u8 depth; - u8 csi_dt; -+ u8 check_variants; - }; - - static const struct unicam_fmt formats[] = { -@@ -152,21 +155,25 @@ static const struct unicam_fmt formats[] - .code = MEDIA_BUS_FMT_YUYV8_2X8, - .depth = 16, - .csi_dt = 0x1e, -+ .check_variants = 1, - }, { - .fourcc = V4L2_PIX_FMT_UYVY, - .code = MEDIA_BUS_FMT_UYVY8_2X8, - .depth = 16, - .csi_dt = 0x1e, -+ .check_variants = 1, - }, { - .fourcc = V4L2_PIX_FMT_YVYU, - .code = MEDIA_BUS_FMT_YVYU8_2X8, - .depth = 16, - .csi_dt = 0x1e, -+ .check_variants = 1, - }, { - .fourcc = V4L2_PIX_FMT_VYUY, - .code = MEDIA_BUS_FMT_VYUY8_2X8, - .depth = 16, - .csi_dt = 0x1e, -+ .check_variants = 1, - }, { - .fourcc = V4L2_PIX_FMT_YUYV, - .code = MEDIA_BUS_FMT_YUYV8_1X16, -@@ -489,14 +496,40 @@ static const struct unicam_fmt *find_for - return NULL; - } - --static const struct unicam_fmt *find_format_by_pix(u32 pixelformat) -+static int check_mbus_format(struct unicam_device *dev, -+ const struct unicam_fmt *format) -+{ -+ struct v4l2_subdev_mbus_code_enum mbus_code; -+ int ret = 0; -+ int i; -+ -+ for (i = 0; !ret && i < MAX_ENUM_MBUS_CODE; i++) { -+ memset(&mbus_code, 0, sizeof(mbus_code)); -+ mbus_code.index = i; -+ -+ ret = v4l2_subdev_call(dev->sensor, pad, enum_mbus_code, -+ NULL, &mbus_code); -+ -+ if (!ret && mbus_code.code == format->code) -+ return 1; -+ } -+ -+ return 0; -+} -+ -+static const struct unicam_fmt *find_format_by_pix(struct unicam_device *dev, -+ u32 pixelformat) - { - unsigned int k; - - for (k = 0; k < ARRAY_SIZE(formats); k++) { - if (formats[k].fourcc == pixelformat || -- formats[k].repacked_fourcc == pixelformat) -+ formats[k].repacked_fourcc == pixelformat) { -+ if (formats[k].check_variants && -+ !check_mbus_format(dev, &formats[k])) -+ continue; - return &formats[k]; -+ } - } - - return NULL; -@@ -832,7 +865,7 @@ static int unicam_try_fmt_vid_cap(struct - struct v4l2_mbus_framefmt *mbus_fmt = &sd_fmt.format; - int ret; - -- fmt = find_format_by_pix(f->fmt.pix.pixelformat); -+ fmt = find_format_by_pix(dev, f->fmt.pix.pixelformat); - if (!fmt) { - /* Pixel format not supported by unicam. Choose the first - * supported format, and let the sensor choose something else. -@@ -917,7 +950,7 @@ static int unicam_s_fmt_vid_cap(struct f - if (ret < 0) - return ret; - -- fmt = find_format_by_pix(f->fmt.pix.pixelformat); -+ fmt = find_format_by_pix(dev, f->fmt.pix.pixelformat); - if (!fmt) { - /* Unknown pixel format - adopt a default. - * This shouldn't happen as try_fmt should have resolved any -@@ -1540,7 +1573,7 @@ static int unicam_enum_framesizes(struct - int ret; - - /* check for valid format */ -- fmt = find_format_by_pix(fsize->pixel_format); -+ fmt = find_format_by_pix(dev, fsize->pixel_format); - if (!fmt) { - unicam_dbg(3, dev, "Invalid pixel code: %x\n", - fsize->pixel_format); -@@ -1579,7 +1612,7 @@ static int unicam_enum_frameintervals(st - }; - int ret; - -- fmt = find_format_by_pix(fival->pixel_format); -+ fmt = find_format_by_pix(dev, fival->pixel_format); - if (!fmt) - return -EINVAL; - |