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

Drivers: add MAX31855 thermocouple-to-digital converter #20741

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

leandrolanzieri
Copy link
Contributor

Contribution description

This adds the driver for the MAX31855 thermocouple-to-digital converter. This PR also adds the SAUL adaption, to be able to read the thermocouple and internal temperatures.

The device The performs cold-junction compensation and digitizes the signal from a K-, J-, N-, T-, S-, R-, or E-type thermocouple (depending on the version of your chip). More info in the datasheet.

Testing procedure

  • You can try the tests/drivers/saul application and add max31855 to the module list:
    Screenshot from 2024-06-11 11-16-45

  • You can try the tests/drivers/max31855 application:
    Screenshot from 2024-06-11 11-17-24

Issues/PRs references

None

@github-actions github-actions bot added Area: tests Area: tests and testing framework Area: drivers Area: Device drivers Area: SAUL Area: Sensor/Actuator Uber Layer labels Jun 11, 2024
@leandrolanzieri leandrolanzieri added the Type: new feature The issue requests / The PR implemements a new feature for RIOT label Jun 11, 2024
@leandrolanzieri leandrolanzieri force-pushed the pr/driver/max31855 branch 2 times, most recently from 40335de to 2cf30eb Compare June 11, 2024 09:49
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/max31855/max31855.c Outdated Show resolved Hide resolved
drivers/include/max31855.h Show resolved Hide resolved
drivers/max31855/max31855.c Show resolved Hide resolved
drivers/max31855/max31855_saul.c Show resolved Hide resolved
Copy link
Contributor

@Teufelchen1 Teufelchen1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes! I think that's as good as it gets for this little driver.

@leandrolanzieri leandrolanzieri added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jun 12, 2024
@riot-ci
Copy link

riot-ci commented Jun 12, 2024

Murdock results

✔️ PASSED

b092f95 tests/drivers: add max31855 test application

Success Failures Total Runtime
10176 0 10177 14m:07s

Artifacts

@leandrolanzieri leandrolanzieri added this pull request to the merge queue Jun 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 12, 2024
@leandrolanzieri
Copy link
Contributor Author

Added Makefile.ci to the test

Merged via the queue into RIOT-OS:master with commit 755f931 Jun 13, 2024
25 checks passed
@leandrolanzieri leandrolanzieri deleted the pr/driver/max31855 branch June 13, 2024 13:07
@mguetschow mguetschow added this to the Release 2024.07 milestone Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: drivers Area: Device drivers Area: SAUL Area: Sensor/Actuator Uber Layer Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants