On Mon, Apr 26, 2021 at 9:14 AM Fabio Estevam <festevam@xxxxxxxxx> wrote: > > Hi Tim, > > I am back at trying to get a proper image captured by the ADV7280 on > an imx6dl board connected to the parallel CSI bus. > > I tried your patches against 5.4 and 5.12: > > https://github.com/Gateworks/linux-imx6/commit/959fbd4 > and > https://patchwork.kernel.org/project/linux-media/patch/20190827215539.1286-1-mmichilot@xxxxxxxxxxxxx/ > > ,but I am still getting horizontal scrolling images after running: > > media-ctl -l "'adv7180 0-0020':0 -> 'ipu1_csi1_mux':4[1]" > media-ctl -l "'ipu1_csi1_mux':5 -> 'ipu1_csi1':0[1]" > media-ctl -l "'ipu1_csi1':1 -> 'ipu1_vdic':0[1]" > media-ctl -l "'ipu1_vdic':2 -> 'ipu1_ic_prp':0[1]" > media-ctl -l "'ipu1_ic_prp':2 -> 'ipu1_ic_prpvf':0[1]" > media-ctl -l "'ipu1_ic_prpvf':1 -> 'ipu1_ic_prpvf capture':0[1]" > media-ctl -V "'adv7180 0-0020':0 [fmt:UYVY2X8/640x480 field:seq-tb]" > media-ctl -V "'ipu1_csi1_mux':5 [fmt:UYVY2X8/640x480]" > media-ctl -V "'ipu1_csi1':1 [fmt:AYUV32/640x480]" > media-ctl -V "'ipu1_vdic':2 [fmt:AYUV32/640x480 field:none]" > media-ctl -V "'ipu1_ic_prp':2 [fmt:AYUV32/640x480 field:none]" > media-ctl -V "'ipu1_ic_prpvf':1 [fmt:AYUV32/640x480 field:none]" > v4l2-ctl -d2 --set-fmt-video=field=none > > gst-launch-1.0 v4l2src device=/dev/video2 ! kmssink > > Also, I saw your commit: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v5.12&id=9483a3f8e1b58ba1d7cd21687d8d0a63a015c36b > > ,but adv7180_get_skip_frames() does not get called. Is there any > missing patch from the mix csi driver that should call > g_skip_frames()? > Fabio, GStreamer does not use the g_skip_frames feature as far as I know. If it did then we likely wouldn't need Steves patch https://github.com/Gateworks/linux-imx6/commit/959fbd4. His commit explains the issue pretty well and yes, this should be mainlined in my opinion unless someone presents a more proper way to handle it. The proper way to set the video standard (PAL vs NTSC) is 'v4l2-ctl --device /dev/v4l-subdev2 --set-standard PAL' (on the proper subdev of course). The https://patchwork.kernel.org/project/linux-media/patch/20190827215539.1286-1-mmichilot@xxxxxxxxxxxxx/ patch is only needed if you are using adv7280 which I thought you mentioned you were at some point. I still don't have a solution for that other than add a kernel module parameter or something. Best regards, Tim
