diff mbox series

[02/11] dt-bindings: usb: mtk-xhci: add optional property to disable usb2 ports

Message ID 1627635002-24521-2-git-send-email-chunfeng.yun@mediatek.com
State Accepted
Commit 51018cde5b55b7b0d65af0d363531cddd360fe11
Headers show
Series [01/11] dt-bindings: usb: mtk-xhci: add support property 'tpl-support' | expand

Commit Message

Chunfeng Yun (云春峰) July 30, 2021, 8:49 a.m. UTC
Add support to disable specific usb2 host ports, it's useful when
a usb2 port is disabled on some platforms, but enabled on others for
the same SoC, another case is that the different package may support
different number of ports.

Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
---
 Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Rob Herring (Arm) Aug. 6, 2021, 8:41 p.m. UTC | #1
On Fri, Jul 30, 2021 at 04:49:53PM +0800, Chunfeng Yun wrote:
> Add support to disable specific usb2 host ports, it's useful when

> a usb2 port is disabled on some platforms, but enabled on others for

> the same SoC, another case is that the different package may support

> different number of ports.

> 

> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>

> ---

>  Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml | 5 +++++

>  1 file changed, 5 insertions(+)

> 

> diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml

> index 49729aab6d1a..61a0e550b5d6 100644

> --- a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml

> +++ b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml

> @@ -154,6 +154,11 @@ properties:

>      description: The mask to disable u3ports, bit0 for u3port0,

>        bit1 for u3port1, ... etc

>  

> +  mediatek,u2p-dis-msk:

> +    $ref: /schemas/types.yaml#/definitions/uint32

> +    description: The mask to disable u2ports, bit0 for u2port0,

> +      bit1 for u2port1, ... etc


You already have this property in the parent node, right? Why are we 
duplicating it in both places?

Rob
Chunfeng Yun (云春峰) Aug. 11, 2021, 8:12 a.m. UTC | #2
On Fri, 2021-08-06 at 14:41 -0600, Rob Herring wrote:
> On Fri, Jul 30, 2021 at 04:49:53PM +0800, Chunfeng Yun wrote:

> > Add support to disable specific usb2 host ports, it's useful when

> > a usb2 port is disabled on some platforms, but enabled on others

> > for

> > the same SoC, another case is that the different package may

> > support

> > different number of ports.

> > 

> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>

> > ---

> >  Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml | 5

> > +++++

> >  1 file changed, 5 insertions(+)

> > 

> > diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtk-

> > xhci.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtk-

> > xhci.yaml

> > index 49729aab6d1a..61a0e550b5d6 100644

> > --- a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml

> > +++ b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml

> > @@ -154,6 +154,11 @@ properties:

> >      description: The mask to disable u3ports, bit0 for u3port0,

> >        bit1 for u3port1, ... etc

> >  

> > +  mediatek,u2p-dis-msk:

> > +    $ref: /schemas/types.yaml#/definitions/uint32

> > +    description: The mask to disable u2ports, bit0 for u2port0,

> > +      bit1 for u2port1, ... etc

> 

> You already have this property in the parent node, right? Why are we 

> duplicating it in both places?

Some SoC may only support xHCI controller, then only use mtk-xhci node
without parent mtu3 node which is used for dual-role controller.
it's mainly for backward compatible.

Thanks a lot

> 

> Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml
index 49729aab6d1a..61a0e550b5d6 100644
--- a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml
+++ b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml
@@ -154,6 +154,11 @@  properties:
     description: The mask to disable u3ports, bit0 for u3port0,
       bit1 for u3port1, ... etc
 
+  mediatek,u2p-dis-msk:
+    $ref: /schemas/types.yaml#/definitions/uint32
+    description: The mask to disable u2ports, bit0 for u2port0,
+      bit1 for u2port1, ... etc
+
   "#address-cells":
     const: 1