[U-Boot] [PATCH 1/1] board_r: remove superfluous #ifdef CONFIG_PRAM

[U-Boot] [PATCH 1/1] board_r: remove superfluous #ifdef CONFIG_PRAM

[Heinrich Schuchardt]

initr_mem() is already enclosed by
	#if defined(CONFIG_PRAM)
	#endif

So there is no need to check CONFIG_PRAM again inside the
function.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 common/board_r.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/common/board_r.c b/common/board_r.c
index 5e7eb2d5cb..98811edc07 100644
--- a/common/board_r.c
+++ b/common/board_r.c
@@ -624,9 +624,7 @@ int initr_mem(void)
 	ulong pram = 0;
 	char memsz[32];
 
-# ifdef CONFIG_PRAM
 	pram = env_get_ulong("pram", 10, CONFIG_PRAM);
-# endif
 	sprintf(memsz, "%ldk", (long int) ((gd->ram_size / 1024) - pram));
 	env_set("mem", memsz);
 
-- 
2.14.2

[U-Boot] [PATCH 1/1] board_r: remove superfluous #ifdef CONFIG_PRAM

[Simon Glass]

On 6 January 2018 at 16:54, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
> initr_mem() is already enclosed by
>         #if defined(CONFIG_PRAM)
>         #endif
>
> So there is no need to check CONFIG_PRAM again inside the
> function.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  common/board_r.c | 2 --
>  1 file changed, 2 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

[U-Boot] [U-Boot, 1/1] board_r: remove superfluous #ifdef CONFIG_PRAM

[Tom Rini]

On Sun, Jan 07, 2018 at 12:54:20AM +0100, Heinrich Schuchardt wrote:

> initr_mem() is already enclosed by
> 	#if defined(CONFIG_PRAM)
> 	#endif
> 
> So there is no need to check CONFIG_PRAM again inside the
> function.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, 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/20180119/6a0752e9/attachment.sig>