summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorScott Shawcroft <scott@tannewt.org>2020-01-03 15:14:37 -0800
committerScott Shawcroft <scott@tannewt.org>2020-01-03 15:14:37 -0800
commit6afb8dadbcc91562deb413638e2ef7e17f3305a6 (patch)
tree998bf9b9e6494de9fee690c80730a34246cd3cf7
parentcfd71d90234c5a2ec757f905b969637d0bcab226 (diff)
Change SPI and I2C in the same way.
-rw-r--r--ports/nrf/common-hal/busio/I2C.c9
-rw-r--r--ports/nrf/common-hal/busio/SPI.c9
2 files changed, 2 insertions, 16 deletions
diff --git a/ports/nrf/common-hal/busio/I2C.c b/ports/nrf/common-hal/busio/I2C.c
index 7a079ff7f..f6f686cdf 100644
--- a/ports/nrf/common-hal/busio/I2C.c
+++ b/ports/nrf/common-hal/busio/I2C.c
@@ -63,7 +63,7 @@ void i2c_reset(void) {
if (never_reset[i]) {
continue;
}
- nrf_twim_disable(twim_peripherals[i].twim.p_twim);
+ nrfx_twim_uninit(&twim_peripherals[i].twim);
twim_peripherals[i].in_use = false;
}
}
@@ -150,13 +150,6 @@ void common_hal_busio_i2c_construct(busio_i2c_obj_t *self, const mcu_pin_obj_t *
// About to init. If we fail after this point, common_hal_busio_i2c_deinit() will set in_use to false.
self->twim_peripheral->in_use = true;
nrfx_err_t err = nrfx_twim_init(&self->twim_peripheral->twim, &config, NULL, NULL);
-
- // A soft reset doesn't uninit the driver so we might end up with a invalid state
- if (err == NRFX_ERROR_INVALID_STATE) {
- nrfx_twim_uninit(&self->twim_peripheral->twim);
- err = nrfx_twim_init(&self->twim_peripheral->twim, &config, NULL, NULL);
- }
-
if (err != NRFX_SUCCESS) {
common_hal_busio_i2c_deinit(self);
mp_raise_OSError(MP_EIO);
diff --git a/ports/nrf/common-hal/busio/SPI.c b/ports/nrf/common-hal/busio/SPI.c
index d2977909e..639bbcdd0 100644
--- a/ports/nrf/common-hal/busio/SPI.c
+++ b/ports/nrf/common-hal/busio/SPI.c
@@ -68,7 +68,7 @@ void spi_reset(void) {
if (never_reset[i]) {
continue;
}
- nrf_spim_disable(spim_peripherals[i].spim.p_reg);
+ nrfx_spim_uninit(&spim_peripherals[i].spim);
}
}
@@ -160,13 +160,6 @@ void common_hal_busio_spi_construct(busio_spi_obj_t *self, const mcu_pin_obj_t *
}
nrfx_err_t err = nrfx_spim_init(&self->spim_peripheral->spim, &config, NULL, NULL);
-
- // A soft reset doesn't uninit the driver so we might end up with a invalid state
- if (err == NRFX_ERROR_INVALID_STATE) {
- nrfx_spim_uninit(&self->spim_peripheral->spim);
- err = nrfx_spim_init(&self->spim_peripheral->spim, &config, NULL, NULL);
- }
-
if (err != NRFX_SUCCESS) {
common_hal_busio_spi_deinit(self);
mp_raise_OSError(MP_EIO);