This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: atomisp: Replace V4L2_PIX_FMT_RGB32 with V4L2_PIX_FMT_RGBX32
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Sat Mar 16 21:48:30 2024 +0000

V4L2_PIX_FMT_RGB32 is deprecated because its definition never made
clear if the 4th byte was alpha data or unused. Instead either
V4L2_PIX_FMT_ARGB32 or V4L2_PIX_FMT_RGBX32 should be used.

V4L2_PIX_FMT_RGB32 internally is mapped to IA_CSS_FRAME_FORMAT_RGBA888
and ia_css_frame_format.h says that the alpha bits there are unused,
so replace V4L2_PIX_FMT_RGB32 with V4L2_PIX_FMT_RGBX32.

Signed-off-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/staging/media/atomisp/pci/atomisp_cmd.c   | 4 ++--
 drivers/staging/media/atomisp/pci/atomisp_ioctl.c | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.c 
b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
index eb37bb6e41f9..2bf71940050b 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_cmd.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
@@ -1123,7 +1123,7 @@ v4l2_fmt_to_sh_fmt(u32 fmt)
                return IA_CSS_FRAME_FORMAT_YUYV;
        case V4L2_PIX_FMT_RGB24:
                return IA_CSS_FRAME_FORMAT_PLANAR_RGB888;
-       case V4L2_PIX_FMT_RGB32:
+       case V4L2_PIX_FMT_RGBX32:
                return IA_CSS_FRAME_FORMAT_RGBA888;
        case V4L2_PIX_FMT_RGB565:
                return IA_CSS_FRAME_FORMAT_RGB565;
@@ -1210,7 +1210,7 @@ u32 atomisp_get_pixel_depth(u32 pixelformat)
        case V4L2_PIX_FMT_RGB24:
        case V4L2_PIX_FMT_YUV444:
                return 24;
-       case V4L2_PIX_FMT_RGB32:
+       case V4L2_PIX_FMT_RGBX32:
                return 32;
        case V4L2_PIX_FMT_JPEG:
        case V4L2_PIX_FMT_CUSTOM_M10MO_RAW:
diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c 
b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
index 6009b7779f13..9d87213a8382 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
@@ -301,7 +301,7 @@ const struct atomisp_format_bridge atomisp_output_fmts[] = {
                .sh_fmt = IA_CSS_FRAME_FORMAT_RAW,
                .description = "Bayer 12"
        }, {
-               .pixelformat = V4L2_PIX_FMT_RGB32,
+               .pixelformat = V4L2_PIX_FMT_RGBX32,
                .depth = 32,
                .mbus_code = V4L2_MBUS_FMT_CUSTOM_RGB32,
                .sh_fmt = IA_CSS_FRAME_FORMAT_RGBA888,

Reply via email to