2018-01-03 08:50:21

by Icenowy Zheng

[permalink] [raw]
Subject: [PATCH RESEND] phy: sun4i-usb: add support for R40 USB PHY

Allwinner R40 features a USB PHY like the one in A64, but with 3 PHYs.

Add support for it.

Signed-off-by: Icenowy Zheng <[email protected]>
Acked-by: Maxime Ripard <[email protected]>
Acked-by: Rob Herring <[email protected]>
---
When resending, the ACK's by Maxime and Rob are added.

Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt | 1 +
drivers/phy/allwinner/phy-sun4i-usb.c | 12 ++++++++++++
2 files changed, 13 insertions(+)

diff --git a/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt b/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt
index c1ce5a0a652e..07ca4ec4a745 100644
--- a/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt
+++ b/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt
@@ -11,6 +11,7 @@ Required properties:
* allwinner,sun8i-a33-usb-phy
* allwinner,sun8i-a83t-usb-phy
* allwinner,sun8i-h3-usb-phy
+ * allwinner,sun8i-r40-usb-phy
* allwinner,sun8i-v3s-usb-phy
* allwinner,sun50i-a64-usb-phy
- reg : a list of offset + length pairs
diff --git a/drivers/phy/allwinner/phy-sun4i-usb.c b/drivers/phy/allwinner/phy-sun4i-usb.c
index aa857be692cf..bee798892b21 100644
--- a/drivers/phy/allwinner/phy-sun4i-usb.c
+++ b/drivers/phy/allwinner/phy-sun4i-usb.c
@@ -112,6 +112,7 @@ enum sun4i_usb_phy_type {
sun8i_a33_phy,
sun8i_a83t_phy,
sun8i_h3_phy,
+ sun8i_r40_phy,
sun8i_v3s_phy,
sun50i_a64_phy,
};
@@ -919,6 +920,16 @@ static const struct sun4i_usb_phy_cfg sun8i_h3_cfg = {
.phy0_dual_route = true,
};

