keever50 commented on PR #15828: URL: https://github.com/apache/nuttx/pull/15828#issuecomment-2695711310
> ### > [Mar 3, 2025](#issuecomment-2695612211) Hmm. I thought adding more to what has to be removed might deliver extra work. However, some commands like set radio frequency dont have to be changed. Some things can stay. maybe. My strategy is to 1: add experimental drivers, 2: add common API, 3: port drivers and then 4: remove legacy. Slowly. When we want to rework the entire complete thing at once, we need to rework everything connected to the wireless ioctl.h For example, the NRF24l01 is also using these commands. They would all require testing before PR can be accepted. It would be very nice to be able to work them one by one without breaking them or porting them all at once. We can take our time with these. So we have a legacy period of extra commands that will slowly go away. But i think as long the already established common ones stay completely untouched and your solution doesn't get away in this process, its fine. I think just moving them a bit down under a comment also is completely OK. I'm really happy that people start to add LoRa to NuttX though! I can really really use these features. Thank you for doing this -- 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
