[U-Boot] [PATCH v2 3/3] usb: gadget: f_sdp: Allow SPL to load and boot FIT via SDP
Lukasz Majewski
lukma at denx.de
Wed Jun 5 10:31:40 UTC 2019
Hi Sjoerd,
> On Wed, 2019-06-05 at 11:40 +0200, Lukasz Majewski wrote:
> > Hi Fabio, Sjoerd
> >
> > > On Tue, Jun 4, 2019 at 5:41 PM Sjoerd Simons
> > > <sjoerd.simons at collabora.co.uk> wrote:
> > >
> > > > Small steps right; Ooi what imx_usb_loader
> > > > configuration/commands are you using to test this? (I find its
> > > > config rather tricky to grasp).
> > >
> > > I simply run:
> > >
> > > sudo ./imx_usb SPL
> > >
> > > and then
> > >
> > > sudo ./imx_usb u-boot-dtb.img
> > >
> > > I suggest you to try U-Boot 2019.01 on a mx6sabreauto first.
> > >
> >
> > Tested-by: Lukasz Majewski <lukma at denx.de>
> >
> > Test HW: i.MX6Q Display5 factory setup.
> >
> >
> >
> > However, one thing puzzles me - the VID / PID used.When I run uuu
> > (mfgtools: SHA1: 13d187304f4faa473d2141409419c5b6f052addb):
> >
> > I see that "Build in config" has following VID/PID:
> > SDPU: SPL 0x0525 0xb4a4 [0x0000..0x04ff]
> > [1] SDPV: SPL1 0x0525 0xb4a4 [0x0500..0x9998]
> > SDPU: SPL 0x0525 0xb4a4 [0x9999..0x9999]
> >
> > But to make the SDPU command working I had to adjust it to be
> > similar to sabreauto (CONFIG_USB_GADGET_VENDOR_NUM=0x0525
> > CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5). Those match to "FB"
> > (fastboot?).
> >
> > Is this a bug or just the "Build in config" information is outdated?
> > With VID/PID set as for [1] (and as we use SDPU command, not FB),
> > the uuu doesn't connect to loaded SPL.
>
> So u-boot for the spl does (arch/arm/mach-imx/spl.c):
> ```
> int g_dnl_bind_fixup(struct usb_device_descriptor *dev, const char
> *name) {
> put_unaligned(CONFIG_USB_GADGET_PRODUCT_NUM + 0xfff,
> &dev->idProduct); ``
Ach... Right.
Thanks for pointing this out.
>
> Iotw with CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 in the config when in
> the SPL the product will be 0xb4a4, which uuu recognizes.
>
> Once in the main u-boot image in principle you shoudl be able to use
> `FB:`, however it doesn't seem `FB: ucmd` is supported for mainline u-
> boot so...
>
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190605/a27a41ae/attachment.sig>
More information about the U-Boot
mailing list