xiaoxiang781216 commented on PR #16746:
URL: https://github.com/apache/nuttx/pull/16746#issuecomment-3090061184

   > > @kywwilson11 please follow the PR standard to make it easier to 
categorize it in the next release: arch/arm/stm32h5: Initial Driver for STM32H5 
Digital Temperature Sensor (DTS)
   > > Is it possible to use the existing test sensor application 
(nuttx-apps/system/sensortest) ?
   > > If sensortest doesn't work with it, please include this example 
application for testing to avoid hidden feature. Please also update the STM32H5 
Documentation/ to include this feature.
   > 
   > My apologies. I will review the PR guidelines before my next pull request.
   > 
   > I also tried sensortest and it worked. Added output to the pull request. I 
was initially using the type sensor_type_ambient_temperature, which isn't 
supported by sensortest (I don't think). But sensor_type_temperature is 
actually more appropriate, so sensortest works. Thanks for the suggestion.
   
   sensortest may need update since many physical sensor types were added 
recently.


-- 
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