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@xxxxxxxxxxxxxxx/msg124048.html > > version 4: > - Refine bindings as per Sakari suggestion: > https://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/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@xxxxxxxxxxxxxxx/msg122575.html > - Use 16 bits register read for this check as per Steve Longerbeam comment: > https://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg122692.html > - Update bindings to document parallel mode support as per Fabio Estevam comment: > https://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/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@xxxxxxxxxxxxxxx/msg122693.html > > version 2: > - Fix comments from Sakari Ailus: > https://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/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@xxxxxxxxxxxxxxx > 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.
