Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1563032

Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver

Show all headers | View raw

Hi Rob,

On 17-01-19 09:36 AM, Rob Herring wrote:
> On Tue, Jan 17, 2017 at 01:35:07PM +0530, Raviteja Garimella wrote:
>> This patch adds device tree bindings documentation for Synopsys
>> USB device controller platform driver.
>
> Bindings describe h/w, not drivers.
>>
>> Signed-off-by: Raviteja Garimella <raviteja.garimella@broadcom.com>
>> ---
>>  .../devicetree/bindings/usb/snps,dw-ahb-udc.txt    | 27 ++++++++++++++++++++++
>>  1 file changed, 27 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/usb/snps,dw-ahb-udc.txt
>>
>> diff --git a/Documentation/devicetree/bindings/usb/snps,dw-ahb-udc.txt b/Documentation/devicetree/bindings/usb/snps,dw-ahb-udc.txt
>> new file mode 100644
>> index 0000000..0c18327
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/usb/snps,dw-ahb-udc.txt
>> @@ -0,0 +1,27 @@
>> +Synopsys USB Device controller.
>> +
>> +The device node is used for Synopsys Designware Cores AHB
>> +Subsystem Device Controller (UDC).
>> +
>> +This device node is used by UDCs integrated it Broadcom's
>> +Northstar2 and Cygnus SoC's.
>
> You need compatible strings for these in addition.
>
We don't need compatibility strings when an IP block is integrated into 
an SoC.  Otherwise each time we add the IP block to a new SoC we would 
need to update ever linux driver that supports that SoC.  That doesn't 
make sense?

Cygnus and Northstar2 use existing drivers for such block as UARTs, SPI 
controllers, NAND controllers, etc, etc.  We haven't added compatibility 
strings for those drivers and won't be.

Perhaps comment above can be:
This device node is used by UDCs integrated it such as Broadcom's
Northstar2 and Cygnus SoC's.
>> +
>> +Required properties:
>> + - compatible: should be "snps,dw-ahb-udc"
>
> This is a different IP than DWC2?
>
>> + - reg: Offset and length of UDC register set
>> + - interrupts: description of interrupt line
>> + - phys: phandle to phy node.
>> + - extcon: phandle to the extcon device. This is optional and
>> +   not required for those that don't require extcon support.
>> +   Extcon support will be required if the UDC is connected to
>> +   a Dual Role Device Phy that supports both Host and Device
>> +   mode based on the external cable.
>
> Drop this. It should be a part of the phy. Also, I don't care to see new
> users of extcon binding because it needs redoing.
>
We may need extcon support to support DRD Phy though.  We have to work 
within the framework that exists in linux today.  If modified in the 
future adapt to it as needed?

Regards,
  Scott

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC v2 0/5] Platform driver support for 'amd5536udc' driver  Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-17 09:10 +0100
  [RFC v2 3/5] UDC: Provide correct arguments for 'dma_pool_create' Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-17 09:10 +0100
  [RFC v2 2/5] UDC: make debug prints compatible with both pci and platform devices Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-17 09:10 +0100
  [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-17 09:20 +0100
    Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Rob Herring <robh@kernel.org> - 2017-01-19 18:40 +0100
      Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Scott Branden <scott.branden@broadcom.com> - 2017-01-19 20:40 +0100
        Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Florian Fainelli <f.fainelli@gmail.com> - 2017-01-19 20:50 +0100
          Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Scott Branden <scott.branden@broadcom.com> - 2017-01-19 21:10 +0100
            Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Florian Fainelli <f.fainelli@gmail.com> - 2017-01-19 21:20 +0100
              Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Ray Jui <ray.jui@broadcom.com> - 2017-01-19 23:10 +0100
                Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Scott Branden <scott.branden@broadcom.com> - 2017-01-19 23:40 +0100
                Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform  driver Florian Fainelli <florian.fainelli@broadcom.com> - 2017-01-20 00:00 +0100
                Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver Rob Herring <robh@kernel.org> - 2017-01-20 15:00 +0100
      Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-20 13:10 +0100
  [RFC v2 5/5] UDC: Add Synopsys UDC Platform driver Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-17 09:20 +0100
  Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys  core driver Florian Fainelli <f.fainelli@gmail.com> - 2017-01-18 20:00 +0100
    Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys  core driver Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-01-18 21:20 +0100
    Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys  core driver Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-19 11:50 +0100
      Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys  core driver Florian Fainelli <f.fainelli@gmail.com> - 2017-01-19 20:30 +0100
        Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys  core driver Raviteja Garimella <raviteja.garimella@broadcom.com> - 2017-01-23 14:10 +0100

csiph-web