Re: [U-Boot] [PATCH] common: fix include guards for CONFIG_MP

2014-09-10 Thread York Sun
On 09/03/2014 01:57 PM, Gabriel Huau wrote:
 This was breaking the build for some boards:
 MPC8536DS MPC8536DS_36BIT MPC8536DS_SDCARD MPC8536DS_SPIFLASH qemu-ppce500
 
 Include only these features for some PPC boards if the configuration for 
 MultiProcessor
 is enabled.
 
 Signed-off-by: Gabriel Huau cont...@huau-gabriel.fr
 Cc: Tom Rini tr...@ti.com
 Cc: York Sun york...@freescale.com
 ---
  common/board_f.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/common/board_f.c b/common/board_f.c
 index 4ece2b6..deea9ca 100644
 --- a/common/board_f.c
 +++ b/common/board_f.c
 @@ -34,7 +34,7 @@
  #ifdef CONFIG_MPC5xxx
  #include mpc5xxx.h
  #endif
 -#if (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
 +#if defined(CONFIG_MP)  (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
  #include asm/mp.h
  #endif
  
 @@ -392,7 +392,7 @@ static int setup_dest_addr(void)
   gd-ram_top = board_get_usable_ram_top(gd-mon_len);
   gd-relocaddr = gd-ram_top;
   debug(Ram top: %08lX\n, (ulong)gd-ram_top);
 -#if (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
 +#if defined(CONFIG_MP)  (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
   /*
* We need to make sure the location we intend to put secondary core
* boot code is reserved and not used by any part of u-boot
 
Acked-by: York Sun york...@freescale.com

York
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] common: fix include guards for CONFIG_MP

2014-09-03 Thread York Sun
On 09/03/2014 01:57 PM, Gabriel Huau wrote:
 This was breaking the build for some boards:
 MPC8536DS MPC8536DS_36BIT MPC8536DS_SDCARD MPC8536DS_SPIFLASH qemu-ppce500
 
 Include only these features for some PPC boards if the configuration for 
 MultiProcessor
 is enabled.
 
 Signed-off-by: Gabriel Huau cont...@huau-gabriel.fr
 Cc: Tom Rini tr...@ti.com
 Cc: York Sun york...@freescale.com
 ---
  common/board_f.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/common/board_f.c b/common/board_f.c
 index 4ece2b6..deea9ca 100644
 --- a/common/board_f.c
 +++ b/common/board_f.c
 @@ -34,7 +34,7 @@
  #ifdef CONFIG_MPC5xxx
  #include mpc5xxx.h
  #endif
 -#if (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
 +#if defined(CONFIG_MP)  (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
  #include asm/mp.h
  #endif
  
 @@ -392,7 +392,7 @@ static int setup_dest_addr(void)
   gd-ram_top = board_get_usable_ram_top(gd-mon_len);
   gd-relocaddr = gd-ram_top;
   debug(Ram top: %08lX\n, (ulong)gd-ram_top);
 -#if (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
 +#if defined(CONFIG_MP)  (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
   /*
* We need to make sure the location we intend to put secondary core
* boot code is reserved and not used by any part of u-boot
 

Looks good.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] common: fix include guards for CONFIG_MP

2014-09-03 Thread Gabriel Huau
This was breaking the build for some boards:
MPC8536DS MPC8536DS_36BIT MPC8536DS_SDCARD MPC8536DS_SPIFLASH qemu-ppce500

Include only these features for some PPC boards if the configuration for 
MultiProcessor
is enabled.

Signed-off-by: Gabriel Huau cont...@huau-gabriel.fr
Cc: Tom Rini tr...@ti.com
Cc: York Sun york...@freescale.com
---
 common/board_f.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/board_f.c b/common/board_f.c
index 4ece2b6..deea9ca 100644
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -34,7 +34,7 @@
 #ifdef CONFIG_MPC5xxx
 #include mpc5xxx.h
 #endif
-#if (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
+#if defined(CONFIG_MP)  (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
 #include asm/mp.h
 #endif
 
@@ -392,7 +392,7 @@ static int setup_dest_addr(void)
gd-ram_top = board_get_usable_ram_top(gd-mon_len);
gd-relocaddr = gd-ram_top;
debug(Ram top: %08lX\n, (ulong)gd-ram_top);
-#if (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
+#if defined(CONFIG_MP)  (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
/*
 * We need to make sure the location we intend to put secondary core
 * boot code is reserved and not used by any part of u-boot
-- 
2.0.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot