[U-Boot] [PATCH 2/3] board/tqm85xx: Clean up PPC_DDR_WIMGE define logic in config file

2011-07-15 Thread Becky Bruce
The way I wrote this in the first time was based on the original
code, but I used a karnaugh map to make it a bit easier to
read.  There should be no functional change.

Signed-off-by: Becky Bruce bec...@kernel.crashing.org
---
 include/configs/TQM85xx.h |7 ++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/include/configs/TQM85xx.h b/include/configs/TQM85xx.h
index 79a958d..b4eef28 100644
--- a/include/configs/TQM85xx.h
+++ b/include/configs/TQM85xx.h
@@ -147,10 +147,15 @@
  * DDR Setup
  */
 #define CONFIG_SYS_DDR_SDRAM_BASE  0x  /* DDR is system memory 
*/
+
 #if defined(CONFIG_TQM_BIGFLASH) || \
-   (!defined(CONFIG_TQM8548_AG)  !defined(CONFIG_TQM8548_BE))
+   defined(CONFIG_TQM8548_AG) || \
+   defined(CONFIG_TQM8548_BE)
 #define CONFIG_SYS_PPC_DDR_WIMGE (MAS2_I | MAS2_G)
+#else
+#define CONFIG_SYS_PPC_DDR_WIMGE (0)
 #endif
+
 #define CONFIG_SYS_SDRAM_BASE  CONFIG_SYS_DDR_SDRAM_BASE
 #ifdef CONFIG_TQM8548_AG
 #define CONFIG_VERY_BIG_RAM
-- 
1.5.6.5


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


Re: [U-Boot] [PATCH 2/3] board/tqm85xx: Clean up PPC_DDR_WIMGE define logic in config file

2011-07-15 Thread Wolfgang Denk
Dear Becky Bruce,

In message 13107687401917-git-send-email-bec...@kernel.crashing.org you wrote:
 The way I wrote this in the first time was based on the original
 code, but I used a karnaugh map to make it a bit easier to
 read.  There should be no functional change.

Ummm...

  #if defined(CONFIG_TQM_BIGFLASH) || \
 - (!defined(CONFIG_TQM8548_AG)  !defined(CONFIG_TQM8548_BE))
 + defined(CONFIG_TQM8548_AG) || \
 + defined(CONFIG_TQM8548_BE)

Sorry, but this looks wrong to me.

Isn't the rule foo  bar == !(!foo || !bar), so that

(!defined(CONFIG_TQM8548_AG)  !defined(CONFIG_TQM8548_BE))

becomes

!(defined(CONFIG_TQM8548_AG) || defined(CONFIG_TQM8548_BE))

?

I think you are missing the not part...

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: w...@denx.de
Heavier than air flying machines are impossible.
-- Lord Kelvin, President, Royal Society, c. 1895
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot