Hi Jacopo,
On 2018-05-16 18:32:26 +0200, Jacopo Mondi wrote:
> Hello,
>this series touches the bindings and the driver handling endpoint
> properties for digital subdevices of the R-Car VIN driver.
>
> The first patch simply documents what are the endpoint properties supported
> at the moment, then the second one extends them with 'data-active'.
>
> As the VIN hardware allows to use HSYNC as data enable signal when the CLCKENB
> pin is left unconnected, the 'data-active' property presence determinates
> if HSYNC has to be used or not as data enable signal. As a consequence, when
> running with embedded synchronism, and there is not HSYNC signal, it becomes
> mandatory to specify 'data-active' polarity in DTS.
>
> To address this, all Gen-2 boards featuring a composite video input and
> running with embedded synchronization, now need that property to be specified
> in DTS. Before adding it, remove un-used properties as 'pclk-sample' and
> 'bus-width' from the Gen-2 bindings, as they are not parsed by the VIN driver
> and only confuse users.
>
> Niklas, as you already know I don't have any Gen2 board. Could you give this
> a spin on Koelsch if you like the series?
I tested this on my Koelsch and capture is still working :-)
>
> Thanks
>j
>
> Jacopo Mondi (6):
> dt-bindings: media: rcar-vin: Describe optional ep properties
> dt-bindings: media: rcar-vin: Document data-active
> media: rcar-vin: Handle data-active property
> media: rcar-vin: Handle CLOCKENB pin polarity
> ARM: dts: rcar-gen2: Remove unused VIN properties
> ARM: dts: rcar-gen2: Add 'data-active' property
>
> Documentation/devicetree/bindings/media/rcar_vin.txt | 18 +-
> arch/arm/boot/dts/r8a7790-lager.dts | 4 +---
> arch/arm/boot/dts/r8a7791-koelsch.dts| 4 +---
> arch/arm/boot/dts/r8a7791-porter.dts | 2 +-
> arch/arm/boot/dts/r8a7793-gose.dts | 4 +---
> arch/arm/boot/dts/r8a7794-alt.dts| 2 +-
> arch/arm/boot/dts/r8a7794-silk.dts | 2 +-
> drivers/media/platform/rcar-vin/rcar-core.c | 10 --
> drivers/media/platform/rcar-vin/rcar-dma.c | 11 +++
> 9 files changed, 42 insertions(+), 15 deletions(-)
>
> --
> 2.7.4
>
--
Regards,
Niklas Söderlund