Open sslupsky opened 2 years ago
I found some other issues with this driver.
Also, @galak, the new DT_HAS property in the KConfig file causes conflicts when using an out-of-tree driver. That is, CMake will compile both the in-tree and out-of-tree drivers and then the linker will fail because of multiple references to the same __device_dts_ord_XXX variable.
Sorry for the late reply, I've been on vacation on the other side of the planet away from home for the whole month of August
@XenuIsWatching I understand you are the code owner for this driver? I have some code for handling the alert pin in a separate repo if you want to have a look at that. I recall I used some code from the lmp90xxx driver. https://github.com/sslupsky/zephyr-out-of-tree/tree/master/drivers/adc
The reason why I didn't implement the ALERT pin was in the case where I was using it, I was IO limited and didn't have the alert pin connected to the microprocessor I was using.... so it was just left floating Pull requests are welcome tho...
- The binding conflicts with another in tree driver: ADS1119
Please give me a link to the line where it is conflicting because I don't see it
- The driver does not support multiple channels
Pull requests are welcome
@XenuIsWatching I understand you are the code owner for this driver? I have some code for handling the alert pin in a separate repo if you want to have a look at that. I recall I used some code from the lmp90xxx driver. https://github.com/sslupsky/zephyr-out-of-tree/tree/master/drivers/adc
Thanks for sending me that, I did take a look at and while reviewing your code and rereading the datasheet. I noticed an issue with the ref_interal and GAIN that would cause an issue with the ads1x13. The ads1113 does have a PGA so it's ref_internal can only be 2048... I'll make a pull request fixing that sometime this week...
I'll make a pull request fixing that sometime this week...
Hi @anangl,
This issue, marked as an Enhancement, was opened a while ago and did not get any traction. It was just assigned to you based on the labels. If you don't consider yourself the right person to address this issue, please re-assing it to the right person.
Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.
@sslupsky you are also encouraged to help moving this issue forward by providing additional information and confirming this request/issue is still relevant to you.
Thanks!
Is your enhancement proposal related to a problem? Please describe. The device bindings and driver do not have allow configuration of the ALERT pin. For instance, when the pin is connected to the mcu and signals an interrupt.
Describe the solution you'd like Add something like the following to the binding (the lmp90xxx binding does something like this) and add support in the driver:
@XenuIsWatching I understand you are the code owner for this driver? I have some code for handling the alert pin in a separate repo if you want to have a look at that. I recall I used some code from the lmp90xxx driver. https://github.com/sslupsky/zephyr-out-of-tree/tree/master/drivers/adc