[GitHub] [incubator-nuttx] Donny9 commented on pull request #2039: driver/sensor: add unified management for sensor

[GitBox <gi...@apache.org>]

Donny9 commented on pull request #2039:
URL: https://github.com/apache/incubator-nuttx/pull/2039#issuecomment-712552641


   > 
   > 
   > Interesting. How do you envision this being used?
   > Would a sensor using this interface be supported upstream or do you intend to have this as an option to internally support sensors via this interface?
   > Would this require porting currently supported sensors to this interface? I wonder if using a common interface would not become restrictive in order to provide the best support for a sensor.
   
   1. I will provide a driver of wtgahrs2 as a example to description this driver model using.
   2.There are 58 types of drivers in current nuttx version. It's impossible for me to transplant all the sensor into this model, because i don't have so many sensor devices and i can't verify them. Currently, all the sensor driver of nuttx are chaotic, They are not unified ioctl cmd, type, event data structure. So, This will cause the upper layer of the application to rely heavily on the sensor driver implementation.
   3.This model of i submitted will not limit the use of the sensor driver before. I hope everyone will use the model in the future to uniformly manage  all sensors, such as linux, rtt and so on. 


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

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