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

feat(Examples,MiscDrivers): Add ADT7420 I2C Sensor Driver and Example for MAX32655EVKIT #853

Merged

Conversation

selimgullulu
Copy link
Contributor

Description

This PR is introducing the driver and the example code for the ADT7420 Temperature Sensor for MAX32655EVKIT. It's tested using the EVAL-ADT7420MBZ with the MAX32655EVKIT. The code is mainly a modification of the I2C_Sensor example (which uses MAX31889). Please see the setup below and the serial output:

MAX32655_ADT7420

image

New example is introduced in folder:
Examples\MAX32655\I2C_Sensor_ADT7420

The misc drivers for the ADT7420 in folder:
Libraries\MiscDrivers\TempSensor\ADT7420

All files are checked and modified by cpplint and clang-format.

selimgullulu and others added 30 commits January 2, 2024 13:58
Initial version of the readme.md created
Made format changes
Added 
.cproject 
.project 
main.c 
and project.mk
populating the .vscode files
Uploaded the .vscode folder files:
flash.gdb
launch
README
settings
tasks
creating the .settings folder (blank language.settings)
updated c_cpp_properties.json
Created blank adt7420_driver.c file
uploaded adt7420_driver.c and adt7420_driver.h
added the .cproject from the I2Csensor
copied the .project from the vscode folder
Changed the I2C_Sensor to I2C_Sensor_ADT7420
got the hello_world .cprojet and renamed the content "Hello_World" to "I2C_Sensor_ADT7420"
got the content from Hello_World and modified the name
renamed from I2C_Sensor to I2C_Sensor_ADT7420
Added the serial output with:
The Device ID is: 0xCB
Removed unnecessary comments and added the Default I2C address comment for EVAL-ADT7420MBZ (0x49)
modified minor format & comment issues
serial output updated with the connection requirements
…ple, however it'll be required for advanced operations such as shutting down the sensor for low power.
No initialization necessary, replaced the word.
@github-actions github-actions bot added the MAX32655 Related to the MAX32655 (ME17) label Jan 3, 2024
@selimgullulu selimgullulu changed the title feat(Examples,PeriphDrivers): Add ADT7420 I2C Sensor Driver and Example for MAX32655EVKIT feat(Examples,MiscDrivers): Add ADT7420 I2C Sensor Driver and Example for MAX32655EVKIT Jan 3, 2024
Copy link
Contributor

@sihyung-maxim sihyung-maxim left a comment

Choose a reason for hiding this comment

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

Source code looks good, thanks for your contributions!

Examples/MAX32655/I2C_Sensor_ADT7420/main.c Outdated Show resolved Hide resolved
Libraries/MiscDrivers/TempSensor/ADT7420/adt7420_driver.c Outdated Show resolved Hide resolved
Libraries/MiscDrivers/TempSensor/ADT7420/adt7420_driver.h Outdated Show resolved Hide resolved
@sihyung-maxim
Copy link
Contributor

/clang-format-run

Updated the readme.md file to reflect the correct decimal temperature reading for the adt7420 sensor
Copy link
Contributor

@Jake-Carter Jake-Carter left a comment

Choose a reason for hiding this comment

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

Thanks @selimgullulu

@Jake-Carter Jake-Carter dismissed sihyung-maxim’s stale review January 9, 2024 21:25

Change requests completed

@Jake-Carter Jake-Carter merged commit 433d2e3 into analogdevicesinc:main Jan 9, 2024
8 checks passed
@selimgullulu
Copy link
Contributor Author

Thanks @selimgullulu

Thank you Jake

@selimgullulu selimgullulu deleted the add_ADT7420_sensor_driver branch January 10, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32655 Related to the MAX32655 (ME17)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants