From mboxrd@z Thu Jan  1 00:00:00 1970
From: maxime.ripard@bootlin.com (Maxime Ripard)
Date: Thu, 7 Jun 2018 17:36:46 +0200
Subject: "media: ov5640: Add horizontal and vertical totals" regression
 issue on i.MX6QDL
In-Reply-To: <CAMty3ZCjNHUeHAJCDjoTHh_w1nNkUFTLjbp1=sYuF2DRiz-E-g@mail.gmail.com>
References: <CAMty3ZCjNHUeHAJCDjoTHh_w1nNkUFTLjbp1=sYuF2DRiz-E-g@mail.gmail.com>
Message-ID: <20180607153646.jlif4hukbyfkvjfx@flea>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Thu, Jun 07, 2018 at 08:02:28PM +0530, Jagan Teki wrote:
> Hi,
> 
> ov5640 camera is breaking with below commit on i.MXQDL platform.
> 
>     commit 476dec012f4c6545b0b7599cd9adba2ed819ad3b
>     Author: Maxime Ripard <maxime.ripard@bootlin.com>
>     Date:   Mon Apr 16 08:36:55 2018 -0400
> 
>     media: ov5640: Add horizontal and vertical totals
> 
>     All the initialization arrays are changing the horizontal and vertical
>     totals for some value.
> 
>     In order to clean up the driver, and since we're going to need that value
>     later on, let's introduce in the ov5640_mode_info structure the horizontal
>     and vertical total sizes, and move these out of the bytes array.
> 
>     Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
>     Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
>     Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
> 
> We have reproduced as below [1] and along with ipu1_csi0 pipeline. I
> haven't debug further please let us know how to move further.
> 
> media-ctl --links "'ov5640 2-003c':0->'imx6-mipi-csi2':0[1]"
> media-ctl --links "'imx6-mipi-csi2':1->'ipu1_csi0_mux':0[1]"
> media-ctl --links "'ipu1_csi0_mux':2->'ipu1_csi0':0[1]"
> media-ctl --links "'ipu1_csi0':2->'ipu1_csi0 capture':0[1]"
> 
> media-ctl --set-v4l2 "'ov5640 2-003c':0[fmt:UYVY2X8/640x480 field:none]"
> media-ctl --set-v4l2 "'imx6-mipi-csi2':1[fmt:UYVY2X8/640x480 field:none]"
> media-ctl --set-v4l2 "'ipu1_csi0_mux':2[fmt:UYVY2X8/640x480 field:none]"
> media-ctl --set-v4l2 "'ipu1_csi0':0[fmt:AYUV32/640x480 field:none]"
> media-ctl --set-v4l2 "'ipu1_csi0':2[fmt:AYUV32/640x480 field:none]"
> 
> [1] https://lkml.org/lkml/2018/5/31/543

Yeah, this has already been reported as part as this serie:
https://www.mail-archive.com/linux-media at vger.kernel.org/msg131655.html

and some suggestions have been done here:
https://www.mail-archive.com/linux-media at vger.kernel.org/msg132570.html

Feel free to help debug this.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com