All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats
@ 2022-06-18 22:23 ` Marek Vasut
  0 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2022-06-18 22:23 UTC (permalink / raw)
  To: linux-media
  Cc: Marek Vasut, Alain Volmat, Alexandre Torgue, Amelie DELAUNAY,
	Hugues FRUCHET, Laurent Pinchart, Philippe CORNU, linux-stm32,
	linux-arm-kernel

Fill in 10, 12, 14 bit Bayer formats into the DCMI driver.
Those are useful e.g. when MT9P006 sensor is connected.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Alain Volmat <alain.volmat@foss.st.com>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: Amelie DELAUNAY <amelie.delaunay@foss.st.com>
Cc: Hugues FRUCHET <hugues.fruchet@foss.st.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Philippe CORNU <philippe.cornu@foss.st.com>
Cc: linux-stm32@st-md-mailman.stormreply.com
Cc: linux-arm-kernel@lists.infradead.org
---
 drivers/media/platform/st/stm32/stm32-dcmi.c | 48 ++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/drivers/media/platform/st/stm32/stm32-dcmi.c b/drivers/media/platform/st/stm32/stm32-dcmi.c
index 09a743cd70040..5f08ba47ea96b 100644
--- a/drivers/media/platform/st/stm32/stm32-dcmi.c
+++ b/drivers/media/platform/st/stm32/stm32-dcmi.c
@@ -1631,6 +1631,54 @@ static const struct dcmi_format dcmi_formats[] = {
 		.fourcc = V4L2_PIX_FMT_SRGGB8,
 		.mbus_code = MEDIA_BUS_FMT_SRGGB8_1X8,
 		.bpp = 1,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SBGGR10,
+		.mbus_code = MEDIA_BUS_FMT_SBGGR10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGBRG10,
+		.mbus_code = MEDIA_BUS_FMT_SGBRG10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGRBG10,
+		.mbus_code = MEDIA_BUS_FMT_SGRBG10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SRGGB10,
+		.mbus_code = MEDIA_BUS_FMT_SRGGB10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SBGGR12,
+		.mbus_code = MEDIA_BUS_FMT_SBGGR12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGBRG12,
+		.mbus_code = MEDIA_BUS_FMT_SGBRG12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGRBG12,
+		.mbus_code = MEDIA_BUS_FMT_SGRBG12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SRGGB12,
+		.mbus_code = MEDIA_BUS_FMT_SRGGB12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SBGGR14,
+		.mbus_code = MEDIA_BUS_FMT_SBGGR14_1X14,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGBRG14,
+		.mbus_code = MEDIA_BUS_FMT_SGBRG14_1X14,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGRBG14,
+		.mbus_code = MEDIA_BUS_FMT_SGRBG14_1X14,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SRGGB14,
+		.mbus_code = MEDIA_BUS_FMT_SRGGB14_1X14,
+		.bpp = 2,
 	},
 };
 
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats
@ 2022-06-18 22:23 ` Marek Vasut
  0 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2022-06-18 22:23 UTC (permalink / raw)
  To: linux-media
  Cc: Marek Vasut, Alain Volmat, Alexandre Torgue, Amelie DELAUNAY,
	Hugues FRUCHET, Laurent Pinchart, Philippe CORNU, linux-stm32,
	linux-arm-kernel

Fill in 10, 12, 14 bit Bayer formats into the DCMI driver.
Those are useful e.g. when MT9P006 sensor is connected.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Alain Volmat <alain.volmat@foss.st.com>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: Amelie DELAUNAY <amelie.delaunay@foss.st.com>
Cc: Hugues FRUCHET <hugues.fruchet@foss.st.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Philippe CORNU <philippe.cornu@foss.st.com>
Cc: linux-stm32@st-md-mailman.stormreply.com
Cc: linux-arm-kernel@lists.infradead.org
---
 drivers/media/platform/st/stm32/stm32-dcmi.c | 48 ++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/drivers/media/platform/st/stm32/stm32-dcmi.c b/drivers/media/platform/st/stm32/stm32-dcmi.c
index 09a743cd70040..5f08ba47ea96b 100644
--- a/drivers/media/platform/st/stm32/stm32-dcmi.c
+++ b/drivers/media/platform/st/stm32/stm32-dcmi.c
@@ -1631,6 +1631,54 @@ static const struct dcmi_format dcmi_formats[] = {
 		.fourcc = V4L2_PIX_FMT_SRGGB8,
 		.mbus_code = MEDIA_BUS_FMT_SRGGB8_1X8,
 		.bpp = 1,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SBGGR10,
+		.mbus_code = MEDIA_BUS_FMT_SBGGR10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGBRG10,
+		.mbus_code = MEDIA_BUS_FMT_SGBRG10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGRBG10,
+		.mbus_code = MEDIA_BUS_FMT_SGRBG10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SRGGB10,
+		.mbus_code = MEDIA_BUS_FMT_SRGGB10_1X10,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SBGGR12,
+		.mbus_code = MEDIA_BUS_FMT_SBGGR12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGBRG12,
+		.mbus_code = MEDIA_BUS_FMT_SGBRG12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGRBG12,
+		.mbus_code = MEDIA_BUS_FMT_SGRBG12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SRGGB12,
+		.mbus_code = MEDIA_BUS_FMT_SRGGB12_1X12,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SBGGR14,
+		.mbus_code = MEDIA_BUS_FMT_SBGGR14_1X14,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGBRG14,
+		.mbus_code = MEDIA_BUS_FMT_SGBRG14_1X14,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SGRBG14,
+		.mbus_code = MEDIA_BUS_FMT_SGRBG14_1X14,
+		.bpp = 2,
+	}, {
+		.fourcc = V4L2_PIX_FMT_SRGGB14,
+		.mbus_code = MEDIA_BUS_FMT_SRGGB14_1X14,
+		.bpp = 2,
 	},
 };
 
-- 
2.35.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats
  2022-06-18 22:23 ` Marek Vasut
@ 2022-06-27  9:39   ` Hugues FRUCHET
  -1 siblings, 0 replies; 6+ messages in thread
From: Hugues FRUCHET @ 2022-06-27  9:39 UTC (permalink / raw)
  To: Marek Vasut, linux-media
  Cc: Alain Volmat, Alexandre Torgue, Amelie DELAUNAY,
	Laurent Pinchart, Philippe CORNU, linux-stm32, linux-arm-kernel

Hi Marek,

Thanks for the patch.

Acked-by: Hugues Fruchet <hugues.fruchet@st.com>

Please note that we have not tested parallel setup with more than 8 bits 
with this driver because of lack of hardware support.
Which board are you using ?
Are the whole 12 bits of the MT9P006 parallel bus wired on your board ?

Best regards,
Hugues.

On 6/19/22 00:23, Marek Vasut wrote:
> Fill in 10, 12, 14 bit Bayer formats into the DCMI driver.
> Those are useful e.g. when MT9P006 sensor is connected.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Alain Volmat <alain.volmat@foss.st.com>
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> Cc: Amelie DELAUNAY <amelie.delaunay@foss.st.com>
> Cc: Hugues FRUCHET <hugues.fruchet@foss.st.com>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Philippe CORNU <philippe.cornu@foss.st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> Cc: linux-arm-kernel@lists.infradead.org
> ---
>   drivers/media/platform/st/stm32/stm32-dcmi.c | 48 ++++++++++++++++++++
>   1 file changed, 48 insertions(+)
> 
> diff --git a/drivers/media/platform/st/stm32/stm32-dcmi.c b/drivers/media/platform/st/stm32/stm32-dcmi.c
> index 09a743cd70040..5f08ba47ea96b 100644
> --- a/drivers/media/platform/st/stm32/stm32-dcmi.c
> +++ b/drivers/media/platform/st/stm32/stm32-dcmi.c
> @@ -1631,6 +1631,54 @@ static const struct dcmi_format dcmi_formats[] = {
>   		.fourcc = V4L2_PIX_FMT_SRGGB8,
>   		.mbus_code = MEDIA_BUS_FMT_SRGGB8_1X8,
>   		.bpp = 1,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SBGGR10,
> +		.mbus_code = MEDIA_BUS_FMT_SBGGR10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGBRG10,
> +		.mbus_code = MEDIA_BUS_FMT_SGBRG10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGRBG10,
> +		.mbus_code = MEDIA_BUS_FMT_SGRBG10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SRGGB10,
> +		.mbus_code = MEDIA_BUS_FMT_SRGGB10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SBGGR12,
> +		.mbus_code = MEDIA_BUS_FMT_SBGGR12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGBRG12,
> +		.mbus_code = MEDIA_BUS_FMT_SGBRG12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGRBG12,
> +		.mbus_code = MEDIA_BUS_FMT_SGRBG12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SRGGB12,
> +		.mbus_code = MEDIA_BUS_FMT_SRGGB12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SBGGR14,
> +		.mbus_code = MEDIA_BUS_FMT_SBGGR14_1X14,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGBRG14,
> +		.mbus_code = MEDIA_BUS_FMT_SGBRG14_1X14,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGRBG14,
> +		.mbus_code = MEDIA_BUS_FMT_SGRBG14_1X14,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SRGGB14,
> +		.mbus_code = MEDIA_BUS_FMT_SRGGB14_1X14,
> +		.bpp = 2,
>   	},
>   };
>   

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats
@ 2022-06-27  9:39   ` Hugues FRUCHET
  0 siblings, 0 replies; 6+ messages in thread
From: Hugues FRUCHET @ 2022-06-27  9:39 UTC (permalink / raw)
  To: Marek Vasut, linux-media
  Cc: Alain Volmat, Alexandre Torgue, Amelie DELAUNAY,
	Laurent Pinchart, Philippe CORNU, linux-stm32, linux-arm-kernel

Hi Marek,

Thanks for the patch.

Acked-by: Hugues Fruchet <hugues.fruchet@st.com>

Please note that we have not tested parallel setup with more than 8 bits 
with this driver because of lack of hardware support.
Which board are you using ?
Are the whole 12 bits of the MT9P006 parallel bus wired on your board ?

Best regards,
Hugues.

On 6/19/22 00:23, Marek Vasut wrote:
> Fill in 10, 12, 14 bit Bayer formats into the DCMI driver.
> Those are useful e.g. when MT9P006 sensor is connected.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Alain Volmat <alain.volmat@foss.st.com>
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> Cc: Amelie DELAUNAY <amelie.delaunay@foss.st.com>
> Cc: Hugues FRUCHET <hugues.fruchet@foss.st.com>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Philippe CORNU <philippe.cornu@foss.st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> Cc: linux-arm-kernel@lists.infradead.org
> ---
>   drivers/media/platform/st/stm32/stm32-dcmi.c | 48 ++++++++++++++++++++
>   1 file changed, 48 insertions(+)
> 
> diff --git a/drivers/media/platform/st/stm32/stm32-dcmi.c b/drivers/media/platform/st/stm32/stm32-dcmi.c
> index 09a743cd70040..5f08ba47ea96b 100644
> --- a/drivers/media/platform/st/stm32/stm32-dcmi.c
> +++ b/drivers/media/platform/st/stm32/stm32-dcmi.c
> @@ -1631,6 +1631,54 @@ static const struct dcmi_format dcmi_formats[] = {
>   		.fourcc = V4L2_PIX_FMT_SRGGB8,
>   		.mbus_code = MEDIA_BUS_FMT_SRGGB8_1X8,
>   		.bpp = 1,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SBGGR10,
> +		.mbus_code = MEDIA_BUS_FMT_SBGGR10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGBRG10,
> +		.mbus_code = MEDIA_BUS_FMT_SGBRG10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGRBG10,
> +		.mbus_code = MEDIA_BUS_FMT_SGRBG10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SRGGB10,
> +		.mbus_code = MEDIA_BUS_FMT_SRGGB10_1X10,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SBGGR12,
> +		.mbus_code = MEDIA_BUS_FMT_SBGGR12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGBRG12,
> +		.mbus_code = MEDIA_BUS_FMT_SGBRG12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGRBG12,
> +		.mbus_code = MEDIA_BUS_FMT_SGRBG12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SRGGB12,
> +		.mbus_code = MEDIA_BUS_FMT_SRGGB12_1X12,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SBGGR14,
> +		.mbus_code = MEDIA_BUS_FMT_SBGGR14_1X14,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGBRG14,
> +		.mbus_code = MEDIA_BUS_FMT_SGBRG14_1X14,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SGRBG14,
> +		.mbus_code = MEDIA_BUS_FMT_SGRBG14_1X14,
> +		.bpp = 2,
> +	}, {
> +		.fourcc = V4L2_PIX_FMT_SRGGB14,
> +		.mbus_code = MEDIA_BUS_FMT_SRGGB14_1X14,
> +		.bpp = 2,
>   	},
>   };
>   

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats
  2022-06-27  9:39   ` Hugues FRUCHET
