xiaoxiang781216 commented on pull request #5874:
URL: https://github.com/apache/incubator-nuttx/pull/5874#issuecomment-1080353361


   > I have a general question. `NPOLLWAITERS` seems to be a more generic 
approach all over the code and buttons seems to be a partial case. Maybe it is 
more reasonable to set `INPUT_BUTTONS_NPOLLWAITERS` default value to `1` 
instead of removing it completely?
   
   The problem is that the notification is unreliable. Since we just have one 
bo_pending per file handle and btn_read will clear bo_pending, the second 
instance may lose the event.
   
   > Or you are going to iterate the code and apply similar change to other 
files?
   
   touch and keyboard just support one poller per file handler, I will provide 
patch for ajoystack and djoystack.


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