On Wed, Dec 18, 2024 at 01:59:37PM +0100, Mattijs Korpershoek wrote: Hello Mattijs,
> Hi Siddharth, > > Thank you for the series. > > On mer., déc. 18, 2024 at 18:15, Siddharth Vadapalli <s-vadapa...@ti.com> > wrote: > > > Hello, > > > > This series add support for USB DFU boot on TI's AM62Px SoC which has > > two instances of USB Controllers, of which the USB0 instance of USB > > which is a DWC3 USB Controller is used for USB DFU Boot: > > ROM => tiboot3.bin => tispl.bin => u-boot.img > > > > > v1 of this series is at: > > https://patchwork.ozlabs.org/project/uboot/cover/20241217131658.2920799-1-s-vadapa...@ti.com/ > > Changes since v1: > > - Dropped the first patch of the v1 series since its equivalent has been > > posted by Jonathan Humphreys <j-humphr...@ti.com> at: > > > > https://patchwork.ozlabs.org/project/uboot/patch/20241217204835.3312765-1-j-humphr...@ti.com/ > > - Based on the feedback from Mattijs Korpershoek on the second patch of > > the v1 series, the patches in this series have been updated. More > > details regarding the changes have been described below the tearline > > of the respective patches. > > - Collected Reviewed-by tags. > > I have tested this on the AM62Px SK EVM. I did *not* apply the > dependency (20241217204835.3312765-1-j-humphr...@ti.com) and it worked > fine for me: > > I did: > $ snagrecover -s am625 -f ./am62px-sk-evm.yaml > > Boot logs: > https://paste.debian.net/1340393/ > > Also tested that I could enumerate the board with "fastboot". > > Tested-by: Mattijs Korpershoek <mkorpersh...@baylibre.com> Thank you for testing this series. The dependency that has been mentioned is for USB DFU *Boot*, while you seem to have tested USB DFU *Flash*. Regards, Siddharth.
