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

Add support for LM75AD I2C Temperature Sensor #2909

Merged
merged 2 commits into from
Jun 5, 2018
Merged

Add support for LM75AD I2C Temperature Sensor #2909

merged 2 commits into from
Jun 5, 2018

Conversation

andrethomas
Copy link
Contributor

Add support for LM75AD I2C Temperature Sensor

Add support for LM75AD I2C Temperature Sensor
@arendst arendst added the enhancement Type - Enhancement that will be worked on label Jun 5, 2018
@arendst
Copy link
Owner

arendst commented Jun 5, 2018

Will be implemented but...

Needs Celsius/Fahrenheit as others do. Simple call but not in the current PR. See other temperature drivers for implementations.

@andrethomas
Copy link
Contributor Author

Damn, can't believe I forgot about the Fahrenheit conversion :|

Update xdrv_26_lm75ad.ino to support C or F depending on Settings.flag.temperature_conversion (#define TEMP_CONVERSION 0 or 1) as stored in [SetOption8] Return temperature in (0 = Celsius or 1 = Fahrenheit)
@andrethomas
Copy link
Contributor Author

Tested update... let me know :)

@arendst arendst merged commit 006a747 into arendst:development Jun 5, 2018
arendst added a commit that referenced this pull request Jun 5, 2018
6.0.0a
* Add support for I2C temperature sensor LM75AD (#2909)
@andrethomas
Copy link
Contributor Author

curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 7, 2018
Add support for LM75AD I2C Temperature Sensor
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 7, 2018
6.0.0a
* Add support for I2C temperature sensor LM75AD (arendst#2909)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type - Enhancement that will be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants