media: imx: imx7-media-csi: Don't set PIXEL_BIT in CSICR1
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Wed, 3 Feb 2021 02:57:18 +0000 (03:57 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Wed, 4 Aug 2021 12:43:51 +0000 (14:43 +0200)
commit8b226173a1e9ea1b77dabb3d5583d376c53164c3
tree711db33da1de6a5d56497b57ee405db8666aa025
parenta581c87c681c6202403d74e8e816968841afe2f7
media: imx: imx7-media-csi: Don't set PIXEL_BIT in CSICR1

The PIXEL_BIT field of the CSICR1 register is documented as setting the
Bayer data width to 10 bits, and is set by the driver for all non-YUV
pixel formats. Test code from NXP showed that the bit shouldn't be set
for Bayer formats, and this was confirmed by experimentation with RAW8
capture (which doesn't work when setting the field) and RAW10 capture
(for which setting the field doesn't seem to make a difference) on
i.MX8MM with an OV5640 sensor connected over CSI-2. Don't set it.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Rui Miguel Silva <rmfrfs@gmail.com>
Tested-by: Martin Kepplinger <martin.kepplinger@puri.sm>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/imx/imx7-media-csi.c