On 06/01/2021 15:56, Chen-Yu Tsai wrote:
> Hi,
>
> On Wed, Jan 6, 2021 at 11:05 PM Andre Przywara <andre.przywara@xxxxxxx> wrote:
>>
>> In recent Allwinner SoCs the first USB host controller (HCI0) shares
>> the first PHY with the MUSB controller. Probably to make this sharing
>> work, we were avoiding to declare this in the DT. This has two
>> shortcomings:
>> - U-Boot (which uses the same .dts) cannot use this port in host mode
>> without a PHY linked, so we were loosing one USB port there.
>> - It requires the MUSB driver to be enabled and loaded, although we
>> don't actually use it.
>>
>> To avoid those issues, let's add this PHY link to the A64 and H6 .dtsi
>> files. After all PHY port 0 *is* connected to HCI0, so we should describe
>> it as this. Remove the part from the boards which were already doing
>> that.
>>
>> This makes it work in U-Boot, also improves compatiblity when no MUSB
>> driver is loaded (for instance in distribution installers).
>>
>> Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx>
>> ---
>> arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts | 4 ----
>> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 4 ++++
>> arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 4 ++++
>
> It would be better if the H6 parts were separated out.
Sure.
>
> Please also add a fixes tag, which helps with backporting.
The whole concept of DT backports raises some eyebrows here, but it's
surely easy enough to do.
Thanks,
Andre.
>
>> 3 files changed, 8 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts
>> index 896f34fd9fc3..d07cf05549c3 100644
>> --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts
>> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts
>> @@ -126,8 +126,6 @@
>> };
>>
>> &ehci0 {
>> - phys = <&usbphy 0>;
>> - phy-names = "usb";
>> status = "okay";
>> };
>>
>> @@ -177,8 +175,6 @@
>> };
>>
>> &ohci0 {
>> - phys = <&usbphy 0>;
>> - phy-names = "usb";
>> status = "okay";
>> };
>>
>> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
>> index 51cc30e84e26..19e9b8ca8432 100644
>> --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
>> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
>> @@ -593,6 +593,8 @@
>> <&ccu CLK_USB_OHCI0>;
>> resets = <&ccu RST_BUS_OHCI0>,
>> <&ccu RST_BUS_EHCI0>;
>> + phys = <&usbphy 0>;
>> + phy-names = "usb";
>> status = "disabled";
>> };
>>
>> @@ -603,6 +605,8 @@
>> clocks = <&ccu CLK_BUS_OHCI0>,
>> <&ccu CLK_USB_OHCI0>;
>> resets = <&ccu RST_BUS_OHCI0>;
>> + phys = <&usbphy 0>;
>> + phy-names = "usb";
>> status = "disabled";
>> };
>>
>> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
>> index 8a62a9fbe347..f593cfeaecc9 100644
>> --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
>> +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
>> @@ -680,6 +680,8 @@
>> <&ccu CLK_USB_OHCI0>;
>> resets = <&ccu RST_BUS_OHCI0>,
>> <&ccu RST_BUS_EHCI0>;
>> + phys = <&usb2phy 0>;
>> + phy-names = "usb";
>> status = "disabled";
>> };
>>
>> @@ -690,6 +692,8 @@
>> clocks = <&ccu CLK_BUS_OHCI0>,
>> <&ccu CLK_USB_OHCI0>;
>> resets = <&ccu RST_BUS_OHCI0>;
>> + phys = <&usb2phy 0>;
>> + phy-names = "usb";
>> status = "disabled";
>> };
>>
>> --
>> 2.17.5
>>
>> --
>> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@xxxxxxxxxxxxxxxx.
>> To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/20210106150525.15403-2-andre.przywara%40arm.com.
