Hello Hugues, thank you for getting back to me, I'll look into your suggestions in the next few days, I'll give you a feedback as soon as I know more. Thanks, Fab > Subject: Re: [PATCH v5 0/5] Add OV5640 parallel interface and RGB565/YUYV support > > Hi Fabrizio, > > Happy to see that this patch series is of interest ;) > > As you can see in mail thread, Maxime Ripard is also testing it: > https://www.mail-archive.com/linux-media@vger.kernel.org/msg124322.html > > For BT656 support, it was not initially planned but it seems > straightforward to code it, and anyway I have to add JPEG support, so I > could add BT656 support in next patch series. > > For your symptom, I would say same as I said to Maxime, check first the > polarity of sync signals (hysnc/vysnc/pclk) between ISP and OV5640. > Note also that several frames are needed to get a non-black picture. > You can also use the colorbar test to validate bus connection between > ISP and sensor. > > Here are the yavta commands that I'm commonly using: > > * grab QVGA RGB565 raw frame (note the skip=10 to get a correct image) > yavta -s 320x240 -n 3 --capture=13 --skip=10 --format=RGB565 /dev/video0 > --file=grab-320x240-rgb565-#.raw > > * grab QVGA YUV frame > yavta -s 320x240 -n 3 --capture=13 --skip=10 --format=YUYV /dev/video0 > --file=grab-320x240-yuyv-#.raw > > * grab QVGA colorbar RGB565 raw frame > yavta -s 320x240 -n 3 -w '0x009f0903 1' --capture=1 --format=RGB565 > /dev/video0 --file=grab-colorbar-320x240-rgb565-#.raw > > * disable colorbars > yavta -s 320x240 -n 3 -w '0x009f0903 0' /dev/video0 > > > Hope this will help ! > > Best regards, > Hugues. > > On 01/08/2018 09:54 PM, Fabrizio Castro wrote: > > Hello Hugues, > > > > thank you for the patch series. > > I am having a go with your patches, and although they seem alright, I don't seem to be able to grab a non-black picture on the iWave > iwg20d in plain DVP mode, but if I switch to BT656 just by setting register 0x4730 to 0x01 (I know, it's a nasty hack...) I can get > something sensible out. > > > > At the moment there is no proper BT656 support in the driver, I was wondering if you have any plans to enhance the ov5640 driver a > little bit further to add proper BT656 support as it may be convenient. > > > > Do you know if someone else was able to get DVP to work by means of this patch series on a non-STM32 platform? > > > > Thanks, > > Fabrizio > > > > > >> Subject: [PATCH v5 0/5] Add OV5640 parallel interface and RGB565/YUYV support > >> > >> Enhance OV5640 CSI driver to support also DVP parallel interface. > >> Add RGB565 (LE & BE) and YUV422 YUYV format in addition to existing > >> YUV422 UYVY format. > >> Some other improvements on chip identifier check and removal > >> of warnings in powering phase around gpio handling. > >> > >> =========== > >> = history = > >> =========== > >> version 5: > >> - Refine bindings as per Sakari suggestion: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg124048.html > >> > >> version 4: > >> - Refine bindings as per Sakari suggestion: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg123609.html > >> - Parallel port control lines polarity can now be configured through > >> devicetree > >> > >> version 3: > >> - Move chip identifier check at probe according to Fabio Estevam comment: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg122575.html > >> - Use 16 bits register read for this check as per Steve Longerbeam comment: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg122692.html > >> - Update bindings to document parallel mode support as per Fabio Estevam comment: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg122576.html > >> - Enable the whole 10 bits parallel output and document 8/10 bits support > >> in ov5640_set_stream_dvp() to answer to Steve Longerbeam comment: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg122693.html > >> > >> version 2: > >> - Fix comments from Sakari Ailus: > >> https://www.mail-archive.com/linux-media@vger.kernel.org/msg122259.html > >> - Revisit ov5640_set_stream_dvp() to only configure DVP at streamon > >> - Revisit ov5640_set_stream_dvp() implementation with fewer register settings > >> > >> version 1: > >> - Initial submission > >> > >> Hugues Fruchet (5): > >> media: ov5640: switch to gpiod_set_value_cansleep() > >> media: ov5640: check chip id > >> media: dt-bindings: ov5640: refine CSI-2 and add parallel interface > >> media: ov5640: add support of DVP parallel interface > >> media: ov5640: add support of RGB565 and YUYV formats > >> > >> .../devicetree/bindings/media/i2c/ov5640.txt | 46 ++- > >> drivers/media/i2c/ov5640.c | 325 ++++++++++++++++++--- > >> 2 files changed, 324 insertions(+), 47 deletions(-) > >> > >> -- > >> 1.9.1 > >> > >> -- > >> To unsubscribe from this list: send the line "unsubscribe devicetree" in > >> the body of a message to majordomo@vger.kernel.org > >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > > > > > Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England > & Wales under Registered No. 04586709. > > Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709. NrybXǧv^)޺{.n+zzz)w*jgݢj.۰\Mgja' ޢ j:+vwjmzZ+ݢj"!i