Re: [PATCH] dts-bindings: usb-uhci: Document newly added properties
On Fri, 2018-01-19 at 17:22 -0600, Rob Herring wrote: > On Tue, Jan 16, 2018 at 11:06:15AM +1100, Benjamin Herrenschmidt wrote: > > Commit 4642d34a439f80e16af0d56ed6258a33abae257a > > "usb/uhci: Add support for Aspeed BMC SoCs" added > > support for a new #ports property, along with > > Aspeed specific compatible strings. This adds > > documentation for them. > > > > Signed-off-by: Benjamin Herrenschmidt> > --- > > Documentation/devicetree/bindings/usb/usb-uhci.txt | 6 ++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/usb/usb-uhci.txt > > b/Documentation/devicetree/bindings/usb/usb-uhci.txt > > index 298133416c97..c1153eb38a9a 100644 > > --- a/Documentation/devicetree/bindings/usb/usb-uhci.txt > > +++ b/Documentation/devicetree/bindings/usb/usb-uhci.txt > > @@ -3,8 +3,14 @@ Generic Platform UHCI Controller > > > > Required properties: > > - compatible : "generic-uhci" (deprecated: "platform-uhci") > > + supports additional Aspeed specific workarounds by also > > + including the "aspeed,ast2400-uhci" or "aspeed,ast2500-uhci" > > + string. > > That's usually why we have SoC compatibles, so the description doesn't > really add anything. Can you format with one valid compatible list per > line: > > compatible : Should be one of: > "aspeed,ast2500-uhci", "generic-uhci" > "aspeed,ast2400-uhci", "generic-uhci" > "generic-uhci" > "platform-uhci" (deprecated) > > We should deprecate "generic-uhci" too IMO, but hardly anyone adds new > UHCI controllers. So I made a mistake in the code, I need both the aspeed specific compatible *and* the platform-uhci one since I didn't amend the driver to match on the aspeed ones. I suppose I could just send a fixup patch as nobody yet has deployed anything yet on the field. > > - reg : Should contain 1 register ranges(address and length) > > - interrupts : UHCI controller interrupt > > +- #ports: (optional) 1 cell, number of ports (overrides > > + auto-detection which is needed among others on some Aspeed > > + parts). > > > > Example: > > > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe devicetree" in > > the body of a message to majord...@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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] dts-bindings: usb-uhci: Document newly added properties
On Tue, Jan 16, 2018 at 11:06:15AM +1100, Benjamin Herrenschmidt wrote: > Commit 4642d34a439f80e16af0d56ed6258a33abae257a > "usb/uhci: Add support for Aspeed BMC SoCs" added > support for a new #ports property, along with > Aspeed specific compatible strings. This adds > documentation for them. > > Signed-off-by: Benjamin Herrenschmidt> --- > Documentation/devicetree/bindings/usb/usb-uhci.txt | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/usb-uhci.txt > b/Documentation/devicetree/bindings/usb/usb-uhci.txt > index 298133416c97..c1153eb38a9a 100644 > --- a/Documentation/devicetree/bindings/usb/usb-uhci.txt > +++ b/Documentation/devicetree/bindings/usb/usb-uhci.txt > @@ -3,8 +3,14 @@ Generic Platform UHCI Controller > > Required properties: > - compatible : "generic-uhci" (deprecated: "platform-uhci") > + supports additional Aspeed specific workarounds by also > + including the "aspeed,ast2400-uhci" or "aspeed,ast2500-uhci" > + string. That's usually why we have SoC compatibles, so the description doesn't really add anything. Can you format with one valid compatible list per line: compatible : Should be one of: "aspeed,ast2500-uhci", "generic-uhci" "aspeed,ast2400-uhci", "generic-uhci" "generic-uhci" "platform-uhci" (deprecated) We should deprecate "generic-uhci" too IMO, but hardly anyone adds new UHCI controllers. > - reg : Should contain 1 register ranges(address and length) > - interrupts : UHCI controller interrupt > +- #ports: (optional) 1 cell, number of ports (overrides > + auto-detection which is needed among others on some Aspeed > + parts). > > Example: > > > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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] dts-bindings: usb-uhci: Document newly added properties
On Mon, Jan 15, 2018 at 6:06 PM, Benjamin Herrenschmidtwrote: > Commit 4642d34a439f80e16af0d56ed6258a33abae257a > "usb/uhci: Add support for Aspeed BMC SoCs" added > support for a new #ports property, along with > Aspeed specific compatible strings. This adds > documentation for them. > > Signed-off-by: Benjamin Herrenschmidt Acked-by: Joel Stanley > --- > Documentation/devicetree/bindings/usb/usb-uhci.txt | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/usb-uhci.txt > b/Documentation/devicetree/bindings/usb/usb-uhci.txt > index 298133416c97..c1153eb38a9a 100644 > --- a/Documentation/devicetree/bindings/usb/usb-uhci.txt > +++ b/Documentation/devicetree/bindings/usb/usb-uhci.txt > @@ -3,8 +3,14 @@ Generic Platform UHCI Controller > > Required properties: > - compatible : "generic-uhci" (deprecated: "platform-uhci") > + supports additional Aspeed specific workarounds by also > + including the "aspeed,ast2400-uhci" or "aspeed,ast2500-uhci" > + string. > - reg : Should contain 1 register ranges(address and length) > - interrupts : UHCI controller interrupt > +- #ports: (optional) 1 cell, number of ports (overrides > + auto-detection which is needed among others on some Aspeed > + parts). > > Example: > > > -- 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] dts-bindings: usb-uhci: Document newly added properties
Commit 4642d34a439f80e16af0d56ed6258a33abae257a "usb/uhci: Add support for Aspeed BMC SoCs" added support for a new #ports property, along with Aspeed specific compatible strings. This adds documentation for them. Signed-off-by: Benjamin Herrenschmidt--- Documentation/devicetree/bindings/usb/usb-uhci.txt | 6 ++ 1 file changed, 6 insertions(+) diff --git a/Documentation/devicetree/bindings/usb/usb-uhci.txt b/Documentation/devicetree/bindings/usb/usb-uhci.txt index 298133416c97..c1153eb38a9a 100644 --- a/Documentation/devicetree/bindings/usb/usb-uhci.txt +++ b/Documentation/devicetree/bindings/usb/usb-uhci.txt @@ -3,8 +3,14 @@ Generic Platform UHCI Controller Required properties: - compatible : "generic-uhci" (deprecated: "platform-uhci") + supports additional Aspeed specific workarounds by also + including the "aspeed,ast2400-uhci" or "aspeed,ast2500-uhci" + string. - reg : Should contain 1 register ranges(address and length) - interrupts : UHCI controller interrupt +- #ports: (optional) 1 cell, number of ports (overrides + auto-detection which is needed among others on some Aspeed + parts). Example: -- 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