MPR121 i2c touch keypad with Wire library

There is a bit banged version of the wire lib, called SoftWire in the library

We recently swapped to using Hardwire but I don’t think all possible cases have been checked

Can you look in the Wire library and see what functionality is missing ?

Hi Roger,

I’ve just checked on Github, there are two types of endTransmission() function in WireBase. It seems those are handling the stop condition the same way as the AVR version of Wire lib:

uint8 WireBase::endTransmission(bool stop) {
uint8 retVal;
if (tx_buf_overflow) {
return EDATA;
}
retVal = process(stop);// Changed so that the return value from process is returned by this function see also the return line below
tx_buf_idx = 0;
tx_buf_overflow = false;
return retVal;//SUCCESS;
}

uint8 WireBase::endTransmission(){
endTransmission(true);
}

I switched the default Wire lib to Hardware I2C about a week ago.

Let me know whether the new version works for you

Hi Roger,

There are good and bad news. The good news is that Adafruit’s MPR121 lib can simply be compiled with latest STM32 Wire lib. There are no missing function calls or other compile errors. The bad is the communication is not working yet, the library cannot see the MPR121 chip on the i2c bus.

I took further investigations: tried to compile and run the Adafruit’s MPR121 test, my Wire lib MPR121 register-read test and my bit-banging MPR121 register-read test scketches on three different boards: Arduino Pro Mini with ATmega328P (powered from 3.3V), Arduino DUE with ATSAM3X8E and STM32 blue pill with STM32F103C8T6. The results are depressing: only the Pro Mini worked well with “factory” Wire library, even the DUE has faulted the tests. BTW all boards passed the bit-banging tests.

I have attached two small sketches to present the problem. Both sketches can be compiled to the three boards mentioned above and can be checked with a real MPR121 device.

(If I have time I will examine the STM32 datasheet to see if its i2c peripheral can handle this special “repeated start” condition.)

Any further idea?

do you have a logic analyser to check the bus speed ?

I think it should be set to the default, which is 100khz, but I have not verified that with a logic analyser as all the I2C devices I have seem to happily work up to and beyond 400k

Unfortunately I have no logic analyzer. I will think about inventing something to ensure the bus speed. But I am afraid it is really not the problem.

I took another try to use SoftWire library in my “Wire tes code” with success! Then I copied the Adafruit MPR121 library and modified it to use SoftWire instead of Wire resulting a good-working touch-button test on my blue pill.

But there is an interesting thing: browsing the source files of STM32 Wire (which is actually HardWire) library I cannot found anything in the transmit code that would care about the “stop” parameter of the endTransmission(bool) fuction. I see, that endTransmission() with no parameter calls endTransmission(bool) with “true”, but in the body of endTransmission(bool) I could not see anything regarding the given “stop” parameter. Maybe I missed something.
In the SoftWire files I found the code snipplets what do a “repeated start” condition instead of “stop” at the end of transmit if “stop” parameter is false, furthermore there is no “start” generation at the beginning of the receiving loop in this case.

As I looked into the STM32 “RM0008 Reference manual” I cannot sure that the hardware peripheral can generate a “repeated start” in 7-bit addressing mode. I saw mentioning “repeated start” condition only in contact of 10-bit addressing mode.

@szilvasyz

I forget who submitted the updated Hardware core that we now use, but I know there was a thread about it on the forum, so perhaps you could track down the person who wrote it and ask them about this, as I don’t have detailed knowledge of the hardware I2C on the STM32.

The alternative is to try using STM’s own official Arduino core for the F103 github.com/stm32duino , and see if that supports repeated start

https://github.com/stm32duino/Arduino_C … e/Wire.cpp
Which makes calls to STM’s HAL

But its possible to look in the HAL code to see what hardware registers its interacting with, and possibly see how to add the feature to libmaple if it exists

BTW.

I can recommend buying a cheap 8 channel logic analyser

There is an excellent video on YouTube https://www.youtube.com/watch?v=dobU-b0_L1I

You can get something working for under $10

e.g. search for “CY7C68013A Module Logic Analyzer”

https://www.ebay.com/sch/i.html?_odkw=C … r&_sacat=0

Or for virtually the same price you can get one in a box with a few cables

Search on “USB 24MHz 8CH Logic Analyzer”

e.g.

https://www.ebay.com/sch/i.html?_odkw=L … r&_sacat=0

These also seem compatible with the old Saleae PC “Logic” software, or can be used with the open source Sigrok Pulseview

@Roger,

I looked at the code of HAL-calling Wire lib you suggested, but I cannot see any trace of using the sendStop parameter (even it is marked az “UNUSED”):

uint8_t TwoWire::endTransmission(uint8_t sendStop)
{
UNUSED(sendStop);
int8_t ret = 4;

if (master == true) {
// transmit buffer (blocking)
switch(i2c_master_write(&_i2c, txAddress, txBuffer, txBufferLength))
{

If you have a github account, I would raise an issue for this on the stm32duino github account as fpSTM is quite proactive about these things, and as he works for STM, sometimes he can get support when things are missing from the HAL

Hi Roger,

Sorry for late answering, I’ve just finished the test code for generating “repeated start” with i2c hardware peripheral. As it can be seen the hardware is able to do it, only i2c libraries don’t realize the “stopless” transmission. I attahced the sketch I worked on to demonstrate the things. Although the sample code does not care about any errors during communication, it is working on my desk. I can read all registers of MPR121 correctly.

In the meantime I created a free Github account (szilvasyz), because I had none.

Do you think there is any chance to put handling of this “stopless transmission” case in the main Wire lib code?

Regards,
Zoltan

I don’t know that much about I2C, but I am happy to change the core as long as it does not break things for other users.

I am also happy to add new API functions if necessary to enable this mode

[RogerClark – Wed Aug 16, 2017 10:29 pm] –
I don’t know that much about I2C, but I am happy to change the core as long as it does not break things for other users.

I am also happy to add new API functions if necessary to enable this mode

Did you get any further with this Roger? Just wondering – bought a cheap MPR121 board from china for my midi sequencer project.
No worries if not, it can lurk in my spares tin

(Hope nobody minds me ressurecting this thread, it just seemed so directly tied to it, didnt seem sensible to start a new one).

Greetings,

There are now several I2C peripherals available needing the i2c RepeatStart option.

I’ve been trying to get a VEML6072 sensor working with BluePill and run into the same issue.
It works just fine with an Arduino UNO where the “endTransmission(false)” works by omitting the i2c_stop state.

Wonder whether anyone have succeeded in using/modifying the Wire library to implement RepeatStart?

Thanks much.

FYI,
I found “SoftWire” solved the problem. That library does include support for RepeatStart.

Please see my example for interfacing a VELM6075 uV sensor with a BluePlill.

Thanks.