| Message ID | 20220123160857.24161-3-laurent.pinchart+renesas@ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | media: Centralize MIPI CSI-2 data types in shared header | expand |
Hello, Ping on this patch in case someone wants to review it. I'll send a pull request shortly for the part of this series that hasn't been merged yet, and this is the last one missing a reviewed-by tag. On Sun, Jan 23, 2022 at 06:08:53PM +0200, Laurent Pinchart wrote: > Replace the hardcoded MIPI CSI-2 data types with macros from > mipi-csi2.h. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > drivers/media/platform/cadence/cdns-csi2tx.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/cadence/cdns-csi2tx.c b/drivers/media/platform/cadence/cdns-csi2tx.c > index 8f8c36056354..58e405b69f67 100644 > --- a/drivers/media/platform/cadence/cdns-csi2tx.c > +++ b/drivers/media/platform/cadence/cdns-csi2tx.c > @@ -15,6 +15,7 @@ > #include <linux/platform_device.h> > #include <linux/slab.h> > > +#include <media/mipi-csi2.h> > #include <media/v4l2-ctrls.h> > #include <media/v4l2-device.h> > #include <media/v4l2-fwnode.h> > @@ -121,12 +122,12 @@ static const struct csi2tx_fmt csi2tx_formats[] = { > { > .mbus = MEDIA_BUS_FMT_UYVY8_1X16, > .bpp = 2, > - .dt = 0x1e, > + .dt = MIPI_CSI2_DT_YUV422_8B, > }, > { > .mbus = MEDIA_BUS_FMT_RGB888_1X24, > .bpp = 3, > - .dt = 0x24, > + .dt = MIPI_CSI2_DT_RGB888, > }, > }; >
diff --git a/drivers/media/platform/cadence/cdns-csi2tx.c b/drivers/media/platform/cadence/cdns-csi2tx.c index 8f8c36056354..58e405b69f67 100644 --- a/drivers/media/platform/cadence/cdns-csi2tx.c +++ b/drivers/media/platform/cadence/cdns-csi2tx.c @@ -15,6 +15,7 @@ #include <linux/platform_device.h> #include <linux/slab.h> +#include <media/mipi-csi2.h> #include <media/v4l2-ctrls.h> #include <media/v4l2-device.h> #include <media/v4l2-fwnode.h> @@ -121,12 +122,12 @@ static const struct csi2tx_fmt csi2tx_formats[] = { { .mbus = MEDIA_BUS_FMT_UYVY8_1X16, .bpp = 2, - .dt = 0x1e, + .dt = MIPI_CSI2_DT_YUV422_8B, }, { .mbus = MEDIA_BUS_FMT_RGB888_1X24, .bpp = 3, - .dt = 0x24, + .dt = MIPI_CSI2_DT_RGB888, }, };
Replace the hardcoded MIPI CSI-2 data types with macros from mipi-csi2.h. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- drivers/media/platform/cadence/cdns-csi2tx.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)