btashton commented on pull request #2039: URL: https://github.com/apache/incubator-nuttx/pull/2039#issuecomment-714225863
I'm tempted to merge this with the understanding that this driver is experimental and we are not locking in the interfaces until the next release. If we did that I would expect that we work to make sure we resolve some of these questions. I think it is important that we consider the cost of turning this in terms of ram and flash for smaller platforms (which also may be lower power) which I think is part of what @v01d is getting at as well. Also there is the sensor cluster interface that I think was intended to solve this same problem. https://github.com/apache/incubator-nuttx/blob/2956b8516baf30c4099bd35c00919ae5c2cc973c/drivers/sensors/README.txt#L49 I dont think it makes a lot of sense to be supporting three different classes of sensor drivers, so hopefully we can get down to one or two. ---------------------------------------------------------------- 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: us...@infra.apache.org
