[PATCH v4 4/7] arm: dts: k3-am625-sk: Enable usb port in u-boot

Sjoerd Simons sjoerd at collabora.com
Tue Jan 16 12:21:19 CET 2024


On Tue, 2024-01-16 at 12:17 +0100, Mattijs Korpershoek wrote:
> Hi Sjoerd
> 
> Thank you for the patch.
> 
> On ven., janv. 12, 2024 at 09:52, Sjoerd Simons
> <sjoerd at collabora.com> wrote:
> 
> > Enable usb0 in all boot phases for use with DFU
> > 
> > Signed-off-by: Sjoerd Simons <sjoerd at collabora.com>
> > 
> > ---
> > 
> > Changes in v4:
> > - Don't force usb0 into peripheral mode
> 
> I know that dr_mode = "peripheral" has been removed as requested by
> Roger here:
> 
> https://lore.kernel.org/all/054aa15f-05b9-4645-ab00-fac1be46dad7@kernel.org/
> 
> However, the reason for this was the following series:
> https://lore.kernel.org/all/20230706-handle-otg-as-periph-v3-0-27e24fa17345@baylibre.com/
> 
> I've spoken to Julien (off-list) about the above series and he
> confirmed
> that there are no plans to keep working this.
> 
> Therefore, I'd prefer to re-instate dr_mode = "peripheral" here to
> make
> this series simpler and focus on enabling DFU.
> 
> Would that be possible ?

If others agree sure; I really don't care either way (I wouldn't use
the port as host in any case), I only addressed it because i got a
complaint about it :)

Roger?

> 
> > 
> > Changes in v3:
> > - Enable usb nodes in all boot phases
> > 
> > Changes in v2:
> > - Only enable usb port 0 DFU in SPL
> > 
> >  arch/arm/dts/k3-am625-sk-u-boot.dtsi | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > diff --git a/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> > b/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> > index fa778b0ff4c..67c9fa2bbc3 100644
> > --- a/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> > +++ b/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> > @@ -46,3 +46,11 @@
> >  &cpsw_port2 {
> >  	status = "disabled";
> >  };
> > +
> > +&usbss0 {
> > +	bootph-all;
> > +};
> > +
> > +&usb0 {
> > +	bootph-all;
> > +};
> > -- 
> > 2.43.0

-- 
Sjoerd Simons
Collabora Ltd.


More information about the U-Boot mailing list