RTOS enabled i2c-driver based on the official i2c-C-api.
Fork of mbed-RtosI2cDriver by
I2cRtosDriver
Overview
- Based on RTOS
- Less busy wait waste of CPU cycles
- ... but some waste of CPU cycles by context switches
- Frees up to 80% of CPU resources
- Fixes the bug described in https://mbed.org/forum/bugs-suggestions/topic/4128/
- Spends minimal time in interrupt context
- Supports I2C Master and Slave mode
- Interface compatible to official I2C lib
- Supports LPC1768 and LPC11U24.
- Reuses parts of the official I2C implementation
- The test and example programs work quite well and the results look promising. But this is by no means a thoroughly regression tested library. There might be some surprises left.
- If you want to avoid the RTOS overhead MODI2C might be a better choice.
Usage
- In existing projects simply replace in the I2C interface class declaration the official type by one of the adapters I2CMasterRtos or I2CSlaveRtos described below. The behavior should be the same.
- You can also use the I2CDriver interface directly.
- You can create several instances of I2CMasterRtos, I2CSlaveRtos and I2CDriver. The interface classes are lightweight and work in parallel.
- See also the tests/examples in I2CDriverTest01.h - I2CDriverTest05.h
- The I2CDriver class is the central interface
- I2CDriver provides a "fat" API for I2C master and slave access
- It supports on the fly changes between master and slave mode.
- All requests are blocking. Other threads might do their work while the calling thread waits for the i2c requests to be completed.
- It ensures mutual exclusive access to the I2C HW.
- This is realized by a static RTOS mutex for each I2C channel. The mutex is taken by the calling thread on any call of an I2CDriver-function.
- Thus accesses are prioritized automatically by the priority of the calling user threads.
- Once having access to the interface the requests are performed with high priority and cannot be interrupted by other threads.
- Optionally the interface can be locked manually. Useful if one wants to perform a sequence of commands without interruption.
- I2CMasterRtos and I2CSlaveRtos provide an interface compatible to the official mbed I2C interface. Additionally
- the constructors provide parameters for defining the frequency and the slave address
- I2CMasterRtos provides a function to read data from a given slave register
- In contrast to the original interface the I2CSlaveRtos::receive() function is blocking, i.e it returns, when the master sends a request to the listening slave. There is no need to poll the receive status in a loop. Optionally a timeout value can be passed to the function.
- The stop function provides a timeout mechanism and returns the status. Thus if someone on the bus inhibits the creation of a stop condition by keeping the scl or the sda line low the mbed master won't get freezed.
- The interface adapters are implemented as object adapters, i.e they hold an I2CDriver-instance, to which they forward the user requests by simple inline functions. The overhead is negligible.
Design
The i2c read and write sequences have been realized in an interrupt service routine. The communicaton between the calling thread and the ISR is realized by a simple static transfer struct and a semaphore ... see i2cRtos_api.c
The start and stop functions still use the busy wait approach. They are not entered that frequently and usually they take less than 12µs at 100kHz bus speed. At 400kHz even less time is consumed. Thus there wouldn't be much benefit if one triggers the whole interrupt/task wait/switch sequence for that short period of time.
Performance
The following performance data have been measured with the small test applications in I2CDriverTest01.h and I2CDriverTest04.h . In these applications a high priority thread, triggered at a rate of 1kHz, reads on each trigger a data packet of given size with given I2C bus speed from a SRF08 ultra sonic ranger or a MPU6050 accelerometer/gyro. At the same time the main thread - running at a lower priority - counts in an endless loop adjacent increments of the mbed's µs-ticker API and calculates a duty cycle from this. These duty cycle measurements are shown in the table below together with the time measured for one read sequence (write address+register; write address and read x byte of data). The measurements have been performed with the ISR/RTOS approach used by this driver and with the busy wait approach used by the official mbed I2C implementation. The i2c implementation can be selected via #define PREFIX in I2CDriver.cpp.
- The time for one read cycle is almost the same for both approaches
- At full load the duty cycle of the low priority thread drops almost to zero for the busy wait approach, whereas with the RTOS/ISR enabled driver it stays at 80%-90% on the LPC1768 and above 65% on the LPC11U24.
- => Especially at low bus speeds and/or high data transfer loads the driver is able to free a significant amount of CPU time.
LPC1768 | 1byte/ms | 4byte/ms | 6byte/ms | 1byte/ms | 6byte/ms | 12byte/ms | 25byte/ms | |
---|---|---|---|---|---|---|---|---|
SRF08 | @ 100kHz | @ 100kHz | @ 100kHz | @ 400kHz | @ 400kHz | @ 400kHz | @ 400kHz | |
rtos/ISR | DC[%] | 91.7 | 91.0 | 90.5 | 93.3 | 91.9 | 90.3 | 86.8 |
t[µs] | 421 | 714 | 910 | 141 | 314 | 518 | 961 | |
busy wait | DC[%] | 57.1 | 27.7 | 8.1 | 85.8 | 68.7 | 48.2 | 3.8 |
t[µs] | 415 | 710 | 907 | 128 | 299 | 503 | 949 |
LPC1768 | 1byte/ms | 4byte/ms | 7byte/ms | 1byte/ms | 6byte/ms | 12byte/ms | 36byte/ms | |
---|---|---|---|---|---|---|---|---|
MPU6050 | @ 100kHz | @ 100kHz | @ 100kHz | @ 400kHz | @ 400kHz | @ 400kHz | @ 400kHz | |
rtos/ISR | DC[%] | 91.5 | 90.7 | 89.3 | 93.0 | 91.6 | 90.0 | 84.2 |
t[µs] | 415 | 687 | 959 | 133 | 254 | 398 | 977 | |
busy wait | DC[%] | 57.7 | 30.5 | 3.3 | 86.5 | 74.3 | 59.7 | 1.2 |
t[µs] | 408 | 681 | 953 | 121 | 243 | 392 | 974 |
LPC11U24 | 1byte/ms | 6byte/ms | 1byte/ms | 6byte/ms | 23byte/ms | |
---|---|---|---|---|---|---|
SRF08 | @ 100kHz | @ 100kHz | @ 400kHz | @ 400kHz | @ 400kHz | |
rtos/ISR | DC[%] | 79.2 | 77.5 | 81.1 | 78.7 | 71.4 |
t[µs] | 474 | 975 | 199 | 374 | 978 | |
busy wait | DC[%] | 51.8 | 2.4 | 80.5 | 63 | 3.3 |
t[µs] | 442 | 937 | 156 | 332 | 928 |
LPC11U24 | 1byte/ms | 6byte/ms | 1byte/ms | 6byte/ms | 32byte/ms | |
---|---|---|---|---|---|---|
MPU6050 | @ 100kHz | @ 100kHz | @ 400kHz | @ 400kHz | @ 400kHz | |
rtos/ISR | DC[%] | 79.1 | 76.8 | 81.0 | 78.6 | 67.1 |
t[µs] | 466 | 922 | 188 | 316 | 985 | |
busy wait | DC[%] | 52.8 | 7.2 | 81.7 | 69.8 | 7.4 |
t[µs] | 433 | 893 | 143 | 268 | 895 |
Diff: I2CDriver.cpp
- Revision:
- 1:90455d5bdd8c
- Parent:
- 0:13c962fecb13
- Child:
- 2:514105beb343
diff -r 13c962fecb13 -r 90455d5bdd8c I2CDriver.cpp --- a/I2CDriver.cpp Sat Apr 13 13:37:29 2013 +0000 +++ b/I2CDriver.cpp Sun Apr 14 06:39:04 2013 +0000 @@ -1,102 +1,197 @@ #include "I2CDriver.h" #include "error.h" -#define ISR2DRV_SIG (1<<0); -#define DRV_USR_SIG (1<<1); +using namespace mbed; +using namespace rtos; -Channel I2CDriver::channels[2]= {0,0}; +#define ISR2DRV_SIG (1<<7) +#define DRV_USR_SIG (1<<6) + +const PinName I2CDriver::c_sdas[] = {p9,p28}; +const PinName I2CDriver::c_scls[] = {p10,p27}; + +I2CDriver::Channel* I2CDriver::s_channels[2] = {0,0}; void I2CDriver::channel_0_ISR() { - osSignalSet( channels[0].driver, ISR2DRV_SIG); - NVIC_DisableIRQ(I2C1_IRQn); //I2C_IRQn + osSignalSet( s_channels[0]->driver, ISR2DRV_SIG); + NVIC_DisableIRQ(I2C1_IRQn); } void I2CDriver::channel_1_ISR() { - osSignalSet( channels[1].driver, ISR2DRV_SIG); - NVIC_DisableIRQ(I2C2_IRQn); //I2C_IRQn + osSignalSet( s_channels[1]->driver, ISR2DRV_SIG); +#if defined(TARGET_LPC1768) || defined(TARGET_LPC2368) + NVIC_DisableIRQ(I2C2_IRQn); +#elif defined(TARGET_LPC11U24) + NVIC_DisableIRQ(I2C_IRQn); +#endif } -void I2CDriver::threadFun(void* const args) +void I2CDriver::threadFun(void const *args) { int channelIdx = (int)args; Channel channel; s_channels[channelIdx] = &channel; channel.driver = Thread::gettid(); - +#if defined(TARGET_LPC1768) || defined(TARGET_LPC2368) if(channelIdx==0)NVIC_SetVector(I2C1_IRQn, (uint32_t)I2CDriver::channel_0_ISR); if(channelIdx==1)NVIC_SetVector(I2C2_IRQn, (uint32_t)I2CDriver::channel_1_ISR); - - I2C i2c(c_sda[channelIdx], c_scl[channelIdx]); - +#elif defined(TARGET_LPC11U24) + NVIC_SetVector(I2C_IRQn, (uint32_t)I2CDriver::channel_1_ISR); +#endif + I2C i2c(c_sdas[channelIdx], c_scls[channelIdx]); + volatile Transfer& tr = channel.transfer; while(1) { - osSignalWait(DRV_USR_SIG,0); - switch(channels[channel].transfer.cmd) { + // wait for requests + osSignalWait(DRV_USR_SIG,osWaitForever); + // check and adapt frequency + if(channel.freq != tr.freq) { + channel.freq = tr.freq; + i2c.frequency(tr.freq); + } + // just doit + switch(tr.cmd) { case START: - if(channel.freq!=channel.transfer.freq) i2c.frequency i2c.start(); break; case STOP: i2c.stop(); break; + case READ: + tr.ret = i2c.read(tr.adr, tr.dta, tr.len, tr.rep); + break; + case READ_FROM_REGISTER: + tr.ret = i2c.write(tr.adr,(const char*)&(tr.reg), 1, true); + if(tr.ret)break; // error => bail out + tr.ret = i2c.read(tr.adr, tr.dta, tr.len, tr.rep); + break; + case READ_BYTE: + tr.ret = i2c.read(tr.ack); + break; + case WRITE: + tr.ret = i2c.write(tr.adr, tr.wdta, tr.len, tr.rep); + break; + case WRITE_BYTE: + tr.ret = i2c.write(tr.ack); + break; + default: + error("call 911"); } - s_threads[2] + // inform the caller + osSignalSet( channel.transfer.caller, DRV_USR_SIG); } } -I2CDriver::I2CDriver(PinName sda, PinName scl) +I2CDriver::I2CDriver(PinName sda, PinName scl):I2C(sda,scl) { // check pins and determine i2c channel int channel=0; - if(sda==sdas[0] && scl==scls[0]) channel=0; // I2C_1 - else if (sda==sdas[1] && scl==scls[1]) channel=1; //I2C_2 - else error("I2CDriver: Invalid I2C pinns selected"); - +#if defined(TARGET_LPC1768) || defined(TARGET_LPC2368) + if(sda==c_sdas[0] && scl==c_scls[0]) channel=0; // I2C_1 + else +#endif + if (sda==c_sdas[1] && scl==c_scls[1]) channel=1; //I2C_2 or I2C + else error("I2CDriver: Invalid I2C pinns selected"); if(s_channels[channel]==0) new Thread(threadFun,(void *)channel,osPriorityRealtime); - - m_channel = *(s_channel[channel]); + m_channel = s_channels[channel]; } -void I2CDriver::sendNwait(){ - osSignalSet( m_channel.driver, DRV_USR_SIG); + +void I2CDriver::sendNwait() +{ + m_channel->transfer.freq = _hz; + m_channel->transfer.caller = Thread::gettid(); + osSignalSet( m_channel->driver, DRV_USR_SIG); osSignalWait(DRV_USR_SIG,osWaitForever); } -void I2CDriver::frequency(int hz); - -int I2CDriver::read(int address, char *data, int length, bool repeated = false); - - -int I2CDriver::read(int ack); - - -int I2CDriver::write(int address, const char *data, int length, bool repeated = false); - - -int I2CDriver::write(int data); - - -void I2CDriver::start(void){ +int I2CDriver::read(int address, char *data, int length, bool repeated) +{ lock(); - m_channel.transfer.freq = _hz; - m_channel.transfer.cmd = START; + m_channel->transfer.cmd = READ; + m_channel->transfer.adr = address; + m_channel->transfer.dta = data; + m_channel->transfer.len = length; + m_channel->transfer.rep = repeated; sendNwait(); - unlock(); + int ret = m_channel->transfer.ret; + unlock(); + return ret; } -void I2CDriver::stop(void){ +int I2CDriver::read(int address, uint8_t regist, char *data, int length, bool repeated) +{ + lock(); + m_channel->transfer.cmd = READ_FROM_REGISTER; + m_channel->transfer.adr = address; + m_channel->transfer.reg = regist; + m_channel->transfer.dta = data; + m_channel->transfer.len = length; + m_channel->transfer.rep = repeated; + sendNwait(); + int ret = m_channel->transfer.ret; + unlock(); + return ret; +} + +int I2CDriver::read(int ack) +{ + lock(); + m_channel->transfer.cmd = READ_BYTE; + m_channel->transfer.ack = ack; + sendNwait(); + int ret = m_channel->transfer.ret; + unlock(); + return ret; +} + +int I2CDriver::write(int address, const char *data, int length, bool repeated) +{ lock(); - m_channel.transfer.freq = _hz; - m_channel.transfer.cmd = STOP; + m_channel->transfer.cmd = WRITE; + m_channel->transfer.adr = address; + m_channel->transfer.wdta = data; + m_channel->transfer.len = length; + m_channel->transfer.rep = repeated; + sendNwait(); + int ret = m_channel->transfer.ret; + unlock(); + return ret; +} + +int I2CDriver::write(int data) +{ + lock(); + m_channel->transfer.cmd = WRITE_BYTE; + m_channel->transfer.ack = data; sendNwait(); - unlock(); + int ret = m_channel->transfer.ret; + unlock(); + return ret; } + +void I2CDriver::start(void) +{ + lock(); + m_channel->transfer.cmd = START; + sendNwait(); + unlock(); +} + + +void I2CDriver::stop(void) +{ + lock(); + m_channel->transfer.cmd = STOP; + sendNwait(); + unlock(); +}