public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 1/5] spl: dfu: add dfu support in SPL
Date: Fri, 03 Jun 2016 11:27:07 +0200	[thread overview]
Message-ID: <20160603112707.4dc66535@amdc2363> (raw)
In-Reply-To: <6C6B28D4DC342643927BEAFCE8707BF67621A483@DBDE04.ent.ti.com>

Hi Ravi,

> Hi Lukasz
>  
> >> >One question:
> >> 
> >> >Would it be possible to develop SPL (MLO) for your platform,
> >> >which does following things:
> >> 
> >> >1. Loads the full-fledge u-boot to SDRAM 2. Starts the u-boot and
> >> >3. u-boot flash all the needed stuff
> >> 
> >> >By using such approach we could restrict our dfu support in SPL 
> >> >u-boot only to receiving data and uploading it to SDRAM (i.e. we 
> >> >wouldn't need to add write support for ext, fat and eMMC).
> >> 
> >> 
> >> This is not possible, because combining MLO and u-boot.img will
> >> not fit in IRAM (512k). The u-boot.img itself is around 640K.
> 
> >My point here is to first load SPL u-boot (which size is around 110
> >KiB), and then download via DFU full-featured u-boot, which would be
> >placed in SDRAM.
> 
> Yes this is good idea, so the SPL-DFU will have only RAM device
> support (to load u-boot into DDR). But we don't have DFU command to
> jump to u-boot after loading u-boot into DDR.

I think that we don't need dfu command in SPL to boot fully-featured
u-boot.

Lets consider following scenario:

- ROM bootloader (IPL) fetches SPL u-boot via USB. Then it passes
  execution to it.
- The running SPL downloads fully-fledged u-boot [*] to RAM and passes
  execution to it (as it is done in board_init_r @ ./common/spl/spl.c).
  There is no need for any special command(s) to do that.
- Fully-blown u-boot flashes all needed images and reboots the board.
- Now the board is setup with proper u-boot, kernel and rootfs

[*] - This version of u-boot only requires to have hardcoded "bootcmd"
env variable. It would consist the list of commands needed for flashing
your target board. The "dfu_alt_info" should be hardcoded as well.


> The DFU basically just
> download firmware to memory devices like mmc/sd/eMMC/RAM.
> So the
> question is how to transfer control to u-boot after downloading the
> u-boot to DDR using SPL-DFU/RAM.

Please consult board_init_r function at ./common/spl/spl.c
 
> 
> >Then u-boot downloads and flash all needed images.
> >One big advantage is that our SPL DFU support can be minimal - we
> >don't need to implement fat, ext ,emmc write support.
> 
> Yes, I agree, if we could workaround above bottle neck.
> 
> Regards
> Ravi
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

  reply	other threads:[~2016-06-03  9:27 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-27 13:39 [U-Boot] [RFC PATCH 0/5] SPL: DFU Support in SPL Ravi Babu
2016-05-27 13:39 ` [U-Boot] [RFC PATCH 1/5] spl: dfu: add dfu support " Ravi Babu
2016-05-30 11:54   ` Lukasz Majewski
2016-05-30 13:43     ` B, Ravi
2016-05-30 14:59       ` Lukasz Majewski
2016-05-31  6:39         ` B, Ravi
2016-05-31  8:39           ` Lukasz Majewski
2016-05-31  9:04             ` B, Ravi
2016-05-31  9:55               ` Lukasz Majewski
2016-05-31 10:34                 ` B, Ravi
2016-05-31 12:47                   ` Lukasz Majewski
2016-05-31 13:31                     ` B, Ravi
2016-05-31 15:13                       ` Lukasz Majewski
2016-06-02 12:39                         ` B, Ravi
2016-06-02 14:14                           ` Lukasz Majewski
2016-06-02 14:22                             ` B, Ravi
2016-06-03  9:27                               ` Lukasz Majewski [this message]
2016-06-03 11:35                                 ` B, Ravi
2016-06-03 11:45                                   ` Lukasz Majewski
2016-05-27 13:39 ` [U-Boot] [RFC PATCH 2/5] spl: dfu: fs: adding ext4/fat filesystem support for SPL-DFU Ravi Babu
2016-05-30 11:59   ` Lukasz Majewski
2016-05-30 12:52     ` B, Ravi
2016-05-30 13:20       ` Lukasz Majewski
2016-05-30 13:52         ` B, Ravi
2016-05-27 13:39 ` [U-Boot] [RFC PATCH 3/5] spl: dfu: adding dfu support functions " Ravi Babu
2016-05-30  5:27   ` Heiko Schocher
2016-05-30 12:04   ` Lukasz Majewski
2016-05-30 12:49     ` B, Ravi
2016-05-27 13:39 ` [U-Boot] [RFC PATCH 4/5] dfu: spl: add generic spl-dfu function in common-spl Ravi Babu
2016-05-30 12:07   ` Lukasz Majewski
2016-05-27 13:39 ` [U-Boot] [RFC PATCH 5/5] dra7x: spl: dfu: adding SPL-DFU support for dra7x platform Ravi Babu
2016-05-30 12:19   ` Lukasz Majewski
2016-05-27 13:42 ` [U-Boot] [RFC PATCH 0/5] SPL: DFU Support in SPL Marek Vasut

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=20160603112707.4dc66535@amdc2363 \
    --to=l.majewski@samsung.com \
    --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