[PATCH v3 4/4] configs: am62x_evm_*: Enable USB and DFU support

Nishanth Menon nm at ti.com
Wed Dec 13 15:23:15 CET 2023


On 06:57-20231213, Bryan Brattlof wrote:
> On December 13, 2023 thus sayeth Sjoerd Simons:
> > Hey Brian,
> > 
> > On Tue, 2023-12-12 at 21:58 -0600, Bryan Brattlof wrote:
> > [...]
> > > > 
> > > >  configs/am62x_evm_a53_defconfig       |  31 ++++++-
> > > >  configs/am62x_evm_r5_usbdfu_defconfig | 111
> > > > ++++++++++++++++++++++++++
> > > 
> > > One of the newer developments was the addition of config fragments[0]
> > > to 
> > > try to minimize all the duplicate defconfigs for all the different 
> > > evaluation boards for the same SoC we have in the pipeline. I'm 
> > > wondering what people would think if we did this for the different 
> > > boot-mode defconfigs for the 62xx extended family like this usbdfu
> > > and 
> > > cpsw? 
> > > 
> > > idk how messy it would get trying to disable one of the primary boot 
> > > modes like MMC and enable USB though... So this may be a idea for 
> > > another day.
> > 
> > Nishanth mentioned the same and it's something i'd like to play with
> 
> Ah Ok! As long as you saw it :)
> 
> > (a beagleplay is on the way, but stuck in customs atm for some 
> > practical tests).
> > 
> 
> Nice! It's a fun little board
> 
> > However I think it would make sense to get this landed first and then
> > try refactoring with config fragments for a second step?
> > 
> 
> I agree. It's on my TODO list to try to clean up some of the messes 
> we've made so I thought I chime in
> 
> Don't let me stop you from adding features

I prefer we do the fragments with this series. We can validate and
update the fragment if required with follow on. but rather not keep
churning the usage model.

Also missing is documentation update on how to build/use dfu mode.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


More information about the U-Boot mailing list