Re: [PATCH v2 2/2] media: atmel-isc: Update device tree binding documentation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 18.01.2019 16:28, Ken Sloat wrote:
> From: Ken Sloat <ksloat@xxxxxxxxxxxxxx>
> 
> Update device tree binding documentation specifying how to
> enable BT656 with CRC decoding.
> 
> Signed-off-by: Ken Sloat <ksloat@xxxxxxxxxxxxxx>
> ---
>   Changes in v2:
>   -Use correct media "bus-type" dt property.
>   
>   Documentation/devicetree/bindings/media/atmel-isc.txt | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/media/atmel-isc.txt b/Documentation/devicetree/bindings/media/atmel-isc.txt
> index bbe0e87c6188..2d4378dfd6c8 100644
> --- a/Documentation/devicetree/bindings/media/atmel-isc.txt
> +++ b/Documentation/devicetree/bindings/media/atmel-isc.txt
> @@ -21,6 +21,11 @@ Required properties for ISC:
>   - pinctrl-names, pinctrl-0
>   	Please refer to pinctrl-bindings.txt.
>   
> +Optional properties for ISC:
> +- bus-type
> +	When set to 6, Bt.656 decoding (embedded sync) with CRC decoding
> +	is enabled.
> +

I don't think this patch is required at all actually, the binding 
complies to the video-interfaces bus specification which includes the 
parallel and bt.656.

Would be worth mentioning below explicitly that parallel and bt.656 are 
supported, or added above that also plain parallel bus is supported ?

>   ISC supports a single port node with parallel bus. It should contain one

here inside the previous line

>   'port' child node with child 'endpoint' node. Please refer to the bindings
>   defined in Documentation/devicetree/bindings/media/video-interfaces.txt.
> 




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux