* [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-08 9:17 ` Chunfeng Yun
0 siblings, 0 replies; 11+ messages in thread
From: Chunfeng Yun @ 2019-05-08 9:17 UTC (permalink / raw)
To: Rob Herring
Cc: Greg Kroah-Hartman, Mark Rutland, Matthias Brugger, linux-usb,
devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Biju Das, Yu Chen, Heikki Krogerus, Min Guo, Felipe Balbi,
Chunfeng Yun
Add a property usb-role-switch to tell the driver that use
USB Role Switch framework to handle the role switch,
it's useful when the driver has already supported other ways,
such as extcon framework etc.
Cc: Biju Das <biju.das@bp.renesas.com>
Cc: Yu Chen <chenyu56@huawei.com>
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
---
v3:
add property type, modify description suggested by Heikki
v2:
describe it in terms of h/w functionality suggested by Rob
v1:
the property is discussed in:
[v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
https://patchwork.kernel.org/patch/10852497/
Mediatek and Hisilicon also try to use it:
[v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
https://patchwork.kernel.org/patch/10918385/
[v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
https://patchwork.kernel.org/patch/10918367/
[v6,10/13] usb: dwc3: Registering a role switch in the DRD code
https://patchwork.kernel.org/patch/10909981/
---
Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
index 0a74ab8dfdc2..cf5a1ad456e6 100644
--- a/Documentation/devicetree/bindings/usb/generic.txt
+++ b/Documentation/devicetree/bindings/usb/generic.txt
@@ -30,6 +30,10 @@ Optional properties:
optional for OTG device.
- adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
optional for OTG device.
+ - usb-role-switch: boolean, indicates that the device is capable of assigning
+ the USB data role (USB host or USB device) for a given
+ USB connector, such as Type-C, Type-B(micro).
+ see connector/usb-connector.txt.
This is an attribute to a USB controller such as:
--
2.21.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-08 9:17 ` Chunfeng Yun
0 siblings, 0 replies; 11+ messages in thread
From: Chunfeng Yun @ 2019-05-08 9:17 UTC (permalink / raw)
To: Rob Herring
Cc: Greg Kroah-Hartman, Mark Rutland, Matthias Brugger, linux-usb,
devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Biju Das, Yu Chen, Heikki Krogerus, Min Guo, Felipe Balbi,
Chunfeng Yun
Add a property usb-role-switch to tell the driver that use
USB Role Switch framework to handle the role switch,
it's useful when the driver has already supported other ways,
such as extcon framework etc.
Cc: Biju Das <biju.das@bp.renesas.com>
Cc: Yu Chen <chenyu56@huawei.com>
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
---
v3:
add property type, modify description suggested by Heikki
v2:
describe it in terms of h/w functionality suggested by Rob
v1:
the property is discussed in:
[v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
https://patchwork.kernel.org/patch/10852497/
Mediatek and Hisilicon also try to use it:
[v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
https://patchwork.kernel.org/patch/10918385/
[v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
https://patchwork.kernel.org/patch/10918367/
[v6,10/13] usb: dwc3: Registering a role switch in the DRD code
https://patchwork.kernel.org/patch/10909981/
---
Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
index 0a74ab8dfdc2..cf5a1ad456e6 100644
--- a/Documentation/devicetree/bindings/usb/generic.txt
+++ b/Documentation/devicetree/bindings/usb/generic.txt
@@ -30,6 +30,10 @@ Optional properties:
optional for OTG device.
- adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
optional for OTG device.
+ - usb-role-switch: boolean, indicates that the device is capable of assigning
+ the USB data role (USB host or USB device) for a given
+ USB connector, such as Type-C, Type-B(micro).
+ see connector/usb-connector.txt.
This is an attribute to a USB controller such as:
--
2.21.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-08 9:17 ` Chunfeng Yun
0 siblings, 0 replies; 11+ messages in thread
From: Chunfeng Yun @ 2019-05-08 9:17 UTC (permalink / raw)
To: Rob Herring
Cc: Mark Rutland, devicetree, Heikki Krogerus, Felipe Balbi,
Greg Kroah-Hartman, linux-usb, Yu Chen, linux-kernel, Biju Das,
Chunfeng Yun, linux-mediatek, Min Guo, Matthias Brugger,
linux-arm-kernel
Add a property usb-role-switch to tell the driver that use
USB Role Switch framework to handle the role switch,
it's useful when the driver has already supported other ways,
such as extcon framework etc.
Cc: Biju Das <biju.das@bp.renesas.com>
Cc: Yu Chen <chenyu56@huawei.com>
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
---
v3:
add property type, modify description suggested by Heikki
v2:
describe it in terms of h/w functionality suggested by Rob
v1:
the property is discussed in:
[v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
https://patchwork.kernel.org/patch/10852497/
Mediatek and Hisilicon also try to use it:
[v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
https://patchwork.kernel.org/patch/10918385/
[v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
https://patchwork.kernel.org/patch/10918367/
[v6,10/13] usb: dwc3: Registering a role switch in the DRD code
https://patchwork.kernel.org/patch/10909981/
---
Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
index 0a74ab8dfdc2..cf5a1ad456e6 100644
--- a/Documentation/devicetree/bindings/usb/generic.txt
+++ b/Documentation/devicetree/bindings/usb/generic.txt
@@ -30,6 +30,10 @@ Optional properties:
optional for OTG device.
- adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
optional for OTG device.
+ - usb-role-switch: boolean, indicates that the device is capable of assigning
+ the USB data role (USB host or USB device) for a given
+ USB connector, such as Type-C, Type-B(micro).
+ see connector/usb-connector.txt.
This is an attribute to a USB controller such as:
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
2019-05-08 9:17 ` Chunfeng Yun
(?)
@ 2019-05-13 17:28 ` Rob Herring
-1 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-05-13 17:28 UTC (permalink / raw)
To: Chunfeng Yun
Cc: Greg Kroah-Hartman, Mark Rutland, Matthias Brugger, linux-usb,
devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Biju Das, Yu Chen, Heikki Krogerus, Min Guo, Felipe Balbi,
Chunfeng Yun
On Wed, 8 May 2019 17:17:44 +0800, Chunfeng Yun wrote:
> Add a property usb-role-switch to tell the driver that use
> USB Role Switch framework to handle the role switch,
> it's useful when the driver has already supported other ways,
> such as extcon framework etc.
>
> Cc: Biju Das <biju.das@bp.renesas.com>
> Cc: Yu Chen <chenyu56@huawei.com>
> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
> ---
> v3:
> add property type, modify description suggested by Heikki
>
> v2:
> describe it in terms of h/w functionality suggested by Rob
>
> v1:
> the property is discussed in:
> [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> https://patchwork.kernel.org/patch/10852497/
>
> Mediatek and Hisilicon also try to use it:
> [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> https://patchwork.kernel.org/patch/10918385/
> [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> https://patchwork.kernel.org/patch/10918367/
>
> [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> https://patchwork.kernel.org/patch/10909981/
> ---
> Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-13 17:28 ` Rob Herring
0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-05-13 17:28 UTC (permalink / raw)
Cc: Greg Kroah-Hartman, Mark Rutland, Matthias Brugger, linux-usb,
devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Biju Das, Yu Chen, Heikki Krogerus, Min Guo, Felipe Balbi,
Chunfeng Yun
On Wed, 8 May 2019 17:17:44 +0800, Chunfeng Yun wrote:
> Add a property usb-role-switch to tell the driver that use
> USB Role Switch framework to handle the role switch,
> it's useful when the driver has already supported other ways,
> such as extcon framework etc.
>
> Cc: Biju Das <biju.das@bp.renesas.com>
> Cc: Yu Chen <chenyu56@huawei.com>
> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
> ---
> v3:
> add property type, modify description suggested by Heikki
>
> v2:
> describe it in terms of h/w functionality suggested by Rob
>
> v1:
> the property is discussed in:
> [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> https://patchwork.kernel.org/patch/10852497/
>
> Mediatek and Hisilicon also try to use it:
> [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> https://patchwork.kernel.org/patch/10918385/
> [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> https://patchwork.kernel.org/patch/10918367/
>
> [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> https://patchwork.kernel.org/patch/10909981/
> ---
> Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-13 17:28 ` Rob Herring
0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-05-13 17:28 UTC (permalink / raw)
To: Chunfeng Yun
Cc: Mark Rutland, devicetree, Heikki Krogerus, Felipe Balbi,
Greg Kroah-Hartman, linux-usb, Yu Chen, linux-kernel, Biju Das,
Chunfeng Yun, linux-mediatek, Min Guo, Matthias Brugger,
linux-arm-kernel
On Wed, 8 May 2019 17:17:44 +0800, Chunfeng Yun wrote:
> Add a property usb-role-switch to tell the driver that use
> USB Role Switch framework to handle the role switch,
> it's useful when the driver has already supported other ways,
> such as extcon framework etc.
>
> Cc: Biju Das <biju.das@bp.renesas.com>
> Cc: Yu Chen <chenyu56@huawei.com>
> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
> ---
> v3:
> add property type, modify description suggested by Heikki
>
> v2:
> describe it in terms of h/w functionality suggested by Rob
>
> v1:
> the property is discussed in:
> [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> https://patchwork.kernel.org/patch/10852497/
>
> Mediatek and Hisilicon also try to use it:
> [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> https://patchwork.kernel.org/patch/10918385/
> [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> https://patchwork.kernel.org/patch/10918367/
>
> [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> https://patchwork.kernel.org/patch/10909981/
> ---
> Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
2019-05-08 9:17 ` Chunfeng Yun
@ 2019-05-17 13:27 ` Heikki Krogerus
-1 siblings, 0 replies; 11+ messages in thread
From: Heikki Krogerus @ 2019-05-17 13:27 UTC (permalink / raw)
To: Chunfeng Yun
Cc: Rob Herring, Greg Kroah-Hartman, Mark Rutland, Matthias Brugger,
linux-usb, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek, Biju Das, Yu Chen, Min Guo, Felipe Balbi
On Wed, May 08, 2019 at 05:17:44PM +0800, Chunfeng Yun wrote:
> Add a property usb-role-switch to tell the driver that use
> USB Role Switch framework to handle the role switch,
> it's useful when the driver has already supported other ways,
> such as extcon framework etc.
>
> Cc: Biju Das <biju.das@bp.renesas.com>
> Cc: Yu Chen <chenyu56@huawei.com>
> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Who is meant to pick this? Can you include this in your series where
you introduce that USB Type-B GPIO connector driver?
FWIW:
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> ---
> v3:
> add property type, modify description suggested by Heikki
>
> v2:
> describe it in terms of h/w functionality suggested by Rob
>
> v1:
> the property is discussed in:
> [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> https://patchwork.kernel.org/patch/10852497/
>
> Mediatek and Hisilicon also try to use it:
> [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> https://patchwork.kernel.org/patch/10918385/
> [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> https://patchwork.kernel.org/patch/10918367/
>
> [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> https://patchwork.kernel.org/patch/10909981/
> ---
> Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
> index 0a74ab8dfdc2..cf5a1ad456e6 100644
> --- a/Documentation/devicetree/bindings/usb/generic.txt
> +++ b/Documentation/devicetree/bindings/usb/generic.txt
> @@ -30,6 +30,10 @@ Optional properties:
> optional for OTG device.
> - adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
> optional for OTG device.
> + - usb-role-switch: boolean, indicates that the device is capable of assigning
> + the USB data role (USB host or USB device) for a given
> + USB connector, such as Type-C, Type-B(micro).
> + see connector/usb-connector.txt.
>
> This is an attribute to a USB controller such as:
>
> --
> 2.21.0
thanks,
--
heikki
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-17 13:27 ` Heikki Krogerus
0 siblings, 0 replies; 11+ messages in thread
From: Heikki Krogerus @ 2019-05-17 13:27 UTC (permalink / raw)
To: Chunfeng Yun
Cc: Mark Rutland, devicetree, Felipe Balbi, Greg Kroah-Hartman,
linux-usb, Yu Chen, linux-kernel, Biju Das, Rob Herring,
linux-mediatek, Min Guo, Matthias Brugger, linux-arm-kernel
On Wed, May 08, 2019 at 05:17:44PM +0800, Chunfeng Yun wrote:
> Add a property usb-role-switch to tell the driver that use
> USB Role Switch framework to handle the role switch,
> it's useful when the driver has already supported other ways,
> such as extcon framework etc.
>
> Cc: Biju Das <biju.das@bp.renesas.com>
> Cc: Yu Chen <chenyu56@huawei.com>
> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Who is meant to pick this? Can you include this in your series where
you introduce that USB Type-B GPIO connector driver?
FWIW:
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> ---
> v3:
> add property type, modify description suggested by Heikki
>
> v2:
> describe it in terms of h/w functionality suggested by Rob
>
> v1:
> the property is discussed in:
> [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> https://patchwork.kernel.org/patch/10852497/
>
> Mediatek and Hisilicon also try to use it:
> [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> https://patchwork.kernel.org/patch/10918385/
> [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> https://patchwork.kernel.org/patch/10918367/
>
> [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> https://patchwork.kernel.org/patch/10909981/
> ---
> Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
> index 0a74ab8dfdc2..cf5a1ad456e6 100644
> --- a/Documentation/devicetree/bindings/usb/generic.txt
> +++ b/Documentation/devicetree/bindings/usb/generic.txt
> @@ -30,6 +30,10 @@ Optional properties:
> optional for OTG device.
> - adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
> optional for OTG device.
> + - usb-role-switch: boolean, indicates that the device is capable of assigning
> + the USB data role (USB host or USB device) for a given
> + USB connector, such as Type-C, Type-B(micro).
> + see connector/usb-connector.txt.
>
> This is an attribute to a USB controller such as:
>
> --
> 2.21.0
thanks,
--
heikki
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
2019-05-17 13:27 ` Heikki Krogerus
(?)
@ 2019-05-20 2:29 ` Chunfeng Yun
-1 siblings, 0 replies; 11+ messages in thread
From: Chunfeng Yun @ 2019-05-20 2:29 UTC (permalink / raw)
To: Heikki Krogerus
Cc: Rob Herring, Greg Kroah-Hartman, Mark Rutland, Matthias Brugger,
linux-usb, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek, Biju Das, Yu Chen, Min Guo, Felipe Balbi
On Fri, 2019-05-17 at 16:27 +0300, Heikki Krogerus wrote:
> On Wed, May 08, 2019 at 05:17:44PM +0800, Chunfeng Yun wrote:
> > Add a property usb-role-switch to tell the driver that use
> > USB Role Switch framework to handle the role switch,
> > it's useful when the driver has already supported other ways,
> > such as extcon framework etc.
> >
> > Cc: Biju Das <biju.das@bp.renesas.com>
> > Cc: Yu Chen <chenyu56@huawei.com>
> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
>
> Who is meant to pick this?
> Can you include this in your series where
> you introduce that USB Type-B GPIO connector driver?
>
Ok, I'll do it if need
> FWIW:
>
> Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
>
> > ---
> > v3:
> > add property type, modify description suggested by Heikki
> >
> > v2:
> > describe it in terms of h/w functionality suggested by Rob
> >
> > v1:
> > the property is discussed in:
> > [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> > https://patchwork.kernel.org/patch/10852497/
> >
> > Mediatek and Hisilicon also try to use it:
> > [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> > https://patchwork.kernel.org/patch/10918385/
> > [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> > https://patchwork.kernel.org/patch/10918367/
> >
> > [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> > https://patchwork.kernel.org/patch/10909981/
> > ---
> > Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
> > index 0a74ab8dfdc2..cf5a1ad456e6 100644
> > --- a/Documentation/devicetree/bindings/usb/generic.txt
> > +++ b/Documentation/devicetree/bindings/usb/generic.txt
> > @@ -30,6 +30,10 @@ Optional properties:
> > optional for OTG device.
> > - adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
> > optional for OTG device.
> > + - usb-role-switch: boolean, indicates that the device is capable of assigning
> > + the USB data role (USB host or USB device) for a given
> > + USB connector, such as Type-C, Type-B(micro).
> > + see connector/usb-connector.txt.
> >
> > This is an attribute to a USB controller such as:
> >
> > --
> > 2.21.0
>
> thanks,
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-20 2:29 ` Chunfeng Yun
0 siblings, 0 replies; 11+ messages in thread
From: Chunfeng Yun @ 2019-05-20 2:29 UTC (permalink / raw)
To: Heikki Krogerus
Cc: Mark Rutland, devicetree, Felipe Balbi, Greg Kroah-Hartman,
linux-usb, Yu Chen, linux-kernel, Biju Das, Rob Herring,
linux-mediatek, Min Guo, Matthias Brugger, linux-arm-kernel
On Fri, 2019-05-17 at 16:27 +0300, Heikki Krogerus wrote:
> On Wed, May 08, 2019 at 05:17:44PM +0800, Chunfeng Yun wrote:
> > Add a property usb-role-switch to tell the driver that use
> > USB Role Switch framework to handle the role switch,
> > it's useful when the driver has already supported other ways,
> > such as extcon framework etc.
> >
> > Cc: Biju Das <biju.das@bp.renesas.com>
> > Cc: Yu Chen <chenyu56@huawei.com>
> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
>
> Who is meant to pick this?
> Can you include this in your series where
> you introduce that USB Type-B GPIO connector driver?
>
Ok, I'll do it if need
> FWIW:
>
> Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
>
> > ---
> > v3:
> > add property type, modify description suggested by Heikki
> >
> > v2:
> > describe it in terms of h/w functionality suggested by Rob
> >
> > v1:
> > the property is discussed in:
> > [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> > https://patchwork.kernel.org/patch/10852497/
> >
> > Mediatek and Hisilicon also try to use it:
> > [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> > https://patchwork.kernel.org/patch/10918385/
> > [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> > https://patchwork.kernel.org/patch/10918367/
> >
> > [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> > https://patchwork.kernel.org/patch/10909981/
> > ---
> > Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
> > index 0a74ab8dfdc2..cf5a1ad456e6 100644
> > --- a/Documentation/devicetree/bindings/usb/generic.txt
> > +++ b/Documentation/devicetree/bindings/usb/generic.txt
> > @@ -30,6 +30,10 @@ Optional properties:
> > optional for OTG device.
> > - adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
> > optional for OTG device.
> > + - usb-role-switch: boolean, indicates that the device is capable of assigning
> > + the USB data role (USB host or USB device) for a given
> > + USB connector, such as Type-C, Type-B(micro).
> > + see connector/usb-connector.txt.
> >
> > This is an attribute to a USB controller such as:
> >
> > --
> > 2.21.0
>
> thanks,
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-20 2:29 ` Chunfeng Yun
0 siblings, 0 replies; 11+ messages in thread
From: Chunfeng Yun @ 2019-05-20 2:29 UTC (permalink / raw)
To: Heikki Krogerus
Cc: Mark Rutland, devicetree, Felipe Balbi, Greg Kroah-Hartman,
linux-usb, Yu Chen, linux-kernel, Biju Das, Rob Herring,
linux-mediatek, Min Guo, Matthias Brugger, linux-arm-kernel
On Fri, 2019-05-17 at 16:27 +0300, Heikki Krogerus wrote:
> On Wed, May 08, 2019 at 05:17:44PM +0800, Chunfeng Yun wrote:
> > Add a property usb-role-switch to tell the driver that use
> > USB Role Switch framework to handle the role switch,
> > it's useful when the driver has already supported other ways,
> > such as extcon framework etc.
> >
> > Cc: Biju Das <biju.das@bp.renesas.com>
> > Cc: Yu Chen <chenyu56@huawei.com>
> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
>
> Who is meant to pick this?
> Can you include this in your series where
> you introduce that USB Type-B GPIO connector driver?
>
Ok, I'll do it if need
> FWIW:
>
> Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
>
> > ---
> > v3:
> > add property type, modify description suggested by Heikki
> >
> > v2:
> > describe it in terms of h/w functionality suggested by Rob
> >
> > v1:
> > the property is discussed in:
> > [v2,2/7] dt-bindings: usb: renesas_usb3: add usb-role-switch property
> > https://patchwork.kernel.org/patch/10852497/
> >
> > Mediatek and Hisilicon also try to use it:
> > [v4,3/6] dt-bindings: usb: mtu3: add properties about USB Role Switch
> > https://patchwork.kernel.org/patch/10918385/
> > [v4,6/6] usb: mtu3: register a USB Role Switch for dual role mode
> > https://patchwork.kernel.org/patch/10918367/
> >
> > [v6,10/13] usb: dwc3: Registering a role switch in the DRD code
> > https://patchwork.kernel.org/patch/10909981/
> > ---
> > Documentation/devicetree/bindings/usb/generic.txt | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/generic.txt b/Documentation/devicetree/bindings/usb/generic.txt
> > index 0a74ab8dfdc2..cf5a1ad456e6 100644
> > --- a/Documentation/devicetree/bindings/usb/generic.txt
> > +++ b/Documentation/devicetree/bindings/usb/generic.txt
> > @@ -30,6 +30,10 @@ Optional properties:
> > optional for OTG device.
> > - adp-disable: tells OTG controllers we want to disable OTG ADP, ADP is
> > optional for OTG device.
> > + - usb-role-switch: boolean, indicates that the device is capable of assigning
> > + the USB data role (USB host or USB device) for a given
> > + USB connector, such as Type-C, Type-B(micro).
> > + see connector/usb-connector.txt.
> >
> > This is an attribute to a USB controller such as:
> >
> > --
> > 2.21.0
>
> thanks,
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-05-20 2:30 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-08 9:17 [v3 PATCH] dt-binding: usb: add usb-role-switch property Chunfeng Yun
2019-05-08 9:17 ` Chunfeng Yun
2019-05-08 9:17 ` Chunfeng Yun
2019-05-13 17:28 ` Rob Herring
2019-05-13 17:28 ` Rob Herring
2019-05-13 17:28 ` Rob Herring
2019-05-17 13:27 ` Heikki Krogerus
2019-05-17 13:27 ` Heikki Krogerus
2019-05-20 2:29 ` Chunfeng Yun
2019-05-20 2:29 ` Chunfeng Yun
2019-05-20 2:29 ` Chunfeng Yun
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.