diff options
| author | Paul Sokolovsky <pfalcon@users.sourceforge.net> | 2015-11-14 16:13:58 +0200 |
|---|---|---|
| committer | Paul Sokolovsky <pfalcon@users.sourceforge.net> | 2015-11-14 16:14:08 +0200 |
| commit | 16d42368a64cca3927ceac26b33015f73d501427 (patch) | |
| tree | 55781462f9dd459a2238f4f736c180707a5e52b8 /stmhal/i2c.c | |
| parent | 908f5159cf9f0ef5b07de243a76b6cfab88b3f0f (diff) | |
| download | micropython-16d42368a64cca3927ceac26b33015f73d501427.tar.gz micropython-16d42368a64cca3927ceac26b33015f73d501427.zip | |
stmhal/modmachine: Initial attempt to add I2C & SPI classes.
In new hardware API, these classes implement master modes of interfaces,
and "mode" parameter is not accepted. Trying to implement new HW API
in terms of older pyb module leaves variuos corner cases:
In new HW API, I2C(1) means "I2C #1 in master mode" (? depends on
interpretation), while in old API, it means "I2C #1, with no settings
changes".
For I2C class, it's easy to make mode optional, because that's last
positional param, but for SPI, there's "baudrate" after it (which
is inconsistent with I2C, which requires "baudrate" to be kwonly-arg).
Diffstat (limited to 'stmhal/i2c.c')
| -rw-r--r-- | stmhal/i2c.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/stmhal/i2c.c b/stmhal/i2c.c index eb6716ce39..82f70471a9 100644 --- a/stmhal/i2c.c +++ b/stmhal/i2c.c @@ -349,7 +349,7 @@ STATIC void pyb_i2c_print(const mp_print_t *print, mp_obj_t self_in, mp_print_ki /// - `gencall` is whether to support general call mode STATIC mp_obj_t pyb_i2c_init_helper(const pyb_i2c_obj_t *self, mp_uint_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) { static const mp_arg_t allowed_args[] = { - { MP_QSTR_mode, MP_ARG_REQUIRED | MP_ARG_INT, {.u_int = 0} }, + { MP_QSTR_mode, MP_ARG_INT, {.u_int = PYB_I2C_MASTER} }, { MP_QSTR_addr, MP_ARG_KW_ONLY | MP_ARG_INT, {.u_int = 0x12} }, { MP_QSTR_baudrate, MP_ARG_KW_ONLY | MP_ARG_INT, {.u_int = MICROPY_HW_I2C_BAUDRATE_DEFAULT} }, { MP_QSTR_gencall, MP_ARG_KW_ONLY | MP_ARG_BOOL, {.u_bool = false} }, |