Re: [PATCH 5/6] DT bindings documentation for Broadcom IPROC USB Device controller.
Hi Rob, On Wed, Feb 1, 2017 at 9:43 PM, Rob Herringwrote: > On Mon, Jan 30, 2017 at 01:26:12PM +0530, Raviteja Garimella wrote: >> The device node is used for UDCs integrated into Broadcom's >> iProc family of SoCs'. The UDC is based on Synopsys Designware >> Cores AHB Subsystem USB Device Controller IP. >> >> Signed-off-by: Raviteja Garimella >> --- >> .../bindings/usb/brcm,iproc-snps-udc.txt | 24 >> ++ >> 1 file changed, 24 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt >> >> diff --git a/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt >> b/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt >> new file mode 100644 >> index 000..537dd4d >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt >> @@ -0,0 +1,24 @@ >> +Broadcom IPROC USB Device controller. >> + >> +The device node is used for UDCs integrated into Broadcom's >> +iProc family of SoCs'. The UDC is based on Synopsys Designware >> +Cores AHB Subsystem Device Controller. >> + >> +Required properties: >> + - compatible: should be "brcm,iproc-snps-udc" > > Adding "snps" in here adds nothing really. Again, please use per SoC > compatible strings. Will do. > >> + - 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 property. Ok. Thanks, Ravi > >> + >> +Example: >> + udc_dwc: usb@664e { >> + compatible = "brcm,iproc-snps-udc"; >> + reg = <0x664e 0x2000>; >> + interrupts = ; >> + phys = <_phy>; >> + extcon = <_phy>"; >> -- >> 2.1.0 >> -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 5/6] DT bindings documentation for Broadcom IPROC USB Device controller.
On Mon, Jan 30, 2017 at 01:26:12PM +0530, Raviteja Garimella wrote: > The device node is used for UDCs integrated into Broadcom's > iProc family of SoCs'. The UDC is based on Synopsys Designware > Cores AHB Subsystem USB Device Controller IP. > > Signed-off-by: Raviteja Garimella> --- > .../bindings/usb/brcm,iproc-snps-udc.txt | 24 > ++ > 1 file changed, 24 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt > > diff --git a/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt > b/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt > new file mode 100644 > index 000..537dd4d > --- /dev/null > +++ b/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt > @@ -0,0 +1,24 @@ > +Broadcom IPROC USB Device controller. > + > +The device node is used for UDCs integrated into Broadcom's > +iProc family of SoCs'. The UDC is based on Synopsys Designware > +Cores AHB Subsystem Device Controller. > + > +Required properties: > + - compatible: should be "brcm,iproc-snps-udc" Adding "snps" in here adds nothing really. Again, please use per SoC compatible strings. > + - 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 property. > + > +Example: > + udc_dwc: usb@664e { > + compatible = "brcm,iproc-snps-udc"; > + reg = <0x664e 0x2000>; > + interrupts = ; > + phys = <_phy>; > + extcon = <_phy>"; > -- > 2.1.0 > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 5/6] DT bindings documentation for Broadcom IPROC USB Device controller.
The device node is used for UDCs integrated into Broadcom's iProc family of SoCs'. The UDC is based on Synopsys Designware Cores AHB Subsystem USB Device Controller IP. Signed-off-by: Raviteja Garimella--- .../bindings/usb/brcm,iproc-snps-udc.txt | 24 ++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt diff --git a/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt b/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt new file mode 100644 index 000..537dd4d --- /dev/null +++ b/Documentation/devicetree/bindings/usb/brcm,iproc-snps-udc.txt @@ -0,0 +1,24 @@ +Broadcom IPROC USB Device controller. + +The device node is used for UDCs integrated into Broadcom's +iProc family of SoCs'. The UDC is based on Synopsys Designware +Cores AHB Subsystem Device Controller. + +Required properties: + - compatible: should be "brcm,iproc-snps-udc" + - 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. + +Example: + udc_dwc: usb@664e { + compatible = "brcm,iproc-snps-udc"; + reg = <0x664e 0x2000>; + interrupts = ; + phys = <_phy>; + extcon = <_phy>"; -- 2.1.0 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html