All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] cmd:spl:fix: Prevent from a build error on boards, which don't support FDT
@ 2012-12-06 15:23 Lukasz Majewski
  2012-12-06 15:33 ` Stefano Babic
  0 siblings, 1 reply; 3+ messages in thread
From: Lukasz Majewski @ 2012-12-06 15:23 UTC (permalink / raw)
  To: u-boot

Do not compile in FDT related code, when it is not supported.

Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
 common/cmd_spl.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/common/cmd_spl.c b/common/cmd_spl.c
index 9ec054a..e3c543b 100644
--- a/common/cmd_spl.c
+++ b/common/cmd_spl.c
@@ -130,10 +130,12 @@ static int spl_export(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 		if (call_bootm(argc, argv, subcmd_list[(int)c->cmd]))
 			return -1;
 		switch ((int)c->cmd) {
+#ifdef CONFIG_OF_LIBFDT
 		case SPL_EXPORT_FDT:
 			printf("Argument image is now in RAM: 0x%p\n",
 				(void *)images.ft_addr);
 			break;
+#endif
 		case SPL_EXPORT_ATAGS:
 			printf("Argument image is now in RAM at: 0x%p\n",
 				(void *)gd->bd->bi_boot_params);
-- 
1.7.2.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] cmd:spl:fix: Prevent from a build error on boards, which don't support FDT
  2012-12-06 15:23 [U-Boot] [PATCH] cmd:spl:fix: Prevent from a build error on boards, which don't support FDT Lukasz Majewski
@ 2012-12-06 15:33 ` Stefano Babic
  2012-12-12 13:15   ` Tom Rini
  0 siblings, 1 reply; 3+ messages in thread
From: Stefano Babic @ 2012-12-06 15:33 UTC (permalink / raw)
  To: u-boot

On 06/12/2012 16:23, Lukasz Majewski wrote:
> Do not compile in FDT related code, when it is not supported.
> 
> Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> ---
>  common/cmd_spl.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/common/cmd_spl.c b/common/cmd_spl.c
> index 9ec054a..e3c543b 100644
> --- a/common/cmd_spl.c
> +++ b/common/cmd_spl.c
> @@ -130,10 +130,12 @@ static int spl_export(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>  		if (call_bootm(argc, argv, subcmd_list[(int)c->cmd]))
>  			return -1;
>  		switch ((int)c->cmd) {
> +#ifdef CONFIG_OF_LIBFDT
>  		case SPL_EXPORT_FDT:
>  			printf("Argument image is now in RAM: 0x%p\n",
>  				(void *)images.ft_addr);
>  			break;
> +#endif
>  		case SPL_EXPORT_ATAGS:
>  			printf("Argument image is now in RAM at: 0x%p\n",
>  				(void *)gd->bd->bi_boot_params);
> 

Acked-by: Stefano Babic <sbabic@denx.de>

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] cmd:spl:fix: Prevent from a build error on boards, which don't support FDT
  2012-12-06 15:33 ` Stefano Babic
@ 2012-12-12 13:15   ` Tom Rini
  0 siblings, 0 replies; 3+ messages in thread
From: Tom Rini @ 2012-12-12 13:15 UTC (permalink / raw)
  To: u-boot

On Thu, Dec 06, 2012 at 04:33:33PM +0100, Stefano Babic wrote:
> On 06/12/2012 16:23, Lukasz Majewski wrote:
> > Do not compile in FDT related code, when it is not supported.
> > 
> > Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
> > Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> > ---
> >  common/cmd_spl.c |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/common/cmd_spl.c b/common/cmd_spl.c
> > index 9ec054a..e3c543b 100644
> > --- a/common/cmd_spl.c
> > +++ b/common/cmd_spl.c
> > @@ -130,10 +130,12 @@ static int spl_export(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> >  		if (call_bootm(argc, argv, subcmd_list[(int)c->cmd]))
> >  			return -1;
> >  		switch ((int)c->cmd) {
> > +#ifdef CONFIG_OF_LIBFDT
> >  		case SPL_EXPORT_FDT:
> >  			printf("Argument image is now in RAM: 0x%p\n",
> >  				(void *)images.ft_addr);
> >  			break;
> > +#endif
> >  		case SPL_EXPORT_ATAGS:
> >  			printf("Argument image is now in RAM at: 0x%p\n",
> >  				(void *)gd->bd->bi_boot_params);
> > 
> 
> Acked-by: Stefano Babic <sbabic@denx.de>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20121212/2b6b3646/attachment.pgp>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-12-12 13:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-06 15:23 [U-Boot] [PATCH] cmd:spl:fix: Prevent from a build error on boards, which don't support FDT Lukasz Majewski
2012-12-06 15:33 ` Stefano Babic
2012-12-12 13:15   ` Tom Rini

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.