diff --git a/conf/modules/gps_sirf.xml b/conf/modules/gps_sirf.xml
index 065c142228..f004cd6521 100644
--- a/conf/modules/gps_sirf.xml
+++ b/conf/modules/gps_sirf.xml
@@ -18,7 +18,7 @@
   <init fun="gps_sirf_init()"/>
   <periodic fun="gps_sirf_periodic_check()" freq="1." autorun="TRUE"/>
   <event fun="gps_sirf_event()"/>
-  <makefile target=target="ap|fbw">
+  <makefile target="ap|fbw">
     <configure name="SIRF_GPS_PORT" default="$(GPS_PORT)" case="upper|lower"/>
     <configure name="SIRF_GPS_BAUD" default="$(GPS_BAUD)"/>
 
diff --git a/conf/modules/sys_time.xml b/conf/modules/sys_time.xml
index 57a78f94c4..e4e0ea460f 100644
--- a/conf/modules/sys_time.xml
+++ b/conf/modules/sys_time.xml
@@ -5,7 +5,7 @@
     <description>
       Sys-time peripheral
     </description>
-    <configure name="SYS_TIME_LED" value="none|num" value="LED number used for systime heartbeat or 'none' to disable"/>
+    <configure name="SYS_TIME_LED" value="none|num" description="LED number used for systime heartbeat or 'none' to disable"/>
   </doc>
   <header>
     <file name="sys_time.h" dir="mcu_periph"/>
@@ -15,7 +15,7 @@
     <define name="SYS_TIME_LED" value="$(SYS_TIME_LED)" cond="ifneq ($(SYS_TIME_LED), none)"/>
     <file name="sys_time.c" dir="mcu_periph"/>
     <file_arch name="sys_time_arch.c" dir="mcu_periph"/>
-    <flag name="LDFLAGS" value="lrt" cond="ifeq ($(ARCH), linux)">
+    <flag name="LDFLAGS" value="lrt" cond="ifeq ($(ARCH), linux)"/>
   </makefile>
 </module>