On 12/28/18 17:59, 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> > --- > Documentation/devicetree/bindings/media/atmel-isc.txt | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/atmel-isc.txt b/Documentation/devicetree/bindings/media/atmel-isc.txt > index bbe0e87c6188..e787edeea7da 100644 > --- a/Documentation/devicetree/bindings/media/atmel-isc.txt > +++ b/Documentation/devicetree/bindings/media/atmel-isc.txt > @@ -25,6 +25,9 @@ ISC supports a single port node with parallel bus. It should contain one > 'port' child node with child 'endpoint' node. Please refer to the bindings > defined in Documentation/devicetree/bindings/media/video-interfaces.txt. > > +If all endpoint bus flags (i.e. hsync-active) are omitted, then CCIR656 > +decoding (embedded sync) with CRC decoding is enabled. Sorry, this is wrong. There is a bus-type property defined in video-interfaces.txt that you should use to determine whether this is a parallel or a Bt.656 bus. BTW, for v2 also CC this to devicetree@xxxxxxxxxxxxxxx, since it has to be reviewed by the DT maintainers. Regards, Hans > + > Example: > isc: isc@f0008000 { > compatible = "atmel,sama5d2-isc"; >