nmaggioni opened a new pull request, #16842:
URL: https://github.com/apache/nuttx/pull/16842

   ## Summary
   
   Enabling a higher channel of the internal ADC had the effect of initializing 
the lower ones as well. Now that happens only if actively requested.
   
   Also, the functions for handling the internal ADC did not follow the typical 
naming used by comparable modules for the same arch and were renamed for 
coherence. Informational logging calls were also made slightly more useful and 
discernible in case of having multiple ADCs.
   
   ## Impact
   
   Config options seem to already prevent most edge cases. As I see it, this 
change could only cause troubles in case somebody did not explicitly enable a 
lower ADC channel yet manually interfaced with it in the board-level code.
   
   ## Testing
   
   These changes were tested on a Linux host, targeting a custom Cortex-M0+ 
(RP2040) board using NuttX v12.8.0 as the baseline. They were then manually 
ported to the current master branch.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to