~ruther/qmk_firmware

8e550fc11a59326cb759593418ce1a062392daaf — Nick Brassel 5 years ago 1aee492
Add thread safety around i2c_master for ChibiOS/ARM.
1 files changed, 51 insertions(+), 12 deletions(-)

M drivers/arm/i2c_master.c
M drivers/arm/i2c_master.c => drivers/arm/i2c_master.c +51 -12
@@ 29,8 29,6 @@
#include <string.h>
#include <hal.h>

static uint8_t i2c_address;

static const I2CConfig i2cconfig = {
#ifdef USE_I2CV1
    I2C1_OPMODE,


@@ 71,27 69,49 @@ __attribute__((weak)) void i2c_init(void) {
}

i2c_status_t i2c_start(uint8_t address) {
    i2c_address = address;
#if I2C_USE_MUTUAL_EXCLUSION
    i2cAcquireBus(&I2C_DRIVER);
#endif

    i2cStart(&I2C_DRIVER, &i2cconfig);
    return I2C_STATUS_SUCCESS;
}

i2c_status_t i2c_transmit(uint8_t address, const uint8_t* data, uint16_t length, uint16_t timeout) {
    i2c_address = address;
#if I2C_USE_MUTUAL_EXCLUSION
    i2cAcquireBus(&I2C_DRIVER);
#endif

    i2cStart(&I2C_DRIVER, &i2cconfig);
    msg_t status = i2cMasterTransmitTimeout(&I2C_DRIVER, (i2c_address >> 1), data, length, 0, 0, MS2ST(timeout));
    msg_t status = i2cMasterTransmitTimeout(&I2C_DRIVER, (address >> 1), data, length, 0, 0, MS2ST(timeout));

#if I2C_USE_MUTUAL_EXCLUSION
    i2cReleaseBus(&I2C_DRIVER);
#endif

    return chibios_to_qmk(&status);
}

i2c_status_t i2c_receive(uint8_t address, uint8_t* data, uint16_t length, uint16_t timeout) {
    i2c_address = address;
#if I2C_USE_MUTUAL_EXCLUSION
    i2cAcquireBus(&I2C_DRIVER);
#endif

    i2cStart(&I2C_DRIVER, &i2cconfig);
    msg_t status = i2cMasterReceiveTimeout(&I2C_DRIVER, (i2c_address >> 1), data, length, MS2ST(timeout));
    msg_t status = i2cMasterReceiveTimeout(&I2C_DRIVER, (address >> 1), data, length, MS2ST(timeout));

#if I2C_USE_MUTUAL_EXCLUSION
    i2cReleaseBus(&I2C_DRIVER);
#endif

    return chibios_to_qmk(&status);
}

i2c_status_t i2c_writeReg(uint8_t devaddr, uint8_t regaddr, const uint8_t* data, uint16_t length, uint16_t timeout) {
    i2c_address = devaddr;
#if I2C_USE_MUTUAL_EXCLUSION
    i2cAcquireBus(&I2C_DRIVER);
#endif

    i2cStart(&I2C_DRIVER, &i2cconfig);

    uint8_t complete_packet[length + 1];


@@ 100,15 120,34 @@ i2c_status_t i2c_writeReg(uint8_t devaddr, uint8_t regaddr, const uint8_t* data,
    }
    complete_packet[0] = regaddr;

    msg_t status = i2cMasterTransmitTimeout(&I2C_DRIVER, (i2c_address >> 1), complete_packet, length + 1, 0, 0, MS2ST(timeout));
    msg_t status = i2cMasterTransmitTimeout(&I2C_DRIVER, (devaddr >> 1), complete_packet, length + 1, 0, 0, MS2ST(timeout));

#if I2C_USE_MUTUAL_EXCLUSION
    i2cReleaseBus(&I2C_DRIVER);
#endif

    return chibios_to_qmk(&status);
}

i2c_status_t i2c_readReg(uint8_t devaddr, uint8_t regaddr, uint8_t* data, uint16_t length, uint16_t timeout) {
    i2c_address = devaddr;
#if I2C_USE_MUTUAL_EXCLUSION
    i2cAcquireBus(&I2C_DRIVER);
#endif

    i2cStart(&I2C_DRIVER, &i2cconfig);
    msg_t status = i2cMasterTransmitTimeout(&I2C_DRIVER, (i2c_address >> 1), &regaddr, 1, data, length, MS2ST(timeout));
    msg_t status = i2cMasterTransmitTimeout(&I2C_DRIVER, (devaddr >> 1), &regaddr, 1, data, length, MS2ST(timeout));

#if I2C_USE_MUTUAL_EXCLUSION
    i2cReleaseBus(&I2C_DRIVER);
#endif

    return chibios_to_qmk(&status);
}

void i2c_stop(void) { i2cStop(&I2C_DRIVER); }
void i2c_stop(void) {
    i2cStop(&I2C_DRIVER);

#if I2C_USE_MUTUAL_EXCLUSION
    i2cReleaseBus(&I2C_DRIVER);
#endif
}