cpu: adjusted pwm driver impl for lpc2387

boards/msba2/include/periph_conf.h

@@ -28,6 +28,7 @@
 #define PWM_0_EN            (1)
 
 /* PWM_0 device configuration */
+#define PWM_0_CHANNELS      (3)
 #define PWM_0_CH0           (3)
 #define PWM_0_CH0_MR        PWM1MR3
 #define PWM_0_CH1           (4)

cpu/lpc2387/periph/pwm.c

@@ -23,6 +23,8 @@
 #include "periph/pwm.h"
 #include "periph_conf.h"
 
+/* guard file in case no PWM device is defined */
+#if PWM_NUMOF
 
 #define PCPWM1      BIT6
 
@@ -47,7 +49,7 @@ int pwm_init(pwm_t dev, pwm_mode_t mode, unsigned int frequency, unsigned int re
                           (PWM_0_FUNC << PWM_0_CH2_PIN * 2);
 
             /* power on PWM1 */
-            PCONP |= BIT6;
+            pwm_poweron(dev);
 
             /* select PWM1 clock */
             PCLKSEL0 &= ~(BIT13);
@@ -78,9 +80,6 @@ int pwm_init(pwm_t dev, pwm_mode_t mode, unsigned int frequency, unsigned int re
             PWM1LER = BIT0 | (1 << PWM_0_CH0) | (1 << PWM_0_CH1) | (1 << PWM_0_CH2);
             break;
 #endif
-        case PWM_UNDEFINED:
-        default:
-            return -1;
     }
 
     return 0;
@@ -110,42 +109,53 @@ int pwm_set(pwm_t dev, int channel, unsigned int value)
             }
             break;
 #endif
-        case PWM_UNDEFINED:
-        default:
-            return -1;
     }
 
     return 0;
 }
 
-int pwm_start(pwm_t dev)
+void pwm_start(pwm_t dev)
 {
     switch (dev) {
 #if PWM_0_EN
         case PWM_0:
-            PCONP |= PCPWM1;        /* enable PWM1 device */
+            PWM1TCR |= BIT0;
             break;
 #endif
-        case PWM_UNDEFINED:
-        default:
-            return -1;
     }
+}
 
-    return 0;
+void pwm_stop(pwm_t dev)
+{
+    switch (dev) {
+#if PWM_0_EN
+        case PWM_0:
+            PWM1TCR &= ~(BIT0);
+            break;
+#endif
+    }
 }
 
-int pwm_stop(pwm_t dev)
+void pwm_poweron(pwm_t dev)
 {
     switch (dev) {
 #if PWM_0_EN
         case PWM_0:
-            PCONP &= ~PCPWM1;       /* disable PWM1 device */
+            PCONP |= PCPWM1;
             break;
 #endif
-        case PWM_UNDEFINED:
-        default:
-            return -1;
     }
+}
 
-    return 0;
+void pwm_poweroff(pwm_t dev)
+{
+    switch (dev) {
+#if PWM_0_EN
+        case PWM_0:
+            PCONP &= ~(PCPWM1);
+            break;
+#endif
+    }
 }
+
+#endif /* PWM_NUMOF */