@ 2022-06-27 10:00     ` Marek Vasut
  -1 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2022-06-27 10:00 UTC (permalink / raw)
  To: Hugues FRUCHET, linux-media
  Cc: Alain Volmat, Alexandre Torgue, Amelie DELAUNAY,
	Laurent Pinchart, Philippe CORNU, linux-stm32, linux-arm-kernel

On 6/27/22 11:39, Hugues FRUCHET wrote:
> Hi Marek,

Hi,

> Thanks for the patch.
> 
> Acked-by: Hugues Fruchet <hugues.fruchet@st.com>
> 
> Please note that we have not tested parallel setup with more than 8 bits 
> with this driver because of lack of hardware support.
> Which board are you using ?

Custom one with MT9P006 sensor.

> Are the whole 12 bits of the MT9P006 parallel bus wired on your board ?

Yes

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats
@ 2022-06-27 10:00     ` Marek Vasut
  0 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2022-06-27 10:00 UTC (permalink / raw)
  To: Hugues FRUCHET, linux-media
  Cc: Alain Volmat, Alexandre Torgue, Amelie DELAUNAY,
	Laurent Pinchart, Philippe CORNU, linux-stm32, linux-arm-kernel

On 6/27/22 11:39, Hugues FRUCHET wrote:
> Hi Marek,

Hi,

> Thanks for the patch.
> 
> Acked-by: Hugues Fruchet <hugues.fruchet@st.com>
> 
> Please note that we have not tested parallel setup with more than 8 bits 
> with this driver because of lack of hardware support.
> Which board are you using ?

Custom one with MT9P006 sensor.

> Are the whole 12 bits of the MT9P006 parallel bus wired on your board ?

Yes

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-06-27 10:02 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-18 22:23 [PATCH] media: stm32: dcmi: Fill in remaining Bayer formats Marek Vasut
2022-06-18 22:23 ` Marek Vasut
2022-06-27  9:39 ` Hugues FRUCHET
2022-06-27  9:39   ` Hugues FRUCHET
2022-06-27 10:00   ` Marek Vasut
2022-06-27 10:00     ` Marek Vasut

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.