diff --git a/bsp/mini2440/run_module.c b/bsp/mini2440/run_module.c
index 41452821f4e941ecb02ecccc917e71d664b094fe..372e67d1fec9fcdf4b6de871f1b3adeb6ec81e0f 100644
--- a/bsp/mini2440/run_module.c
+++ b/bsp/mini2440/run_module.c
@@ -16,7 +16,7 @@
 #include <dfs_posix.h>
 #include "string.h"
 
-#ifdef RT_USING_FINSH
+#if defined(RT_USING_FINSH) && defined(RT_USING_MODULE)
 #include <finsh.h>
 static char buffer[4096];
 
diff --git a/bsp/mini2440/today.c b/bsp/mini2440/today.c
index 2ee6a4e599ceb1ef55b4bb27d0db1c77ae42ebf6..4066f4228e6cabc18d3e1d6ccfb7b59e67d3a71c 100644
--- a/bsp/mini2440/today.c
+++ b/bsp/mini2440/today.c
@@ -148,7 +148,7 @@ rt_bool_t today_workbench_event_handler(rtgui_widget_t *widget, rtgui_event_t *e
     {
         struct rtgui_event_mouse* emouse = (struct rtgui_event_mouse*)event;
         //if (emouse->button == RTGUI_MOUSE_BUTTON_UP | RTGUI_MOUSE_BUTTON_LEFT)
-	if(0)
+		if(0)
 		{
             /* active home view */
             if (workbench->current_view != home_view)
@@ -192,7 +192,7 @@ static void today_entry(void* parameter)
     /* add function view */
 	rtgui_widget_get_rect(RTGUI_WIDGET(workbench), &rect);
 	function_view = (struct rtgui_view*)rtgui_list_view_create(function_list,
-		sizeof(function_list)/sizeof(struct rtgui_list_item), &rect);
+		sizeof(function_list)/sizeof(struct rtgui_list_item), &rect, RTGUI_LIST_VIEW_LIST);
 	rtgui_workbench_add_view(workbench, function_view);
 
 	rtgui_workbench_event_loop(workbench);
diff --git a/bsp/mini2440/touch.c b/bsp/mini2440/touch.c
index a8400aea64dff3218e6308d2e2567c4c268329b4..e23a7e1ddd286b32fb693cd0d6c2ce6299761167 100644
--- a/bsp/mini2440/touch.c
+++ b/bsp/mini2440/touch.c
@@ -255,7 +255,7 @@ static void rt_touch_handler(int irqno)
 	}
 
 	/* clear interrupt */
-	INTPND |= (rt_uint32_t)(1 << INTADC);
+	INTPND |= (1ul << INTADC);
 }
 
 /* RT-Thread Device Interface */
@@ -279,7 +279,7 @@ static rt_err_t rtgui_touch_init (rt_device_t dev)
 	rt_hw_interrupt_umask(INTADC);
 
 	/* clear interrupt */
-	INTPND |= (rt_uint32_t)(1 << INTADC);
+	INTPND |= (1ul << INTADC);
 	
 	SUBSRCPND |= BIT_SUB_TC;
 	SUBSRCPND |= BIT_SUB_ADC;