You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The bipolar-mask property could be dropped in favor of the channel configuration be me more compliant to the ADC api
Since AD7124 has two programmable excitation current sources, we could add two properties to set the current sources values (can be different) and enable the ADC_CONFIGURABLE_EXCITATION_CURRENT_SOURCE_PIN config
AD7124 provides two GPIO output pins, it could be useful to have a GPIO driver.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Hi everyone. Writing some code to read a PT100 thermocouple via an AD7124 ADC, I was digging in the driver code and made some discoveries:
The
zephyr,differential
property on channels is ignored. Bipolarity can be set using thebipolar-mask
property on the device nodezephyr/drivers/adc/adc_ad7124.c
Line 418 in 74056a7
The
bipolar-mask
property could be dropped in favor of the channel configuration be me more compliant to the ADC apiSince AD7124 has two programmable excitation current sources, we could add two properties to set the current sources values (can be different) and enable the ADC_CONFIGURABLE_EXCITATION_CURRENT_SOURCE_PIN config
AD7124 provides two GPIO output pins, it could be useful to have a GPIO driver.
If it sounds reasonable, i could take in charge.
Beta Was this translation helpful? Give feedback.
All reactions