libmaple i2c API « LeafLabs Garden

LeafLabs Garden » Development

libmaple i2c API

(97 posts) (15 voices)

Started 5 years ago by bnewbold
Latest reply from 4rdu1n0

« Previous 1…456 7 Next »

redfox74
Member

Hi Anton,
thank's for your post .. I check the last branch of i2c code Perry and the other people of leaflabs are doing a good work. I had patch some routine because i try to use i2c1 and i2c2 ,but all it work fine , the high density need some small adjustment,but after that all is ok.
Perry where i can send my patch for you ? So you can check it.
I saw i2c work fine with my oscilloscope , I not yet start to test directly on the sensor but i think that it could be work.
What doing the patch on wire lib that you told in the past post ?

Are you news on AeroQuad firmware ? I'm support the development of that software i help Ted in his work on matrixtable for develop hexa , Okto copter , and some other functionlity.
My old board MP1.0 was fully supported by AQ project .
Check here : http://www.virtualrobotix.com are you interest to help us to port AQ on ARM ?
Best
Roberto

Posted 4 years ago #
anton
Member

Roberto, patching Wire lib looks trivial except for the need to take a decision on how hardware and soft realisation will coexist. In my opinion software lib should be kept as stm32 i2c likes to conflict with other periphery.

I have ported AQ with Wii sensors. But it wont to fly. It looks like high priority i2c interrupts hide pin state change ones so reciever signal reads unreliable.

Posted 4 years ago #
redfox74
Member

Dear Friends,
i'm trying to use the i2c lib , the software revision wire.h and the hardware implementation , but with the first one wire.h i have problem to reading right value. But work better of i2c.h implementation. I read temp 118.002344 and it change if i touch the sensor and pressure is 1500.xx instead of 900-1000 ?!
I try to read the value from pressure sensor BMP085 .
The i2c version instead read 7f value. I try it on maple v5 .

This is my implementation of i2c function , my first answer is i write a right implementation for i2c.h api ? I know that is totaly different respect of wire.h . I hope to understand right implementation.
Could you help me .. i became crazy i hate i2c bus :(

void BMP085::writemem(byte _addr, byte _val) {
#ifndef WIRE
uint8 buff[1];
buff[0]=_val;
msgs[0].addr = _addr;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = buff;
i2c_master_xfer(I2C2, msgs, 1);
#else
Wire.beginTransmission(_dev_address); // start transmission to device
Wire.send(_addr); // send register address
Wire.send(_val); // send value to write
Wire.endTransmission(); // end transmission
#endif
}

void BMP085::readmem(byte _addr, int _nbytes, byte __buff[]) {
#ifndef WIRE
int i;
/* Repeated start condition */

msgs[0].addr = _addr;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = 0x0;

msgs[1].addr = _addr;
msgs[1].flags = I2C_MSG_READ;
msgs[1].length = _nbytes;
msgs[1].data = __buff;

i2c_master_xfer(I2C2, msgs, 2);

Serial1.print("nbyte:");
Serial1.println(_nbytes);
for (i=0;i<=_nbytes;i++)
{
Serial1.print((int)__buff[i]);
Serial1.print(".");
}
Serial1.println("");

#else

Wire.beginTransmission(_dev_address); // start transmission to device
Wire.send(_addr); // sends register address to read from
Wire.endTransmission(); // end transmission
Wire.beginTransmission(_dev_address); // start transmission to device
Wire.requestFrom(_dev_address, _nbytes);// send data n-bytes read

byte i = 0;
while (Wire.available()) {
__buff[i] = Wire.receive(); // receive DATA
i++;
}
Wire.endTransmission(); // end transmission
#endif
}

Posted 4 years ago #

anton
Member

Roberto, try it:

uint8 buff[2];
buff[0]=_addr;
buff[1]=_val;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 2;
msgs[0].data = buff;
i2c_master_xfer(I2C2, msgs, 1);

and use oscilloscope or logic analiser

Posted 4 years ago #

anton
Member

Like this: http://cgi.ebay.com/CY7C68013A-USB-Logic-Analyzer-Core-Board-Schematics-/150576521703 with USBeeAX or Saleae firmware

Posted 4 years ago #
redfox74
Member

Hi Anton,
thank you very much for your suggestion .. I saw a big mistake on my inmplementation , because one is address of device _dev_address and the other is the address inside the device _addr . I doing this port at 3 am of this night :)
This is my new implementation ... what do you think ?
For readmem ... write is bit simple :)
best
Roberto

void BMP085::writemem(byte _addr, byte _val) {
#ifndef WIRE
uint8 buff[2];
buff[0]=_addr;
buff[1]=_val;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 2;
msgs[0].data = buff;
i2c_master_xfer(I2C2, msgs, 1);

/*
uint8 buff[1];
buff[0]=_val;
msgs[0].addr = _addr;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = buff;
i2c_master_xfer(I2C2, msgs, 1);
*/

#else
Wire.beginTransmission(_dev_address); // start transmission to device
Wire.send(_addr); // send register address
Wire.send(_val); // send value to write
Wire.endTransmission(); // end transmission
#endif
}

