Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

May 16 firmware: AIRPI i2c BMP085 and BMP180 temp and pressure sensors misread #599

Closed
g7ruh opened this issue May 20, 2014 · 9 comments
Closed

May 16 firmware: AIRPI i2c BMP085 and BMP180 temp and pressure sensors misread #599

g7ruh opened this issue May 20, 2014 · 9 comments

Comments

@g7ruh
Copy link

g7ruh commented May 20, 2014

May 16th firmware causes temp and pressure sensors to misread
Uploading pressure reading issue.PNG . . .
Uploading temp reading issue.PNG . . .

reverting to May 13th firmware fixes the problem. This is on 2 PIs here : https://xively.com/develop/a73wzKTeenrmwkbKnrZq and here https://xively.com/develop/5sS4LOiW-HpUN0kuPKG0 Note that the graphs show average so take time to "change". Viewing raw data shows fix works.

reposted here as requested by popcornmix
@popcornmix
Copy link
Collaborator

Looks to be caused by:
#318

@cbeytas can you help with this?

@popcornmix popcornmix mentioned this issue May 21, 2014
Merged
@cinhil
Copy link

cinhil commented May 21, 2014

Hello,
I also have i2c issues since 16 May firmware. Reverting to 13th May fixes the problem.
I have sometimes read and write problem on RTC clock ds1307. It fails in 10% of read/write test.

@cinhil
Copy link

cinhil commented May 25, 2014

Hello,
With a revert of "Perform I2C combined transactions when possible" commit, I do not have any problem with the RTC DS1307 anymore.
Regards,

@Swap-File
Copy link

I also have i2c issues with my Minimu9 http://www.pololu.com/product/1268 running Minimu9-ahrs https://github.com/DavidEGrayson/minimu9-ahrs since the May 16th firmware. I reverted to May 13th firmware and it fixed it.

@popcornmix
Copy link
Collaborator

The combined I2C transactions commit is now disabled by default.
Please run rpi-update and test.

@g7ruh
Copy link
Author

g7ruh commented May 27, 2014

Hi popcornmix,

updated and tested, all back to normal, many thanks for all your hard work making the pi GREAT (and others too)

Roger aka G7RUH

@popcornmix
Copy link
Collaborator

Exellent. Okay to close?

@Swap-File
Copy link

It's tested and working for me too. Thanks!

@g7ruh
Copy link
Author

g7ruh commented May 28, 2014

Thanks Swap-File too.
Popcornmix, fine to close.

@g7ruh g7ruh closed this as completed May 28, 2014
0day-ci pushed a commit to 0day-ci/linux that referenced this issue Sep 19, 2016
@notro
i2c: bcm2835: Add support for combined write-read transfer
b4ac87c 
Some SMBus protocols use Repeated Start Condition to switch from write
mode to read mode. Devices like MMA8451 won't work without it.

When downstream implemented support for this in i2c-bcm2708, it broke
support for some devices, so a module parameter was added and combined
transfer was disabled by default.
See raspberrypi/linux#599
It doesn't seem to have been any investigation into what the problem
really was. Later there was added a timeout on the polling loop.

One of the devices mentioned to partially stop working was DS1307.

I have run thousands of transfers to a DS1307 (rtc), MMA8451 (accel)
and AT24C32 (eeprom) in parallel without problems.

Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@popcornmix @Swap-File @g7ruh @cinhil