* [v3 PATCH] dt-binding: usb: add usb-role-switch property
@ 2019-05-08 9:17 Chunfeng Yun
2019-05-13 17:28 ` Rob Herring
2019-05-17 13:27 ` Heikki Krogerus
0 siblings, 2 replies; 4+ 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] 4+ messages in thread* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
2019-05-08 9:17 [v3 PATCH] dt-binding: usb: add usb-role-switch property Chunfeng Yun
@ 2019-05-13 17:28 ` Rob Herring
2019-05-17 13:27 ` Heikki Krogerus
1 sibling, 0 replies; 4+ 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] 4+ messages in thread
* Re: [v3 PATCH] dt-binding: usb: add usb-role-switch property
2019-05-08 9:17 [v3 PATCH] dt-binding: usb: add usb-role-switch property Chunfeng Yun
2019-05-13 17:28 ` Rob Herring
@ 2019-05-17 13:27 ` Heikki Krogerus
2019-05-20 2:29 ` Chunfeng Yun
1 sibling, 1 reply; 4+ 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] 4+ 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
0 siblings, 0 replies; 4+ 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] 4+ messages in thread
end of thread, other threads:[~2019-05-20 2:30 UTC | newest]
Thread overview: 4+ 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-13 17:28 ` Rob Herring
2019-05-17 13:27 ` Heikki Krogerus
2019-05-20 2:29 ` Chunfeng Yun
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).