]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
media: i2c: og01a1b: Specify monochrome media bus format instead of Bayer
authorVladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Sat, 23 Aug 2025 13:22:06 +0000 (16:22 +0300)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 9 Sep 2025 13:59:16 +0000 (15:59 +0200)
The OmniVision OG01A1B image sensor is a monochrome sensor, it supports
8-bit and 10-bit RAW output formats only.

That said the planar greyscale Y8/Y10 media formats are more appropriate
for the sensor instead of the originally and arbitrary selected SGRBG one,
since there is no red, green or blue color components.

Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
drivers/media/i2c/og01a1b.c

index 78d5d406e4b72a36d40977b388fd95d31daf8ed4..b7d0b677975d5eecc3936725e3461b559b22e824 100644 (file)
@@ -682,7 +682,7 @@ static void og01a1b_update_pad_format(const struct og01a1b_mode *mode,
 {
        fmt->width = mode->width;
        fmt->height = mode->height;
-       fmt->code = MEDIA_BUS_FMT_SGRBG10_1X10;
+       fmt->code = MEDIA_BUS_FMT_Y10_1X10;
        fmt->field = V4L2_FIELD_NONE;
 }
 
@@ -828,7 +828,7 @@ static int og01a1b_enum_mbus_code(struct v4l2_subdev *sd,
        if (code->index > 0)
                return -EINVAL;
 
-       code->code = MEDIA_BUS_FMT_SGRBG10_1X10;
+       code->code = MEDIA_BUS_FMT_Y10_1X10;
 
        return 0;
 }
@@ -840,7 +840,7 @@ static int og01a1b_enum_frame_size(struct v4l2_subdev *sd,
        if (fse->index >= ARRAY_SIZE(supported_modes))
                return -EINVAL;
 
-       if (fse->code != MEDIA_BUS_FMT_SGRBG10_1X10)
+       if (fse->code != MEDIA_BUS_FMT_Y10_1X10)
                return -EINVAL;
 
        fse->min_width = supported_modes[fse->index].width;