esp-idf-hal
esp-idf-hal copied to clipboard
esp-rs
Add support for ESP internal temperature sensor
What the title says. Currently very much WIP. PR created to help give feedback.
Closes #333
The interesting point is when you start implement the callback functions provided by the driver. It should allow you to setup a callback on certain temperature condition. In our crate terminology we call it a subscribe, if you want to peak in other drivers how similar things are handled.
since it seams to work with the flags we are now using in esp-idf-sys. We could also change it here. Rustcompiler give us the flag as esp_idf_soc_temp_sensor_supported.
If you are unsure how to proceed now to get the rest of the exposed api integrated, we can wrap it up here and i can integrate the rest ontop of it, or if you feel advantageous we can go deeper getting the callbacks to work with the api. I am not sure whats your background so i leave it up to you.
since it seams to work with the flags we are now using in esp-idf-sys. We could also change it here. Rustcompiler give us the flag as
esp_idf_soc_temp_sensor_supported.
Would that then be #[cfg(esp_idf_soc_temp_sensor_supported)]?
If you are unsure how to proceed now to get the rest of the exposed api integrated, we can wrap it up here and i can integrate the rest ontop of it, or if you feel advantageous we can go deeper getting the callbacks to work with the api. I am not sure whats your background so i leave it up to you.
If you don't mind me asking questions and it perhaps taking a bit longer then if you would do it yourself, I would like to try it. If I find out it's too much work we can still only merge my basic support in the future.
Is there any further progress on this? I’m having problems with a chip overheating and would like to quantify the problem better. Usable basic support would be helpful, if other work is paused. I’d be happy to help out, although I’m new enough to this stuff I don’t know how much use I’d be.
