On Sun, Jan 27, 2019 at 2:36 PM Steve Longerbeam <slongerbeam@xxxxxxxxx> wrote: > > Hi Tim, > > On 1/25/19 3:57 PM, Steve Longerbeam wrote: > <snip> > >> Now lets go back to a 480p60 source but this time include the vdic > >> (which isn't necessary but should still work right?) > > > > No. First, the CSI will only capture in bt.656 mode if it sees > > interlaced fields (bt.656 interlaced sync codes). Well, let me > > rephrase, the CSI does support progressive BT.656 but I have never > > tested that myself. > > One more comment here. It would be great if you could test a progressive > bt.656 sensor (example: imx6q-gw54xx tda19971 480p60Hz YUV via BT656 > IPU1_CSI0) since as I said I've never been able to test this myself. > > Since it is progressive there's no need for the VDIC, so try these > pipelines: > > mode0: sensor -> mux -> csi -> /dev/videoN > mode1: sensor -> mux -> csi -> ic_prp -> ic_prpenc -> /dev/videoN > Steve, These both work fine in all cases for 480p60Hz YUV via BT656. When I use 480p60Hz 'RGB' via BT656 mode0 works fine as the CSI does the colorspace conversion needed for coda but for mode1 I end up with the itu601 colorimetery issue from before that I'm still trying to find a solution for. Tim