linux-kernel - Re: [PATCH v2] arm64: dts: rockchip: enable otg/drd operation of usb_host0

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <aecc866c-685b-8def-a197-d1246070aa54@wolfvision.net>
Date:   Mon, 9 May 2022 11:49:08 +0200
From:   Michael Riesch <michael.riesch@...fvision.net>
To:     Peter Geis <pgwipeout@...il.com>
Cc:     devicetree <devicetree@...r.kernel.org>,
        arm-mail-list <linux-arm-kernel@...ts.infradead.org>,
        "open list:ARM/Rockchip SoC..." <linux-rockchip@...ts.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Heiko Stuebner <heiko@...ech.de>,
        Liang Chen <cl@...k-chips.com>,
        Nicolas Frattaroli <frattaroli.nicolas@...il.com>,
        Frank Wunderlich <frank-w@...lic-files.de>
Subject: Re: [PATCH v2] arm64: dts: rockchip: enable otg/drd operation of
 usb_host0_xhci in rk3568

Hello Peter,

On 5/2/22 11:54, Peter Geis wrote:
> On Mon, May 2, 2022 at 2:46 AM Michael Riesch
> <michael.riesch@...fvision.net> wrote:
>>
>> This USB 3.0 controller of the Rockchip RK3568 is capable of OTG/DRD
>> operation. Enable it in the device tree.
> 
> I think I missed a conversation here, the xhci0 controller is the otg
> port for both rk3566 and rk3568. The only difference is on rk3566 it
> is a usb2 only controller, where on rk3568 it is also a usb3
> controller. It was set to host only because my original phy patches
> didn't support otg mode. Since then Samuel's series fixed that.
> 
> Your original patch was correct.

I have already replied to v1, but at this point I would like to
 - thank you for pointing this out
 - and send a friendly reminder to Heiko: please apply v1 of this
   patch :-)

Thanks and best regards,
Michael

> 
> Always,
> Peter
> 
>>
>> Signed-off-by: Michael Riesch <michael.riesch@...fvision.net>
>> ---
>> v2:
>>  - As pointed out by Nicolas, this is RK3568 specific. Moved change to
>>    rk3568.dtsi.
>>
>>  arch/arm64/boot/dts/rockchip/rk3568.dtsi | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/boot/dts/rockchip/rk3568.dtsi b/arch/arm64/boot/dts/rockchip/rk3568.dtsi
>> index 5eafddf62edc..bbfe8f3d68b7 100644
>> --- a/arch/arm64/boot/dts/rockchip/rk3568.dtsi
>> +++ b/arch/arm64/boot/dts/rockchip/rk3568.dtsi
>> @@ -134,6 +134,7 @@ power-domain@...568_PD_PIPE {
>>  };
>>
>>  &usb_host0_xhci {
>> +       dr_mode = "otg";
>>         phys = <&usb2phy0_otg>, <&combphy0 PHY_TYPE_USB3>;
>>         phy-names = "usb2-phy", "usb3-phy";
>>  };
>> --
>> 2.30.2