+static const struct sun4i_usb_phy_cfg sun8i_r40_cfg = {
+ .num_phys = 3,
+ .type = sun8i_r40_phy,
+ .disc_thresh = 3,
+ .phyctl_offset = REG_PHYCTL_A33,
+ .dedicated_clocks = true,
+ .enable_pmu_unk1 = true,
+ .phy0_dual_route = true,
+};
+
static const struct sun4i_usb_phy_cfg sun8i_v3s_cfg = {
.num_phys = 1,
.type = sun8i_v3s_phy,
@@ -948,6 +959,7 @@ static const struct of_device_id sun4i_usb_phy_of_match[] = {
{ .compatible = "allwinner,sun8i-a33-usb-phy", .data = &sun8i_a33_cfg },
{ .compatible = "allwinner,sun8i-a83t-usb-phy", .data = &sun8i_a83t_cfg },
{ .compatible = "allwinner,sun8i-h3-usb-phy", .data = &sun8i_h3_cfg },
+ { .compatible = "allwinner,sun8i-r40-usb-phy", .data = &sun8i_r40_cfg },
{ .compatible = "allwinner,sun8i-v3s-usb-phy", .data = &sun8i_v3s_cfg },
{ .compatible = "allwinner,sun50i-a64-usb-phy",
.data = &sun50i_a64_cfg},
--
2.14.2


2018-01-15 15:37:28

by Hermann Lauer

[permalink] [raw]
Subject: Re: [RESEND] phy: sun4i-usb: add support for R40 USB PHY

On Wed, Jan 03, 2018 at 04:49:44PM +0800, Icenowy Zheng wrote:
> Allwinner R40 features a USB PHY like the one in A64, but with 3 PHYs.
>
> Add support for it.
>
> Signed-off-by: Icenowy Zheng <[email protected]>
> Acked-by: Maxime Ripard <[email protected]>
> Acked-by: Rob Herring <[email protected]>

You may add

Tested-by: [email protected]

as my Banana Pi M2 Ultra is running over a week now with an usb-network
adapter which is needed until devicetree entries for the Banana Pi M2 Ultra
are available for the ccu register enabling allowing access to the ethernet phy.

Thanks !

> ---
> When resending, the ACK's by Maxime and Rob are added.
>
> Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt | 1 +
> drivers/phy/allwinner/phy-sun4i-usb.c | 12 ++++++++++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt b/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt
> index c1ce5a0a652e..07ca4ec4a745 100644
> --- a/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt
> +++ b/Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt
> @@ -11,6 +11,7 @@ Required properties:
> * allwinner,sun8i-a33-usb-phy
> * allwinner,sun8i-a83t-usb-phy
> * allwinner,sun8i-h3-usb-phy
> + * allwinner,sun8i-r40-usb-phy
> * allwinner,sun8i-v3s-usb-phy
> * allwinner,sun50i-a64-usb-phy
> - reg : a list of offset + length pairs
> diff --git a/drivers/phy/allwinner/phy-sun4i-usb.c b/drivers/phy/allwinner/phy-sun4i-usb.c
> index aa857be692cf..bee798892b21 100644
> --- a/drivers/phy/allwinner/phy-sun4i-usb.c
> +++ b/drivers/phy/allwinner/phy-sun4i-usb.c
> @@ -112,6 +112,7 @@ enum sun4i_usb_phy_type {
> sun8i_a33_phy,
> sun8i_a83t_phy,
> sun8i_h3_phy,
> + sun8i_r40_phy,
> sun8i_v3s_phy,
> sun50i_a64_phy,
> };
> @@ -919,6 +920,16 @@ static const struct sun4i_usb_phy_cfg sun8i_h3_cfg = {
> .phy0_dual_route = true,
> };
>
> +static const struct sun4i_usb_phy_cfg sun8i_r40_cfg = {
> + .num_phys = 3,
> + .type = sun8i_r40_phy,
> + .disc_thresh = 3,
> + .phyctl_offset = REG_PHYCTL_A33,
> + .dedicated_clocks = true,
> + .enable_pmu_unk1 = true,
> + .phy0_dual_route = true,
> +};
> +
> static const struct sun4i_usb_phy_cfg sun8i_v3s_cfg = {
> .num_phys = 1,
> .type = sun8i_v3s_phy,
> @@ -948,6 +959,7 @@ static const struct of_device_id sun4i_usb_phy_of_match[] = {
> { .compatible = "allwinner,sun8i-a33-usb-phy", .data = &sun8i_a33_cfg },
> { .compatible = "allwinner,sun8i-a83t-usb-phy", .data = &sun8i_a83t_cfg },
> { .compatible = "allwinner,sun8i-h3-usb-phy", .data = &sun8i_h3_cfg },
> + { .compatible = "allwinner,sun8i-r40-usb-phy", .data = &sun8i_r40_cfg },
> { .compatible = "allwinner,sun8i-v3s-usb-phy", .data = &sun8i_v3s_cfg },
> { .compatible = "allwinner,sun50i-a64-usb-phy",
> .data = &sun50i_a64_cfg},

--
Netzwerkadministration/Zentrale Dienste, Interdiziplinaeres
Zentrum fuer wissenschaftliches Rechnen der Universitaet Heidelberg
IWR; INF 205; 69120 Heidelberg; Tel: (06221)54-14405 Fax: -14427
Email: [email protected]

2018-01-19 06:27:27

by Chen-Yu Tsai

[permalink] [raw]
Subject: Re: [RESEND] phy: sun4i-usb: add support for R40 USB PHY

Hi Kishon,

On Mon, Jan 15, 2018 at 11:06 PM, Hermann Lauer
<[email protected]> wrote:
> On Wed, Jan 03, 2018 at 04:49:44PM +0800, Icenowy Zheng wrote:
>> Allwinner R40 features a USB PHY like the one in A64, but with 3 PHYs.
>>
>> Add support for it.
>>
>> Signed-off-by: Icenowy Zheng <[email protected]>
>> Acked-by: Maxime Ripard <[email protected]>
>> Acked-by: Rob Herring <[email protected]>
>
> You may add
>
> Tested-by: [email protected]

Gentle ping for this patch to be included in 4.16

ChenYu

2018-01-19 06:31:35

by Icenowy Zheng

[permalink] [raw]
Subject: Re: [RESEND] phy: sun4i-usb: add support for R40 USB PHY



于 2018年1月19日 GMT+08:00 下午2:25:09, Chen-Yu Tsai <[email protected]> 写到:
>Hi Kishon,
>
>On Mon, Jan 15, 2018 at 11:06 PM, Hermann Lauer
><[email protected]> wrote:
>> On Wed, Jan 03, 2018 at 04:49:44PM +0800, Icenowy Zheng wrote:
>>> Allwinner R40 features a USB PHY like the one in A64, but with 3
>PHYs.
>>>
>>> Add support for it.
>>>
>>> Signed-off-by: Icenowy Zheng <[email protected]>
>>> Acked-by: Maxime Ripard <[email protected]>
>>> Acked-by: Rob Herring <[email protected]>
>>
>> You may add
>>
>> Tested-by: [email protected]
>
>Gentle ping for this patch to be included in 4.16

I think maybe I forgot PATCH in title so it didn't enter patchwork?

>
>ChenYu
>
>_______________________________________________
>linux-arm-kernel mailing list
>[email protected]
>http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

2018-01-19 08:16:48

by Kishon Vijay Abraham I

[permalink] [raw]
Subject: Re: [RESEND] phy: sun4i-usb: add support for R40 USB PHY

Hi,

On Friday 19 January 2018 11:55 AM, Chen-Yu Tsai wrote:
> Hi Kishon,
>
> On Mon, Jan 15, 2018 at 11:06 PM, Hermann Lauer
> <[email protected]> wrote:
>> On Wed, Jan 03, 2018 at 04:49:44PM +0800, Icenowy Zheng wrote:
>>> Allwinner R40 features a USB PHY like the one in A64, but with 3 PHYs.
>>>
>>> Add support for it.
>>>
>>> Signed-off-by: Icenowy Zheng <[email protected]>
>>> Acked-by: Maxime Ripard <[email protected]>
>>> Acked-by: Rob Herring <[email protected]>
>>
>> You may add
>>
>> Tested-by: [email protected]
>
> Gentle ping for this patch to be included in 4.16

Sent the pull request for 4.16 to Greg on Jan 2nd. So this can't be included in
4.16. Sorry.

Thanks
Kishon

2018-02-16 19:03:07

by Kishon Vijay Abraham I

[permalink] [raw]
Subject: Re: [RESEND] phy: sun4i-usb: add support for R40 USB PHY



On Friday 19 January 2018 01:43 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Friday 19 January 2018 11:55 AM, Chen-Yu Tsai wrote:
>> Hi Kishon,
>>
>> On Mon, Jan 15, 2018 at 11:06 PM, Hermann Lauer
>> <[email protected]> wrote:
>>> On Wed, Jan 03, 2018 at 04:49:44PM +0800, Icenowy Zheng wrote:
>>>> Allwinner R40 features a USB PHY like the one in A64, but with 3 PHYs.
>>>>
>>>> Add support for it.
>>>>
>>>> Signed-off-by: Icenowy Zheng <[email protected]>
>>>> Acked-by: Maxime Ripard <[email protected]>
>>>> Acked-by: Rob Herring <[email protected]>
>>>
>>> You may add
>>>
>>> Tested-by: [email protected]
>>
>> Gentle ping for this patch to be included in 4.16
>
> Sent the pull request for 4.16 to Greg on Jan 2nd. So this can't be included in
> 4.16. Sorry.

merged now, thanks!

-Kishon