devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dts-bindings: usb-uhci: Document newly added properties
@ 2018-01-16  0:06 Benjamin Herrenschmidt
       [not found] ` <1516061175.31850.140.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2018-01-16  0:06 UTC (permalink / raw)
  To: devicetree; +Cc: Joel Stanley, linux-usb-u79uwXL29TY76Z2rM5mHXA

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 <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
---
 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 devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] dts-bindings: usb-uhci: Document newly added properties
       [not found] ` <1516061175.31850.140.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
@ 2018-01-18 20:15   ` Joel Stanley
  2018-01-19 23:22   ` Rob Herring
  1 sibling, 0 replies; 4+ messages in thread
From: Joel Stanley @ 2018-01-18 20:15 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: devicetree, linux-usb-u79uwXL29TY76Z2rM5mHXA

On Mon, Jan 15, 2018 at 6:06 PM, Benjamin Herrenschmidt
<benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> 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 <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>

Acked-by: Joel Stanley <joel-U3u1mxZcP9KHXe+LvDLADg@public.gmane.org>


> ---
>  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 majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dts-bindings: usb-uhci: Document newly added properties
       [not found] ` <1516061175.31850.140.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
  2018-01-18 20:15   ` Joel Stanley
@ 2018-01-19 23:22   ` Rob Herring
  2018-01-20  0:38     ` Benjamin Herrenschmidt
  1 sibling, 1 reply; 4+ messages in thread
From: Rob Herring @ 2018-01-19 23:22 UTC (permalink / raw)
  To: Benjamin Herrenschmidt
  Cc: devicetree, Joel Stanley, linux-usb-u79uwXL29TY76Z2rM5mHXA

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 <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
> ---
>  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 majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dts-bindings: usb-uhci: Document newly added properties
  2018-01-19 23:22   ` Rob Herring
@ 2018-01-20  0:38     ` Benjamin Herrenschmidt
  0 siblings, 0 replies; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2018-01-20  0:38 UTC (permalink / raw)
  To: Rob Herring; +Cc: devicetree, Joel Stanley, linux-usb-u79uwXL29TY76Z2rM5mHXA

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 <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
> > ---
> >  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 majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-01-20  0:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-16  0:06 [PATCH] dts-bindings: usb-uhci: Document newly added properties Benjamin Herrenschmidt
     [not found] ` <1516061175.31850.140.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2018-01-18 20:15   ` Joel Stanley
2018-01-19 23:22   ` Rob Herring
2018-01-20  0:38     ` Benjamin Herrenschmidt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).