From mboxrd@z Thu Jan 1 00:00:00 1970 From: pheragu@codeaurora.org Subject: Re: [PATCH 1/2] dt-bindings: Documentation for qcom,eud Date: Thu, 02 Aug 2018 11:44:50 -0700 Message-ID: <4d6732e9172431bbf24ff470be345481@codeaurora.org> References: <1533061333-7680-1-git-send-email-pheragu@codeaurora.org> <1533061333-7680-2-git-send-email-pheragu@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Rob Herring Cc: linux-arm-msm@vger.kernel.org, Linux Kernel Mailing List , tsoni@codeaurora.org, ckadabi@codeaurora.org, bryanh@codeaurora.org, psodagud@codeaurora.org, satyap@codeaurora.org List-Id: linux-arm-msm@vger.kernel.org On 2018-08-01 07:04, Rob Herring wrote: > On Tue, Jul 31, 2018 at 12:22 PM Prakruthi Deepak Heragu > wrote: >> >> Documentation for Embedded USB Debugger (EUD) device tree bindings. > > If you want bindings reviewed, send them to the correct list and > people (hint: run get_maintainers.pl). > > New bindings should not use extcon. > Why should we not use extcon? Are there any alternatives which you would suggest? We were looking at graphs instead of extcon. >> >> Signed-off-by: Satya Durga Srinivasu Prabhala >> Signed-off-by: Prakruthi Deepak Heragu >> --- >> .../devicetree/bindings/soc/qcom/qcom,msm-eud.txt | 28 >> ++++++++++++++++++++++ >> 1 file changed, 28 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/soc/qcom/qcom,msm-eud.txt >> >> diff --git >> a/Documentation/devicetree/bindings/soc/qcom/qcom,msm-eud.txt >> b/Documentation/devicetree/bindings/soc/qcom/qcom,msm-eud.txt >> new file mode 100644 >> index 0000000..ea63f1c >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,msm-eud.txt >> @@ -0,0 +1,28 @@ >> +* Qualcomm Technologies Inc Embedded USB Debugger (EUD) >> + >> +The EUD (Embedded USB Debugger) is a mini-USB hub implemented >> +on chip to support the USB-based debug and trace capabilities. >> + >> +Required properties: >> + >> + - compatible: Should be "qcom,msm-eud" >> + - interrupts: Interrupt number >> + - reg: Should be address and size of EUD register space >> + >> +Driver notifies clients via extcon for VBUS attach/detach >> +and charger enable/disable events. Clients registered for these >> +notifications should have extcon property set to eud. >> + >> +An example for EUD device node: >> + >> + eud: qcom,msm-eud@88e0000 { >> + compatible = "qcom,msm-eud"; >> + interrupts = ; >> + reg = <0x88e0000 0x4000>; >> + }; >> + >> +An example for EUD extcon client: >> + >> + usb3 { >> + extcon = <&eud>; >> + }; >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora >> Forum, >> a Linux Foundation Collaborative Project >>