U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH] spl: Add option SPL_PAYLOAD
Date: Mon, 9 Jul 2018 11:55:07 -0400	[thread overview]
Message-ID: <20180709155507.GP27264@bill-the-cat> (raw)
In-Reply-To: <VI1PR04MB207834FB03A095ECB85459CF9A440@VI1PR04MB2078.eurprd04.prod.outlook.com>

On Mon, Jul 09, 2018 at 03:26:14PM +0000, York Sun wrote:
> On 06/14/2018 02:39 PM, York Sun wrote:
> > Some legacy boards use RAW image for SPL boot. Add Kconfig option
> > SPL_PAYLOAD to set alternative image.
> > 
> > Signed-off-by: York Sun <york.sun@nxp.com>
> > 
> > ---
> > 
> >  Makefile           |  4 ++--
> >  common/spl/Kconfig | 10 ++++++++++
> >  2 files changed, 12 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Makefile b/Makefile
> > index 6a190e7..36459f1 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -1115,8 +1115,8 @@ u-boot.sha1:	u-boot.bin
> >  u-boot.dis:	u-boot
> >  		$(OBJDUMP) -d $< > $@
> >  
> > -ifdef CONFIG_TPL
> > -SPL_PAYLOAD := tpl/u-boot-with-tpl.bin
> > +ifneq ($(CONFIG_SPL_PAYLOAD),)
> > +SPL_PAYLOAD := $(CONFIG_SPL_PAYLOAD:"%"=%)
> >  else
> >  SPL_PAYLOAD := u-boot.bin
> >  endif
> > diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> > index 1f14797..72b77d7 100644
> > --- a/common/spl/Kconfig
> > +++ b/common/spl/Kconfig
> > @@ -552,6 +552,16 @@ config SYS_OS_BASE
> >  
> >  endif # SPL_OS_BOOT
> >  
> > +config SPL_PAYLOAD
> > +	string "SPL payload"
> > +	default "tpl/u-boot-with-tpl.bin" if TPL
> > +	default "u-boot.bin"
> > +	help
> > +	  Payload for SPL boot. For backward compability, default to
> > +	  u-boot.bin, i.e. RAW image without any header. In case of
> > +	  TPL, tpl/u-boot-with-tpl.bin. For new boards, suggest to
> > +	  use u-boot.img.
> > +
> >  config SPL_PCI_SUPPORT
> >  	bool "Support PCI drivers"
> >  	help
> > 
> 
> Any comment?

Reviewed-by: Tom Rini <trini@konsulko.com>

And feel free to take it via your tree with other patches if needed,
thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180709/8316f035/attachment.sig>

  reply	other threads:[~2018-07-09 15:55 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-14 21:38 [U-Boot] [RFC PATCH] spl: Add option SPL_PAYLOAD York Sun
2018-07-09 15:26 ` York Sun
2018-07-09 15:55   ` Tom Rini [this message]
2018-07-27 15:39 ` York Sun

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=20180709155507.GP27264@bill-the-cat \
    --to=trini@konsulko.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