Re: [PATCH][MEDIA]i.MX6 CSI: Fix MIPI camera operation in RAW/Bayer mode

[Philipp Zabel]

On Tue, 2017-10-17 at 14:53 +0100, Russell King - ARM Linux wrote:
> On Tue, Oct 17, 2017 at 03:26:19PM +0200, Krzysztof Hałasa wrote:
> > Fabio Estevam  writes:
> > 
> > > > --- a/drivers/staging/media/imx/imx-media-csi.c
> > > > +++ b/drivers/staging/media/imx/imx-media-csi.c
> > > > @@ -340,7 +340,7 @@ static int csi_idmac_setup_channel(struct csi_priv 
> > > > *priv)
> > > > case V4L2_PIX_FMT_SGBRG8:
> > > > case V4L2_PIX_FMT_SGRBG8:
> > > > case V4L2_PIX_FMT_SRGGB8:
> > > > -   burst_size = 8;
> > > > +   burst_size = 16;
> > > > passthrough = true;
> > > > passthrough_bits = 8;
> > > > break;
> > > 
> > > Russell has sent the same fix and Philipp made a comment about the
> > > possibility of using 32-byte or 64-byte bursts:
> > > http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2017-October/111960.html
> > 
> > Great.
> > 
> > Sometimes I wonder how many people are working on exactly the same
> > stuff.
> 
> I do wish the patch was merged (which fixes a real problem) rather than
> hanging around for optimisation questions.  We can always increase it
> in the future if it's deemed that a larger burst size is beneficial.

I am sorry, that patch should have been part of last week's pull
request. I'll send another one for -rc6.

regards
Philipp

Re: [PATCH][MEDIA]i.MX6 CSI: Fix MIPI camera operation in RAW/Bayer mode

[Fabio Estevam]

On Tue, Oct 17, 2017 at 11:53 AM, Russell King - ARM Linux
 wrote:

> I do wish the patch was merged (which fixes a real problem) rather than
> hanging around for optimisation questions.  We can always increase it
> in the future if it's deemed that a larger burst size is beneficial.

Agreed.

Re: [PATCH][MEDIA]i.MX6 CSI: Fix MIPI camera operation in RAW/Bayer mode

[Russell King - ARM Linux]

On Tue, Oct 17, 2017 at 03:26:19PM +0200, Krzysztof Hałasa wrote:
> Fabio Estevam  writes:
> 
> >> --- a/drivers/staging/media/imx/imx-media-csi.c
> >> +++ b/drivers/staging/media/imx/imx-media-csi.c
> >> @@ -340,7 +340,7 @@ static int csi_idmac_setup_channel(struct csi_priv 
> >> *priv)
> >> case V4L2_PIX_FMT_SGBRG8:
> >> case V4L2_PIX_FMT_SGRBG8:
> >> case V4L2_PIX_FMT_SRGGB8:
> >> -   burst_size = 8;
> >> +   burst_size = 16;
> >> passthrough = true;
> >> passthrough_bits = 8;
> >> break;
> >
> > Russell has sent the same fix and Philipp made a comment about the
> > possibility of using 32-byte or 64-byte bursts:
> > http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2017-October/111960.html
> 
> Great.
> 
> Sometimes I wonder how many people are working on exactly the same
> stuff.

I do wish the patch was merged (which fixes a real problem) rather than
hanging around for optimisation questions.  We can always increase it
in the future if it's deemed that a larger burst size is beneficial.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up
According to speedtest.net: 8.21Mbps down 510kbps up

Re: [PATCH][MEDIA]i.MX6 CSI: Fix MIPI camera operation in RAW/Bayer mode

[Krzysztof Hałasa]

Fabio Estevam  writes:

>> --- a/drivers/staging/media/imx/imx-media-csi.c
>> +++ b/drivers/staging/media/imx/imx-media-csi.c
>> @@ -340,7 +340,7 @@ static int csi_idmac_setup_channel(struct csi_priv *priv)
>> case V4L2_PIX_FMT_SGBRG8:
>> case V4L2_PIX_FMT_SGRBG8:
>> case V4L2_PIX_FMT_SRGGB8:
>> -   burst_size = 8;
>> +   burst_size = 16;
>> passthrough = true;
>> passthrough_bits = 8;
>> break;
>
> Russell has sent the same fix and Philipp made a comment about the
> possibility of using 32-byte or 64-byte bursts:
> http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2017-October/111960.html

Great.

Sometimes I wonder how many people are working on exactly the same
stuff.
-- 
Krzysztof Halasa

Industrial Research Institute for Automation and Measurements PIAP
Al. Jerozolimskie 202, 02-486 Warsaw, Poland

Re: [PATCH][MEDIA]i.MX6 CSI: Fix MIPI camera operation in RAW/Bayer mode

[Fabio Estevam]

Hi Krzysztof,

On Tue, Oct 17, 2017 at 4:27 AM, Krzysztof Hałasa  wrote:
> Without this fix, in 8-bit Y/Bayer mode, every other 8-byte group
> of pixels is lost.
> Not sure about possible side effects, though.
>
> Signed-off-by: Krzysztof Hałasa 
>
> --- a/drivers/staging/media/imx/imx-media-csi.c
> +++ b/drivers/staging/media/imx/imx-media-csi.c
> @@ -340,7 +340,7 @@ static int csi_idmac_setup_channel(struct csi_priv *priv)
> case V4L2_PIX_FMT_SGBRG8:
> case V4L2_PIX_FMT_SGRBG8:
> case V4L2_PIX_FMT_SRGGB8:
> -   burst_size = 8;
> +   burst_size = 16;
> passthrough = true;
> passthrough_bits = 8;
> break;

Russell has sent the same fix and Philipp made a comment about the
possibility of using 32-byte or 64-byte bursts:
http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2017-October/111960.html