All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 03/22] Add core support for a bloblist to convey data from SPL
Date: Tue, 2 Oct 2018 04:21:31 -0700	[thread overview]
Message-ID: <CAPnjgZ31r75cF=eqqFcTPPCp66EiBGyAk1-tXSRSL4GQkwtm_A@mail.gmail.com> (raw)
In-Reply-To: <20180927045448.mfbugtmpqbzfv37k@jiji>

Hi Andreas,

On 26 September 2018 at 21:54, Andreas Dannenberg <dannenberg@ti.com> wrote:
> Hi Simon,
> some quick comments/notes inlined...
>
> On Wed, Sep 26, 2018 at 03:55:01PM -0600, Simon Glass wrote:
>> At present there is no standard way in U-Boot to pass information from SPL
>> to U-Boot proper. But sometimes SPL wants to convey information to U-Boot
>> that U-Boot cannot easily figure out. For example, if SPL sets up SDRAM
>> then it might want to pass the size of SDRAM, or the location of each
>> bank, to U-Boot proper.
>>
>> Add a new 'bloblist' feature which provides this. A bloblist is set up in
>> the first phase of U-Boot that runs (i.e. TPL or SPL). The location of
>> this info may be in SRAM or CAR (x86 cache-as-RAM) or somewhere else.
>>
>> Information placed in this region is preserved (with a checksum) through
>> TPL and SPL and ends up in U-Boot. At this point it is copied into SDRAM
>> so it can be used after relocation.
>>
>
> Seems generally quite useful. For example on different TI platforms we
> read out various board-specific information from an EEPROM in SPL (in
> addition to DDR related stuff) which needs to be passed to U-Boot proper
> and for that we have established the concept of a "scratch pad SRAM"
> which essentially serves the same purpose but is very rudimentary for
> example with no protection against corruption whereas the bloblists can
> use CRC... very nice.

OK good. I was a bit unsure about whether CRC was worth it. But I can
see cases where something might go wrong.

Will take care of your comments, thanks.

Regards,
Simon

  reply	other threads:[~2018-10-02 11:21 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-26 21:54 [U-Boot] [PATCH 00/22] spl: Add features for passing info from SPL to U-Boot proper Simon Glass
2018-09-26 21:54 ` [U-Boot] [PATCH 01/22] log: Correct definition of log_msg_ret() Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 02/22] spl: Add support for logging in SPL and TPL Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 03/22] Add core support for a bloblist to convey data from SPL Simon Glass
2018-09-27  4:54   ` Andreas Dannenberg
2018-10-02 11:21     ` Simon Glass [this message]
2018-09-26 21:55 ` [U-Boot] [PATCH 04/22] spl: Set up the bloblist in SPL Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 05/22] bloblist: Locate bloblist in U-Boot Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 06/22] test: Add a simple test for bloblist Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 07/22] Add bloblist documentation Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 08/22] spl: Support hash, input, pch, pci, rtc, tpm in SPL Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 09/22] spl: Add a define for SPL_TPL_PROMPT Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 10/22] spl: Make SPL_DISABLE_BANNER_PRINT a positive option Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 11/22] spl: Add a comment to spl_set_bd() Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 12/22] spl: Print a message if we are unable to load an image Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 13/22] sandbox: Add a memory map to the sandbox README Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 14/22] test/py: Add a way to pass flags to sandbox Simon Glass
2018-10-01 15:59   ` Stephen Warren
2018-09-26 21:55 ` [U-Boot] [PATCH 15/22] sandbox: Add an option to display of-platdata in SPL Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 16/22] sandbox: Add a new 'sb' command Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 17/22] sandbox: Allow puts() output before global_data is set up Simon Glass
2018-09-26 22:24   ` Joe Hershberger
2018-09-26 21:55 ` [U-Boot] [PATCH 18/22] sandbox: Refactor code to create os_jump_to_file() Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 19/22] sandbox: Use malloc() and free() from os layer Simon Glass
2018-09-26 22:22   ` Joe Hershberger
2018-09-26 21:55 ` [U-Boot] [PATCH 20/22] sandbox: Filter arguments when starting U-Boot Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 21/22] sandbox: Boot in U-Boot through the standard call Simon Glass
2018-09-26 21:55 ` [U-Boot] [PATCH 22/22] spl: Add support for passing handoff info to U-Boot proper Simon Glass
2018-10-01 13:06 ` [U-Boot] [PATCH 00/22] spl: Add features for passing info from SPL " Lukasz Majewski
2018-10-01 17:32   ` Simon Glass
2018-10-08  2:30 ` Kever Yang
2018-10-09  3:40   ` Simon Glass

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='CAPnjgZ31r75cF=eqqFcTPPCp66EiBGyAk1-tXSRSL4GQkwtm_A@mail.gmail.com' \
    --to=sjg@chromium.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.