linguini1 commented on PR #17161: URL: https://github.com/apache/nuttx/pull/17161#issuecomment-3392366731
> I would suggest to switch this implementation to a lowerhalf netdev before it gets merged, else we will just add more work for that transition to happen. This makes sense, and I'll also wait to see what others say. However, I always feel bad for contributors in these cases. Not long ago I was first told that sensor drivers should all be implemented with uORB going forward. Someone I know was told that GNSS drivers should use the GNSS lower half, which was undocumented at the time. I wonder if there's somewhere very prominent we can put these "requirements" so contributors are aware. I know they're encouraged to ask the mailing list before adding new features but that creates more work for everyone. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
