diff --git a/stmhal/i2c.c b/stmhal/i2c.c
index b8de855304cf324072167004fa558035400c75e4..96222bf336b9926593c9266f3c330812134e1f12 100644
--- a/stmhal/i2c.c
+++ b/stmhal/i2c.c
@@ -367,7 +367,7 @@ STATIC mp_obj_t pyb_i2c_send(uint n_args, const mp_obj_t *args, mp_map_t *kw_arg
 }
 STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_i2c_send_obj, 1, pyb_i2c_send);
 
-/// \method recv(send, addr=0x00, timeout=5000)
+/// \method recv(recv, addr=0x00, timeout=5000)
 ///
 /// Receive data on the bus:
 ///
diff --git a/stmhal/pin.c b/stmhal/pin.c
index 5b0f998de62a30a7607d02a7503e11a312e4ae64..d390926360f9f3975c1dfc57cc35dcf1ca849b46 100644
--- a/stmhal/pin.c
+++ b/stmhal/pin.c
@@ -241,6 +241,7 @@ STATIC MP_DEFINE_CONST_CLASSMETHOD_OBJ(pin_debug_obj, (mp_obj_t)&pin_debug_fun_o
 ///     - `Pin.PULL_DOWN` - enable the pull-down resistor.
 ///
 /// Returns: `None`.
+// TODO allow keyword args
 STATIC mp_obj_t pin_obj_init(uint n_args, mp_obj_t *args) {
     pin_obj_t *self = args[0];
 
diff --git a/stmhal/servo.c b/stmhal/servo.c
index 68e211bcb3791d687203b419040ed706ee55d553..6f87197bd2336a38cbfd608a061e4b4b0445b755 100644
--- a/stmhal/servo.c
+++ b/stmhal/servo.c
@@ -203,6 +203,7 @@ STATIC MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(pyb_servo_pulse_width_obj, 1, 2, pyb_
 
 /// \method calibration([pulse_min, pulse_max, pulse_centre, [pulse_angle_90, pulse_speed_100]])
 /// Get or set the calibration of the servo timing.
+// TODO should accept 1 arg, a 5-tuple of values to set
 STATIC mp_obj_t pyb_servo_calibration(uint n_args, const mp_obj_t *args) {
     pyb_servo_obj_t *self = args[0];
     if (n_args == 1) {
diff --git a/stmhal/spi.c b/stmhal/spi.c
index c1b8e75d050cc67acd5e7a256f0d69529d08f5f1..a07ebd21a93b6b599a558b2e9ab1dd27e1061fc2 100644
--- a/stmhal/spi.c
+++ b/stmhal/spi.c
@@ -198,7 +198,7 @@ STATIC void pyb_spi_print(void (*print)(void *env, const char *fmt, ...), void *
     }
 }
 
-/// \method init(mode, baudrate=328125, *, polarity=1, phase=1, bits=8, firstbit=SPI.MSB, ti=false, crc=None)
+/// \method init(mode, baudrate=328125, *, polarity=1, phase=1, bits=8, firstbit=SPI.MSB, ti=False, crc=None)
 ///
 /// Initialise the SPI bus with the given parameters:
 ///