void BMP085::readmem(byte _addr, int _nbytes, byte __buff[]) {
#ifndef WIRE
int i;
/* Repeated start condition */
uint8 ibuff[2];

ibuff[0]=_addr;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = ibuff;
i2c_master_xfer(I2C2, msgs, 1);

__buff[0]=_addr;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = 0x0;

msgs[1].addr = _dev_address;
msgs[1].flags = I2C_MSG_READ;
msgs[1].length = _nbytes;
msgs[1].data = __buff;

i2c_master_xfer(I2C2, msgs, 2);

Serial1.print("nbyte:");
Serial1.println(_nbytes);
for (i=0;i<=_nbytes;i++)
{
Serial1.print((int)__buff[i]);
Serial1.print(".");
}
Serial1.println("");

#else

Wire.beginTransmission(_dev_address); // start transmission to device
Wire.send(_addr); // sends register address to read from
Wire.endTransmission(); // end transmission
Wire.beginTransmission(_dev_address); // start transmission to device
Wire.requestFrom(_dev_address, _nbytes);// send data n-bytes read

byte i = 0;
while (Wire.available()) {
__buff[i] = Wire.receive(); // receive DATA
i++;
}
Wire.endTransmission(); // end transmission
#endif
}

Now I try on device :)

Posted 4 years ago #
redfox74
Member

Hi Anton,
i doing some test , but the firmware freeze :(
it's strange because with the wire.h i read the data without any kind of frezee :(
Best
Roberto

Posted 4 years ago #
redfox74
Member

I saw that in the i2c implementation there're a lot of while(1) without the timeout exit could be that the reason of firmware freeze.

Posted 4 years ago #
redfox74
Member

I doing other test the hardware i2c confirm that freeze the wire.h instead the temperature when i put my hand on sensor go down instead of go up ... some suggestions ?
Same code on avr644p work fine ..
Best
Roberto

Posted 4 years ago #

anton
Member

void BMP085::readmem(byte _addr, int _nbytes, byte __buff[]) {
#ifndef WIRE
int i;
/* Repeated start condition */
uint8 ibuff[2];

ibuff[0]=_addr;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = ibuff;

msgs[1].addr = _dev_address;
msgs[1].flags = I2C_MSG_READ;
msgs[1].length = _nbytes;
msgs[1].data = __buff;

i2c_master_xfer(I2C2, msgs, 2);

Posted 4 years ago #

anton
Member

And add this string in i2c.c

/*
* Master transmitter: write first byte to fill shift register.
* We should get another TXE interrupt immediately to fill DR again.
*/
if (msg->length != 1)

Posted 4 years ago #

redfox74
Member

Hi Anton,
thank you very much , in the last update i yet change the code as your suggestions .. I forget to put your last patch .. now i put it and the lib work .
Now i read the data from barometer.
I have only two problem now :
If i deactivate debug , the bus freeze again .
Secondo problem is about the data , i see the same value that i saw with software i2c implementation , so i think that the problem is the conversion of value retrive by barometer and not the lib that retrive the data ... the code is the same that i use on avr micro , but so i think that there're some different in type avr is 8 bit instead the STM32 is a 32 bit x example.
Best
Roberto
Thanks again...

Posted 4 years ago #
redfox74
Member

So I doing some test ... if i mantain the debug string i havent problem , if i delete it the program immediatly freeze.
If i put a delay after a write of 1 millisecond it restart to work .
Some suggestions ?

void BMP085::readmem(byte _addr, int _nbytes, byte __buff[]) {
/* Repeated start condition */
/*
ibuff[0]=_addr;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = ibuff;
i2c_master_xfer(I2C2, msgs, 1);
*/
#ifndef WIRE
int i;
/* Repeated start condition */
uint8 ibuff[2];

ibuff[0]=_addr;
msgs[0].addr = _dev_address;
msgs[0].flags = 0;
msgs[0].length = 1;
msgs[0].data = ibuff;

msgs[1].addr = _dev_address;
msgs[1].flags = I2C_MSG_READ;
msgs[1].length = _nbytes;
msgs[1].data = __buff;

i2c_master_xfer(I2C2, msgs, 2);
//I2C_DELAY2;
delay(1); // WITH THIS WORK FINE WITHOUT FREEZE
/*
Serial1.print("nbyte:"); WIT THIS WORK FINE WITOUT FREEZE
Serial1.println(_nbytes);
for (i=0;i<=_nbytes;i++)
{
Serial1.print((int)__buff[i]);
Serial1.print(".");
}
Serial1.println("");
*/

Best
Roberto

Posted 4 years ago #
mbolivar
LeafLabs

Roberto,

Thanks for your continuing work on this!

In answer to your question from a while ago:

Perry where i can send my patch for you ? So you can check it.

You should know that Perry has a day job, and doesn't have much time to read the forums. If you'd like to contact him directly with a patch, the best thing to do is to clone libmaple on github, and then make a pull request.

Posted 4 years ago #
anton
Member

Roberto, have you compared raw byte data both on arduino and maple?
Delays may be required by the sensor.

Posted 4 years ago #

RSS feed for this topic

« Previous 1…456 7 Next »

Reply »

You must log in to post.