diff options
author | Thorsten von Eicken <tve@voneicken.com> | 2020-03-31 16:03:01 -0700 |
---|---|---|
committer | Damien George <damien.p.george@gmail.com> | 2020-04-23 10:59:07 +1000 |
commit | 1ae7e0e561134c39471e3000eced33da732b4103 (patch) | |
tree | a1d1f7393863cf0fddf8dcae82aabd1e04768971 /ports/esp32/esp32_rmt.c | |
parent | a177831c46a7e217fed6562510e12183377053f7 (diff) |
esp32: Consolidate check_esp_err functions and add IDF error string.
This commit consolidates a number of check_esp_err functions that check
whether an ESP-IDF return code is OK and raises an exception if not. The
exception raised is an OSError with the error code as the first argument
(negative if it's ESP-IDF specific) and the ESP-IDF error string as the
second argument.
This commit also fixes esp32.Partition.set_boot to use check_esp_err, and
uses that function for a unit test.
Diffstat (limited to 'ports/esp32/esp32_rmt.c')
-rw-r--r-- | ports/esp32/esp32_rmt.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/ports/esp32/esp32_rmt.c b/ports/esp32/esp32_rmt.c index 799875266..25b7b3808 100644 --- a/ports/esp32/esp32_rmt.c +++ b/ports/esp32/esp32_rmt.c @@ -56,17 +56,6 @@ typedef struct _esp32_rmt_obj_t { rmt_item32_t *items; } esp32_rmt_obj_t; -// Defined in machine_time.c; simply added the error message -// Fixme: Should use this updated error hadline more widely in the ESP32 port. -// At least update the method in machine_time.c. -STATIC esp_err_t check_esp_err(esp_err_t code) { - if (code) { - mp_raise_msg(&mp_type_OSError, (mp_rom_error_text_t)esp_err_to_name(code)); - } - - return code; -} - STATIC mp_obj_t esp32_rmt_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *all_args) { static const mp_arg_t allowed_args[] = { { MP_QSTR_id, MP_ARG_REQUIRED | MP_ARG_INT, {.u_int = -1} }, |