[PATCH] media: imx-csi: fix burst size for 16 bit

[PATCH] media: imx-csi: fix burst size for 16 bit

[Jan Luebbe]

A burst_size of 4 does not work for the 16 bit passthrough formats, so
we use 8 instead.

Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
---
 drivers/staging/media/imx/imx-media-csi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c
index 1112d8f67a18..08b636084286 100644
--- a/drivers/staging/media/imx/imx-media-csi.c
+++ b/drivers/staging/media/imx/imx-media-csi.c
@@ -410,7 +410,7 @@ static int csi_idmac_setup_channel(struct csi_priv *priv)
 	case V4L2_PIX_FMT_SGRBG16:
 	case V4L2_PIX_FMT_SRGGB16:
 	case V4L2_PIX_FMT_Y16:
-		burst_size = 4;
+		burst_size = 8;
 		passthrough = true;
 		passthrough_bits = 16;
 		break;
-- 
2.17.0

Re: [PATCH] media: imx-csi: fix burst size for 16 bit

[Philipp Zabel]

On Thu, 2018-05-03 at 18:32 +0200, Jan Luebbe wrote:
> A burst_size of 4 does not work for the 16 bit passthrough formats, so
> we use 8 instead.
> 
> Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
> ---
>  drivers/staging/media/imx/imx-media-csi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c
> index 1112d8f67a18..08b636084286 100644
> --- a/drivers/staging/media/imx/imx-media-csi.c
> +++ b/drivers/staging/media/imx/imx-media-csi.c
> @@ -410,7 +410,7 @@ static int csi_idmac_setup_channel(struct csi_priv *priv)
>  	case V4L2_PIX_FMT_SGRBG16:
>  	case V4L2_PIX_FMT_SRGGB16:
>  	case V4L2_PIX_FMT_Y16:
> -		burst_size = 4;
> +		burst_size = 8;

This seems to be the equivalent to commit 37ea9830139b3 ("media: imx-
csi: fix burst size"), but for 16-bit formats.

Acked-by: Philipp Zabel <p.zabel@pengutronix.de>

regards
Philipp

Re: [PATCH] media: imx-csi: fix burst size for 16 bit

[Steve Longerbeam]

Acked-by: Steve Longerbeam <steve_longerbeam@mentor.com>


On 05/03/2018 09:40 AM, Philipp Zabel wrote:
> On Thu, 2018-05-03 at 18:32 +0200, Jan Luebbe wrote:
>> A burst_size of 4 does not work for the 16 bit passthrough formats, so
>> we use 8 instead.
>>
>> Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
>> ---
>>   drivers/staging/media/imx/imx-media-csi.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c
>> index 1112d8f67a18..08b636084286 100644
>> --- a/drivers/staging/media/imx/imx-media-csi.c
>> +++ b/drivers/staging/media/imx/imx-media-csi.c
>> @@ -410,7 +410,7 @@ static int csi_idmac_setup_channel(struct csi_priv *priv)
>>   	case V4L2_PIX_FMT_SGRBG16:
>>   	case V4L2_PIX_FMT_SRGGB16:
>>   	case V4L2_PIX_FMT_Y16:
>> -		burst_size = 4;
>> +		burst_size = 8;
> This seems to be the equivalent to commit 37ea9830139b3 ("media: imx-
> csi: fix burst size"), but for 16-bit formats.
>
> Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> regards
> Philipp