On 09/06/17 16:07, Rob Herring wrote: > On Wed, Jun 07, 2017 at 04:46:14PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil <hans.verkuil@xxxxxxxxx> >> >> Needed for boards that wire the CEC pin in such a way that it >> is unavailable when the HPD is low. >> >> Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> >> Cc: Krzysztof Kozlowski <krzk@xxxxxxxxxx> >> Cc: Andrzej Hajda <a.hajda@xxxxxxxxxxx> >> Cc: devicetree@xxxxxxxxxxxxxxx >> --- >> Documentation/devicetree/bindings/media/s5p-cec.txt | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/media/s5p-cec.txt b/Documentation/devicetree/bindings/media/s5p-cec.txt >> index 4bb08d9d940b..261af4d1a791 100644 >> --- a/Documentation/devicetree/bindings/media/s5p-cec.txt >> +++ b/Documentation/devicetree/bindings/media/s5p-cec.txt >> @@ -17,6 +17,12 @@ Required properties: >> - samsung,syscon-phandle - phandle to the PMU system controller >> - hdmi-phandle - phandle to the HDMI controller >> >> +Optional: >> + - needs-hpd : if present the CEC support is only available when the HPD >> + is high. Some boards only let the CEC pin through if the HPD is high, for >> + example if there is a level converter that uses the HPD to power up >> + or down. > > Seems like something common. Can you document in a common location? Should we do the same with hdmi-phandle? It is also used by CEC drivers to find the HDMI driver. Currently only used by s5p-cec and stih-cec, but there will be more. I guess this would be a sensible place to document this: Documentation/devicetree/bindings/media/cec.txt Regards, Hans > >> + >> Example: >> >> hdmicec: cec@100B0000 { >> -- >> 2.11.0 >> >> _______________________________________________ >> dri-devel mailing list >> dri-devel@xxxxxxxxxxxxxxxxxxxxx >> https://lists.freedesktop.org/mailman/listinfo/dri-devel