public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Agner <stefan@agner.ch>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/8] imx: add USB Serial Download Protocol (SDP) support
Date: Fri, 08 Sep 2017 09:37:13 -0700	[thread overview]
Message-ID: <d38fe2ea15900f26ea1845bff308ddcb@agner.ch> (raw)
In-Reply-To: <CAB2nKc_bHnX6OgdYthn2ra8Mti8TQxbBp+uC7vp_-MBBzNVbgA@mail.gmail.com>

Hi Vincent,

On 2017-09-08 01:27, Vincent Prince wrote:
> Hi everyone,
> 
> I managed to get SDP protocol to work with on my custom i.MX6 board,
> so big thanks to Stefan for your work!
> 
> I'm now trying to load swupdate ramdisk with u-boot sdp command,
> without success.
> Is it a possible use case ?

Yes it should work, we use the U-Boot sdp command to download a squashfs
based ramdisk in our Toradex Easy Installer:
http://developer.toradex.com/software/toradex-easy-installer

> 
> Here my steps:
> 
> In u-boot:
> => sdp 0
> 
> On Host,
> $ imx_usb
> 
> config file <C:\Users\vpr\IMXUSBLOADER2\delivery\\imx_usb.conf>
> vid=0x15a2 pid=0x0061 file_name=mx6_usb_rom.conf -> vid=0x0525
> pid=0xb4a4 file_name=mx6_usb_sdp_spl.conf
> vid=0x0525 pid=0xa4a5 file_name=mx6_usb_uboot.conf
> config file <C:\Users\vpr\IMXUSBLOADER2\delivery\\mx6_usb_uboot.conf>
> parse C:\Users\vpr\IMXUSBLOADER2\delivery\\mx6_usb_uboot.conf
> Trying to open device vid=0x0525 pid=0xa4a5.........................
> Could not open device vid=0x0525 pid=0xa4a5

Is VID/PID showing up in Device Manager?

We did saw some problems with certain USB host implementation, so it
might be a device dependency... I would also try a different
machine/maybe Linux machine...

> 
> VID/PID in imx_usb.conf are correctly detected, and correct
> configuration file is launched (mx6_usb_uboot.conf),
> it contains:
> 
> mx6_usb_sdp_uboot
> hid,1024,0x10000000,1G,0x00907000,0x31000
> test.txt:load 0x12100000
> 
> My imx_usb.conf is the following:
> 
> #vid:pid, config_file
> 0x15a2:0x0061, mx6_usb_rom.conf, 0x0525:0xb4a4, mx6_usb_sdp_spl.conf
> 0x0525:0xa4a5, mx6_usb_uboot.conf

Yeah assuming you use the Toradex branch of imx_loader currently this
should work.

Our script looks like this:
mx6_usb_sdp_uboot
#hid/bulk,[old_header,]max packet size, {ram start, ram size}(repeat
valid ram areas)
hid,1024,0x10000000,1G,0x00907000,0x31000
#Load complete FIT image to $ramdisk_addr_r
tezi.itb:load 0x12100000
#Load script to $loadaddr and jump to it
boot-sdp.scr:load 0x12000000,jump 0x12000000

--
Stefan


> 
> Am I missing something,
> Thanks,
> Vincent
> 
> 2017-09-01 22:09 GMT+02:00 Stefan Agner <stefan@agner.ch>:
>>
>>
>> On September 1, 2017 12:25:44 PM PDT, Fabio Estevam <festevam@gmail.com> wrote:
>>>On Fri, Sep 1, 2017 at 3:54 PM, Fabio Estevam <festevam@gmail.com>
>>>wrote:
>>>
>>>> I have tested this method and it works, thanks.
>>>>
>>>> Do you plan to usptream this method?
>>>
>>>Or I can also put your patch as part of my series that adds SDP
>>>support for imx6qsabresd if you prefer.
>>
>> Let me do a separate, I guess it could be controversial...
>>
>> --
>> Stefan
>> --
>> Sent from my Android device with K-9 Mail. Please excuse my brevity.
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> https://lists.denx.de/listinfo/u-boot

  reply	other threads:[~2017-09-08 16:37 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-16 18:00 [U-Boot] [PATCH v2 0/8] imx: add USB Serial Download Protocol (SDP) support Stefan Agner
2017-08-16 18:00 ` [U-Boot] [PATCH v2 1/8] imx: move imximage header to common location Stefan Agner
2017-08-16 18:00 ` [U-Boot] [PATCH v2 2/8] usb: gadget: add SDP driver Stefan Agner
2017-08-17 10:04   ` Łukasz Majewski
2017-08-17 13:49   ` Stefano Babic
2017-08-17 13:54     ` Marek Vasut
2017-08-17 13:59       ` Stefano Babic
2017-08-17 14:01       ` Łukasz Majewski
2017-08-16 18:00 ` [U-Boot] [PATCH v2 3/8] usb: gadget: sdp: extend images compatible for jumps Stefan Agner
2017-08-16 18:00 ` [U-Boot] [PATCH v2 4/8] cmd: add sdp command Stefan Agner
2017-08-16 18:00 ` [U-Boot] [PATCH v2 5/8] spl: add serial download protocol (SDP) support Stefan Agner
2017-08-17 10:10   ` Łukasz Majewski
2017-08-16 18:00 ` [U-Boot] [PATCH v2 6/8] doc: add Serial Download Protocol documentation Stefan Agner
2017-08-17 10:21   ` Łukasz Majewski
2017-08-16 18:00 ` [U-Boot] [PATCH v2 7/8] apalis/colibri_imx6: use independent USB PID for SPL Stefan Agner
2017-08-17 10:23   ` Łukasz Majewski
2017-08-16 18:00 ` [U-Boot] [PATCH v2 8/8] apalis/colibri_imx6: enable SDP by default Stefan Agner
2017-08-23  8:43 ` [U-Boot] [PATCH v2 0/8] imx: add USB Serial Download Protocol (SDP) support Stefano Babic
2017-08-23 12:28   ` Fabio Estevam
2017-09-01 16:48 ` Fabio Estevam
2017-09-01 18:19   ` Stefan Agner
2017-09-01 18:54     ` Fabio Estevam
2017-09-01 19:25       ` Fabio Estevam
2017-09-01 20:09         ` Stefan Agner
2017-09-08  8:27           ` Vincent Prince
2017-09-08 16:37             ` Stefan Agner [this message]
2017-09-11 14:15               ` Vincent Prince
2017-09-11 16:19                 ` Stefan Agner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d38fe2ea15900f26ea1845bff308ddcb@agner.ch \
    --to=stefan@agner.ch \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox