[U-Boot] [PATCH] LMB: Fix undefined lmb_reserve() on non-lmb platforms

[U-Boot] [PATCH] LMB: Fix undefined lmb_reserve() on non-lmb platforms

[Marek Vasut]

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
Cc: Mike Frysinger <vapier@gentoo.org>
Cc: Stephen Warren <swarren@nvidia.com>
Cc: Tom Warren <TWarren@nvidia.com>
Cc: Graeme Russ <graeme.russ@gmail.com>
---
 common/cmd_bootm.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
index 9ad2535..5685232 100644
--- a/common/cmd_bootm.c
+++ b/common/cmd_bootm.c
@@ -186,6 +186,7 @@ static void boot_start_lmb(bootm_headers_t *images)
 	board_lmb_reserve(&images->lmb);
 }
 #else
+#define lmb_reserve(lmb, base, size)
 static inline void boot_start_lmb(bootm_headers_t *images) { }
 #endif
 
-- 
1.7.9.1

[U-Boot] [PATCH] LMB: Fix undefined lmb_reserve() on non-lmb platforms

[Marek Vasut]

Dear Marek Vasut,

> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Wolfgang Denk <wd@denx.de>
> Cc: Mike Frysinger <vapier@gentoo.org>
> Cc: Stephen Warren <swarren@nvidia.com>
> Cc: Tom Warren <TWarren@nvidia.com>
> Cc: Graeme Russ <graeme.russ@gmail.com>
> ---
>  common/cmd_bootm.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
> index 9ad2535..5685232 100644
> --- a/common/cmd_bootm.c
> +++ b/common/cmd_bootm.c
> @@ -186,6 +186,7 @@ static void boot_start_lmb(bootm_headers_t *images)
>  	board_lmb_reserve(&images->lmb);
>  }
>  #else
> +#define lmb_reserve(lmb, base, size)

Though I have to admit, this is compiler-rape. It's again depending on the fact 
that preprocessor will remove this altogether and compiler then won't use any of 
the variables :-(

>  static inline void boot_start_lmb(bootm_headers_t *images) { }
>  #endif

Best regards,
Marek Vasut

[U-Boot] [PATCH] LMB: Fix undefined lmb_reserve() on non-lmb platforms

[Mike Frysinger]

Acked-by: Mike Frysinger <vapier@gentoo.org>
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120401/447515c4/attachment.pgp>

[U-Boot] [PATCH] LMB: Fix undefined lmb_reserve() on non-lmb platforms

[Wolfgang Denk]

Dear Marek Vasut,

In message <1333288786-10296-1-git-send-email-marex@denx.de> you wrote:
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Wolfgang Denk <wd@denx.de>
> Cc: Mike Frysinger <vapier@gentoo.org>
> Cc: Stephen Warren <swarren@nvidia.com>
> Cc: Tom Warren <TWarren@nvidia.com>
> Cc: Graeme Russ <graeme.russ@gmail.com>
> ---
>  common/cmd_bootm.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The human mind ordinarily operates at only ten percent of its capaci-
ty - the rest is overhead for the operating system.