diff --git a/bsp/gd32450z-eval/rtconfig.py b/bsp/gd32450z-eval/rtconfig.py
index 8e01cef1692384c37fc8308c01fbdae1d5085b88..70ffabba2f82de7a3a4fc573e0124d109bff9ca3 100644
--- a/bsp/gd32450z-eval/rtconfig.py
+++ b/bsp/gd32450z-eval/rtconfig.py
@@ -3,7 +3,7 @@ import os
 # toolchains options
 ARCH='arm'
 CPU='cortex-m4'
-CROSS_TOOL='gcc'
+CROSS_TOOL='keil'
 
 if os.getenv('RTT_CC'):
     CROSS_TOOL = os.getenv('RTT_CC')
@@ -17,14 +17,13 @@ if  CROSS_TOOL == 'gcc':
     print 'Not support gcc yet!'
     print '================================================='
     PLATFORM    = 'gcc'
-    exit(0)
 elif CROSS_TOOL == 'keil':
     PLATFORM 	= 'armcc'
     EXEC_PATH 	= r'C:/Keil_v5'
 elif CROSS_TOOL == 'iar':
     PLATFORM 	= 'iar'
     EXEC_PATH 	= r'C:/Program Files (x86)/IAR Systems/Embedded Workbench 8.0'
-    
+
 if os.getenv('RTT_EXEC_PATH'):
 	EXEC_PATH = os.getenv('RTT_EXEC_PATH')
 
diff --git a/bsp/imx6ul/SConstruct b/bsp/imx6ul/SConstruct
index d3363ef87ecb14fb05df42adbf97d3d9872a8cd4..eaf40ea2d8bb2f70f94b3e30e3822e18606162b4 100644
--- a/bsp/imx6ul/SConstruct
+++ b/bsp/imx6ul/SConstruct
@@ -15,7 +15,7 @@ TARGET = 'rtthread-imx6.' + rtconfig.TARGET_EXT
 env = Environment(tools = ['mingw'],
 	AS = rtconfig.AS, ASFLAGS = rtconfig.AFLAGS,
 	CC = rtconfig.CC, CCFLAGS = rtconfig.CFLAGS,
-        CXX= rtconfig.CXX, CXXFLAGS = rtconfig.CFLAGS,
+    CXX= rtconfig.CXX, CXXFLAGS = rtconfig.CFLAGS,
 	AR = rtconfig.AR, ARFLAGS = '-rc',
 	LINK = rtconfig.LINK, LINKFLAGS = rtconfig.LFLAGS)
 env.PrependENVPath('PATH', rtconfig.EXEC_PATH)
diff --git a/bsp/lpc824/SConstruct b/bsp/lpc824/SConstruct
index 1a46ab9bed78bf431938159a973c5d5e39016bb3..9d111f88211820ddbc3fa343291d9365c5bdfbc0 100644
--- a/bsp/lpc824/SConstruct
+++ b/bsp/lpc824/SConstruct
@@ -8,8 +8,7 @@ if os.getenv('RTT_ROOT'):
 else:
     #RTT_ROOT = os.path.join(Dir('#').get_abspath(), 'rt-thread')
 	RTT_ROOT = os.path.normpath(os.getcwd() + '/../..')
-print RTT_ROOT
-	
+
 sys.path = sys.path + [os.path.join(RTT_ROOT, 'tools')]
 from building import *
 
diff --git a/bsp/mini4020/rtconfig.py b/bsp/mini4020/rtconfig.py
index f3aac45cdb46d75ca12b94ae22d025eadf6070a5..9d7b859db8c0987c025d54c91a45382d61ee475a 100644
--- a/bsp/mini4020/rtconfig.py
+++ b/bsp/mini4020/rtconfig.py
@@ -10,7 +10,7 @@ CROSS_TOOL 	= 'keil'
 if os.getenv('RTT_CC'):
 	CROSS_TOOL = os.getenv('RTT_CC')
 
-if  CROSS_TOOL == 'gcc':
+if CROSS_TOOL == 'gcc':
     print '================ERROR============================'
     print 'Not support gcc yet!'
     print '================================================='
diff --git a/tools/keil.py b/tools/keil.py
index 07e8254169435a00d63a67647eadc29674fe65b6..1dbb55634d5f68921965bdcd671be021ab3100eb 100644
--- a/tools/keil.py
+++ b/tools/keil.py
@@ -226,7 +226,7 @@ def MDK45Project(tree, target, script):
             if CPPDEFINES:
                 CPPDEFINES += group['CPPDEFINES']
             else:
-                CPPDEFINES += group['CPPDEFINES']
+                CPPDEFINES = group['CPPDEFINES']
 
         # get each group's link flags
         if group.has_key('LINKFLAGS') and group['LINKFLAGS']:
@@ -332,9 +332,9 @@ def MDKProject(target, script):
         # get each group's definitions
         if group.has_key('CPPDEFINES') and group['CPPDEFINES']:
             if CPPDEFINES:
-                CPPDEFINES += ';' + group['CPPDEFINES']
-            else:
                 CPPDEFINES += group['CPPDEFINES']
+            else:
+                CPPDEFINES = group['CPPDEFINES']
 
         # get each group's link flags
         if group.has_key('LINKFLAGS') and group['LINKFLAGS']: