TimJTi commented on PR #8789:
URL: https://github.com/apache/nuttx/pull/8789#issuecomment-1464158020

   > > @TimJTi please make sure it is compatible with the new sensor subsystem, 
test it with apps/testing/sensortest. I didn't see sensor_lowerhalf_s in our 
code.
   > 
   > Umm...didn't know about that. I'll take a look.
   
   @acassis 
   
   Proximity: not really appropriate to this sensor as it does not provide 
meaningful data in cm (or any measure). It is more of a "near/close" sensor. 
Typical use is mobile phone to detect that the phone is close to your ear.
   
   Light: The use of this sensor is not to provide 
periodic/accurate/timestamped precision ambient light measurements.
   
   There are other sensors that do not "conform" to this methodology, and the 
sensor_test app has very poor help and no documentation.
   
   Perhaps I COULD revisit this at a later date once the "new" sensor system is 
properly bedded in?


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to