RE: [PATCH v13 02/33] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2018-04-10 Thread Fabrizio Castro
Hello Niklas, > Subject: Re: [PATCH v13 02/33] dt-bindings: media: rcar_vin: add device tree > support for r8a774[35] > > Hi Fabrizio, > > On 2018-04-10 09:55:29 +0000, Fabrizio Castro wrote: > > Dear All, > > > > this patch was originally sent on the

RE: [PATCH v13 02/33] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2018-04-10 Thread Fabrizio Castro
To: Laurent Pinchart <laurent.pinch...@ideasonboard.com>; Hans Verkuil > <hverk...@xs4all.nl>; linux-media@vger.kernel.org > Cc: linux-renesas-...@vger.kernel.org; TOMOHARU FUKAWA > <tomoharu.fukawa...@renesas.com>; Kieran Bingham > <kieran.bing...@ideaso

RE: [PATCH v5 4/5] media: ov5640: add support of DVP parallel interface

2018-02-05 Thread Fabrizio Castro
Hello Maxime, thank you for your feedback. > > > +/* > > > + * configure parallel port control lines polarity > > > + * > > > + * POLARITY CTRL0 > > > + * - [5]:PCLK polarity (0: active low, 1: active high) > > > + * - [1]:HREF polarity (0: active low, 1: active high) > > > + * - [0]:VSYNC

RE: [PATCH v5 4/5] media: ov5640: add support of DVP parallel interface

2018-02-01 Thread Fabrizio Castro
Hello Hugues, > Subject: [PATCH v5 4/5] media: ov5640: add support of DVP parallel interface > > Add support of DVP parallel mode in addition of > existing MIPI CSI mode. The choice between two modes > and configuration is made through device tree. > > Signed-off-by: Hugues Fruchet

RE: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2018-01-24 Thread Fabrizio Castro
allow > checkpatch.pl to validate compatible string values. > > Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> > Reviewed-by: Biju Das <biju@bp.renesas.com> > --- > v1->v2: > * Fixed double "change" in changelog > > Documentation/

RE: [PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number list

2018-01-24 Thread Fabrizio Castro
g to > match with other documentation. > > Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> > Reviewed-by: Biju Das <biju@bp.renesas.com> > --- > v1->v2: > * new patch triggered by Geert's comment, see the below link for details: > https

RE: [PATCH v5 0/5] Add OV5640 parallel interface and RGB565/YUYV support

2018-01-09 Thread Fabrizio Castro
40 -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/20

RE: [PATCH v5 0/5] Add OV5640 parallel interface and RGB565/YUYV support

2018-01-08 Thread Fabrizio Castro
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

RE: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-12-19 Thread Fabrizio Castro
l activate the right code. > However, it is good practice to document compatible strings for the > specific SoC as this allows SoC specific changes to the driver if > needed, in addition to document SoC support and therefore allow > checkpatch.pl to validate compatible string value

[PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number list

2017-11-16 Thread Fabrizio Castro
Change the sorting of the part numbers from descending to ascending to match with other documentation. Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> Reviewed-by: Biju Das <biju@bp.renesas.com> --- v1->v2: * new patch triggered by Geert's comment, see

[PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-16 Thread Fabrizio Castro
o document SoC support and therefore allow checkpatch.pl to validate compatible string values. Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> Reviewed-by: Biju Das <biju@bp.renesas.com> --- v1->v2: * Fixed double "change" in changelog Documentation/dev

[PATCH v2 0/4] Add VIN support to r8a774[35]

2017-11-16 Thread Fabrizio Castro
Hello, this series documents VIN related dt-bindings for r8a774[35], adds VIN[012] nodes to the r8a7743 SoC dtsi and adds VIN[01] nodes to the r8a7745 SoC dtsi. Best regards, Fabrizio Castro (4): dt-bindings: media: rcar_vin: Reverse SoC part number list dt-bindings: media: rcar_vin: add

RE: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-16 Thread Fabrizio Castro
Hi Geert, > >> > >> Please keep the list sorted by SoC part number. > > > > It is sorted, just in descending order. Do you want me to re-order the full > > list in ascending order? > > That may be a good idea, given the current order is non-standard and > counter-intuitive. sure, dropping this

RE: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-16 Thread Fabrizio Castro
Hello Geert, thank you for your comment! > Subject: Re: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree > support for r8a774[35] > > On Thu, Nov 16, 2017 at 1:11 PM, Fabrizio Castro > <fabrizio.cas...@bp.renesas.com> wrote: > > --- a/Documentatio

RE: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-16 Thread Fabrizio Castro
Hello Geert, thank you for your comment! > > Add compatible strings for r8a7743 and r8a7745. No driver change > > change is needed as "renesas,rcar-gen2-vin" will activate the right > > double "change" ps, do you think a v2 is in order? Thanks, Fab Renesas Electronics Europe Ltd, Dukes

[PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-16 Thread Fabrizio Castro
n addition to document SoC support and therefore allow checkpatch.pl to validate compatible string values. Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> Reviewed-by: Biju Das <biju@bp.renesas.com> --- Documentation/devicetree/bindings/media/rcar_vin.txt | 5 ++

[PATCH 0/2] Add VIN support to r8a7743

2017-11-16 Thread Fabrizio Castro
Hello, this series documents VIN related dt-bindings for r8a774[35], and adds VIN[012] nodes to the r8a7743 SoC dtsi. Best regards, Fabrizio Castro (2): dt-bindings: media: rcar_vin: add device tree support for r8a774[35] ARM: dts: r8a7743: add VIN dt support .../devicetree/bindings/media