esp8266: Add the methods duty_u16() and duty_ns() for consistency.

robert-hh · robert-hh · commit 53591e060598 · 2023-02-26T15:58:10.000+01:00
diff --git a/ports/esp8266/machine_pwm.c b/ports/esp8266/machine_pwm.c
@@ -53,10 +53,12 @@ STATIC void mp_machine_pwm_print(const mp_print_t *print, mp_obj_t self_in, mp_p
 }
 
 STATIC void mp_machine_pwm_init_helper(machine_pwm_obj_t *self, size_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) {
-    enum { ARG_freq, ARG_duty };
+    enum { ARG_freq, ARG_duty, ARG_duty_u16, ARG_duty_ns };
     static const mp_arg_t allowed_args[] = {
         { MP_QSTR_freq, MP_ARG_INT, {.u_int = -1} },
         { MP_QSTR_duty, MP_ARG_INT, {.u_int = -1} },
+        { MP_QSTR_duty_u16, MP_ARG_INT, {.u_int = -1} },
+        { MP_QSTR_duty_ns, MP_ARG_INT, {.u_int = -1} },
     };
     mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)];
     mp_arg_parse_all(n_args, pos_args, kw_args, MP_ARRAY_SIZE(allowed_args), allowed_args, args);
@@ -74,6 +76,15 @@ STATIC void mp_machine_pwm_init_helper(machine_pwm_obj_t *self, size_t n_args, c
     if (args[ARG_duty].u_int != -1) {
         pwm_set_duty(args[ARG_duty].u_int, self->channel);
     }
+    if (args[ARG_duty_u16].u_int != -1) {
+        pwm_set_duty(args[ARG_duty].u_int * 100 / 65536, self->channel);
+    }
+    if (args[ARG_duty_u16].u_int != -1) {
+        uint32_t freq = pwm_get_freq(0);
+        if (freq > 0) {
+            pwm_set_duty(args[ARG_duty].u_int * freq / 10000000, self->channel);
+        }
+    }
 
     if (pin_mode[self->pin->phys_port] == GPIO_MODE_OPEN_DRAIN) {
         mp_hal_pin_open_drain(self->pin->phys_port);
@@ -121,23 +132,47 @@ STATIC void mp_machine_pwm_freq_set(machine_pwm_obj_t *self, mp_int_t freq) {
     pwm_start();
 }
 
-STATIC mp_obj_t mp_machine_pwm_duty_get(machine_pwm_obj_t *self) {
+STATIC void set_active(machine_pwm_obj_t *self, bool set_pin) {
     if (!self->active) {
         pwm_add(self->pin->phys_port, self->pin->periph, self->pin->func);
         self->active = 1;
-
-        if (pin_mode[self->pin->phys_port] == GPIO_MODE_OPEN_DRAIN) {
+        if (set_pin && pin_mode[self->pin->phys_port] == GPIO_MODE_OPEN_DRAIN) {
             mp_hal_pin_open_drain(self->pin->phys_port);
         }
     }
+}
+
+STATIC mp_obj_t mp_machine_pwm_duty_get(machine_pwm_obj_t *self) {
+    set_active(self, true);
     return MP_OBJ_NEW_SMALL_INT(pwm_get_duty(self->channel));
 }
 
 STATIC void mp_machine_pwm_duty_set(machine_pwm_obj_t *self, mp_int_t duty) {
-    if (!self->active) {
-        pwm_add(self->pin->phys_port, self->pin->periph, self->pin->func);
-        self->active = 1;
-    }
+    set_active(self, false);
     pwm_set_duty(duty, self->channel);
     pwm_start();
 }
+
+STATIC mp_obj_t mp_machine_pwm_duty_get_u16(machine_pwm_obj_t *self) {
+    set_active(self, true);
+    return MP_OBJ_NEW_SMALL_INT(pwm_get_duty(self->channel) * 65536 / 1024);
+}
+
+STATIC void mp_machine_pwm_duty_set_u16(machine_pwm_obj_t *self, mp_int_t duty) {
+    set_active(self, false);
+    pwm_set_duty(duty * 1024 / 65536, self->channel);
+    pwm_start();
+}
+
+STATIC mp_obj_t mp_machine_pwm_duty_get_ns(machine_pwm_obj_t *self) {
+    set_active(self, true);
+    uint32_t freq = pwm_get_freq(0);
+    return MP_OBJ_NEW_SMALL_INT(pwm_get_duty(self->channel) * 976563 / freq);
+}
+
+STATIC void mp_machine_pwm_duty_set_ns(machine_pwm_obj_t *self, mp_int_t duty) {
+    set_active(self, false);
+    uint32_t freq = pwm_get_freq(0);
+    pwm_set_duty(duty * freq / 976562, self->channel); // 1e9/1024 = 976562.5
+    pwm_start();
+}
diff --git a/ports/esp8266/mpconfigport.h b/ports/esp8266/mpconfigport.h
@@ -66,6 +66,7 @@
 #define MICROPY_PY_MACHINE_PWM      (1)
 #define MICROPY_PY_MACHINE_PWM_INIT (1)
 #define MICROPY_PY_MACHINE_PWM_DUTY (1)
+#define MICROPY_PY_MACHINE_PWM_DUTY_U16_NS  (1)
 #define MICROPY_PY_MACHINE_PWM_INCLUDEFILE "ports/esp8266/machine_pwm.c"
 #define MICROPY_PY_MACHINE_I2C      (1)
 #define MICROPY_PY_MACHINE_SOFTI2C  (1)
