[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2039: driver/sensor: add unified management for sensor

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

xiaoxiang781216 edited a comment on pull request #2039:
URL: https://github.com/apache/incubator-nuttx/pull/2039#issuecomment-712641292


   Yes, it could be added very easily with the current implementation. To simplify the design, we have two concept about the buffer size:
   
   1. Driver can control the buffer size in no batch mode by sensor_lowerhalf_s::buffer_bytes
   2. Application can control through ioctl e.g. SIOC_SET_BUFFER_SIZE(of course, another patch)
   3. sensor upper will extend the buffer as need in batch mode to accumulate the bulk data
   
   The first two items let user decide how much the jitter or latency should be used, the last item avoid the driver writer involve the complex buffer management.


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