Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII;
 format=flowed
Content-Transfer-Encoding: 7bit
Date:   Thu, 02 Aug 2018 11:44:50 -0700
From:   pheragu@codeaurora.org
To:     Rob Herring <robh@kernel.org>
Cc:     linux-arm-msm@vger.kernel.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        tsoni@codeaurora.org, ckadabi@codeaurora.org,
        bryanh@codeaurora.org, psodagud@codeaurora.org,
        satyap@codeaurora.org
Subject: Re: [PATCH 1/2] dt-bindings: Documentation for qcom,eud
In-Reply-To: <CABGGiswcMicPypOz2eW01NosuAHUw2B3KVxzrOfanEfT=Srf9A@mail.gmail.com>
References: <1533061333-7680-1-git-send-email-pheragu@codeaurora.org>
 <1533061333-7680-2-git-send-email-pheragu@codeaurora.org>
 <CABGGiswcMicPypOz2eW01NosuAHUw2B3KVxzrOfanEfT=Srf9A@mail.gmail.com>
Message-ID: <4d6732e9172431bbf24ff470be345481@codeaurora.org>
User-Agent: Roundcube Webmail/1.2.5
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 2018-08-01 07:04, Rob Herring wrote:
> On Tue, Jul 31, 2018 at 12:22 PM Prakruthi Deepak Heragu
> <pheragu@codeaurora.org> 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 <satyap@codeaurora.org>
>> Signed-off-by: Prakruthi Deepak Heragu <pheragu@codeaurora.org>
>> ---
>>  .../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 = <GIC_SPI 492 IRQ_TYPE_LEVEL_HIGH>;
>> +               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
>>