linux-kernel - Re: [PATCH v2] arm64: allwinner: a64: orangepi-zero-plus2: add usb otg

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 for Android: free password hash cracker in your pocket

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

Date:   Mon, 15 Jan 2018 16:33:54 +0100
From:   Maxime Ripard <maxime.ripard@...e-electrons.com>
To:     Icenowy Zheng <icenowy@...c.io>
Cc:     linux-arm-kernel@...ts.infradead.org,
        Jagan Teki <jagan@...rulasolutions.com>,
        Mark Rutland <mark.rutland@....com>,
        devicetree@...r.kernel.org,
        Catalin Marinas <catalin.marinas@....com>,
        linux-sunxi@...glegroups.com, Will Deacon <will.deacon@....com>,
        linux-kernel@...r.kernel.org, Chen-Yu Tsai <wens@...e.org>,
        Rob Herring <robh+dt@...nel.org>,
        Michael Trimarchi <michael@...rulasolutions.com>
Subject: Re: [PATCH v2] arm64: allwinner: a64: orangepi-zero-plus2: add usb
 otg

On Mon, Jan 15, 2018 at 04:05:07PM +0800, Icenowy Zheng wrote:
> 在 2018年1月15日星期一 CST 下午4:01:39，Maxime Ripard 写道：
> > Hi,
> > 
> > On Fri, Jan 12, 2018 at 02:39:04PM +0530, Jagan Teki wrote:
> > > Add usb otg support for orangepi-zero-plus2 board:
> > > - Add usb_otg node with dr_mode as 'otg'
> > > - USB0-IDDET connected to PA21
> > > - VBUS connected through DCIN which always on
> > > 
> > > Tested mass storage function.
> > > 
> > > Signed-off-by: Jagan Teki <jagan@...rulasolutions.com>
> > > ---
> > > Changes for v2:
> > > - Change dr_mode to peripheral
> > > 
> > >  .../dts/allwinner/sun50i-h5-orangepi-zero-plus2.dts    | 18
> > >  ++++++++++++++++++ 1 file changed, 18 insertions(+)
> > > 
> > > diff --git
> > > a/arch/arm64/boot/dts/allwinner/sun50i-h5-orangepi-zero-plus2.dts
> > > b/arch/arm64/boot/dts/allwinner/sun50i-h5-orangepi-zero-plus2.dts index
> > > af43533..6511190 100644
> > > --- a/arch/arm64/boot/dts/allwinner/sun50i-h5-orangepi-zero-plus2.dts
> > > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h5-orangepi-zero-plus2.dts
> > > @@ -73,6 +73,10 @@
> > > 
> > >  	};
> > >  
> > >  };
> > > 
> > > +&ehci0 {
> > > +	status = "okay";
> > > +};
> > > +
> > > 
> > >  &mmc0 {
> > >  
> > >  	pinctrl-names = "default";
> > >  	pinctrl-0 = <&mmc0_pins_a>;
> > > 
> > > @@ -111,6 +115,10 @@
> > > 
> > >  	status = "okay";
> > >  
> > >  };
> > > 
> > > +&ohci0 {
> > > +	status = "okay";
> > > +};
> > > +
> > > 
> > >  &uart0 {
> > >  
> > >  	pinctrl-names = "default";
> > >  	pinctrl-0 = <&uart0_pins_a>;
> > > 
> > > @@ -122,3 +130,13 @@
> > > 
> > >  	pinctrl-0 = <&uart1_pins>, <&uart1_rts_cts_pins>;
> > >  	status = "okay";
> > >  
> > >  };
> > > 
> > > +
> > > +&usb_otg {
> > > +	dr_mode = "peripheral";
> > 
> > If VBUS is always on, this can only operate safely in host mode, not
> > peripheral.
> 
> I think this kind of device is designed to directly drain power from the host 
> in peripheral mode.

Then the commit log has it backwards, VBUS is not connected through
DCIN, DCIN is VBUS.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)