From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sylwester Nawrocki Subject: Re: [PATCH 1/2] dt-bindings: add media/cec.txt Date: Mon, 12 Jun 2017 14:59:08 +0200 Message-ID: References: <20170609175401.40204-1-hverkuil@xs4all.nl> <20170609175401.40204-2-hverkuil@xs4all.nl> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <20170609175401.40204-2-hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> Content-language: en-GB Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Hans Verkuil Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, Hans Verkuil , Krzysztof Kozlowski , Andrzej Hajda , Benjamin Gaignard , Rob Herring List-Id: devicetree@vger.kernel.org On 06/09/2017 07:54 PM, Hans Verkuil wrote: > From: Hans Verkuil > > Document common HDMI CEC bindings. Add this to the MAINTAINERS file > as well. > > Signed-off-by: Hans Verkuil Acked-by: Sylwester Nawrocki > --- > Documentation/devicetree/bindings/media/cec.txt | 8 ++++++++ > MAINTAINERS | 1 + > 2 files changed, 9 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/cec.txt > > diff --git a/Documentation/devicetree/bindings/media/cec.txt b/Documentation/devicetree/bindings/media/cec.txt > new file mode 100644 > index 000000000000..22d7aae3d3d7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/cec.txt > @@ -0,0 +1,8 @@ > +Common bindings for HDMI CEC adapters > + > +- hdmi-phandle: phandle to the HDMI controller. For a common property I would rather make it "hdmi-controller", this "-phandle" part feels like appending "_pointer" to variable names in C code. But since you are just rearranging existing documentation and Rob agrees with that I have no objections. > +- 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. > diff --git a/MAINTAINERS b/MAINTAINERS > index 053c3bdd1fe5..4ac340d189a3 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3137,6 +3137,7 @@ F: include/media/cec.h > F: include/media/cec-notifier.h > F: include/uapi/linux/cec.h > F: include/uapi/linux/cec-funcs.h > +F: Documentation/devicetree/bindings/media/cec.txt > > CELL BROADBAND ENGINE ARCHITECTURE > M: Arnd Bergmann