On Fri, Jun 9, 2017 at 9:11 AM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > 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. Yes. > 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 Sounds good. You can do this as a follow-up to this patch if you want. For this one: Acked-by: Rob Herring <robh@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html