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.

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

Support bulk read feature implemented in linux driver #115

Open
CJKohler opened this issue Jan 4, 2023 · 4 comments
Open

Support bulk read feature implemented in linux driver #115

CJKohler opened this issue Jan 4, 2023 · 4 comments

Comments

@CJKohler
Copy link

CJKohler commented Jan 4, 2023

It looks like the W1_Therm driver now supports bulk-read, which sends one command to all devices on the bus to read and convert the temperature. These can then be quickly read-out, without having to wait 750 msec for each sensor.

It looks like Bulk Read support was added in May 2020 in the linux driver:
https://github.com/raspberrypi/linux/blob/rpi-5.15.y/drivers/w1/slaves/w1_therm.c

trigger_bulk_read() - function to trigger a bulk read on the bus

This post suggests you need to write "trigger\n" to the therm_bulk_read file

Any chance you could add this to the w1thermsensor package?

@timofurrer
Copy link
Owner

@CJKohler looks indeed like a nice addition. Unfortunately, I don't have enough spare time right now to work on this, but would be more than happy to review a PR and get this feature released!

@then3rd
Copy link

then3rd commented Aug 9, 2023

I'd love to see this implemented. I would love to help, however it's a bit over my skillet at this time.

@h-wissing
Copy link

Hello,
I made that post in RPi-forum because I did not get a reply here from Mr. Furrer. I also addressed another issue. When using DS 2482-800 8-channel I2C to W1 converter only the first channel has the bulk read file / ability. Another problem is, that for initiation - writing "trigger /n" to the file you need supervisor rights or have to change them in bevorehand. All ohter commands can be used by standard user.

Regards

@timofurrer
Copy link
Owner

I still don't have the bandwidth to implement this. However, we are open for contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants