ppisa commented on PR #16491: URL: https://github.com/apache/nuttx/pull/16491#issuecomment-2976788682
> > On the other hand all i still quite complex code. It can be simplified if the power of two are only limits allowed (in he fact +0x4000 and -0x4000) as I have proposed #15319. There is problem that limits cannot be probably defined to cause only the event and clear the counter, so correct solution would need compare events and that way block their use for something useful. > > Couldn't these limits be set dynamically? One possible solution would be to create a custom `ioctl` to set them (checking for their boundaries, of course). > > On the other hand all i still quite complex code. It can be simplified if the power of two are only limits allowed (in he fact +0x4000 and -0x4000) as I have proposed #15319. There is problem that limits cannot be probably defined to cause only the event and clear the counter, so correct solution would need compare events and that way block their use for something useful. > > Couldn't these limits be set dynamically? One possible solution would be to create a custom `ioctl` to set them (checking for their boundaries, of course). It can be added value for some applications. On the other hand, when accumulation into 32-bits is demanded then maximal value suitable for processing and overflow detection is the best choice because it lowers number of interrupts and related system load and extends maximal allowed events handling latency which is guaranteed to not cause position lost due multiple or undetected overflow. -- 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
