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 <festevam@xxxxxxxxx> 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
