keever50 commented on PR #15828:
URL: https://github.com/apache/nuttx/pull/15828#issuecomment-2695199224

   @linguini1 I see that you added the IOCTL commands, however i would suggest 
looking at https://github.com/apache/nuttx/issues/15856 maybe. The common API 
is still being worked on. I just started implementing the features into ioctl.h
   Maybe you can look at that? It is fine if you want to do it yourself too.
   The main difference between my common IOCTL and yours is i tagged LoRa 
related commands with WLIOC_LORA_x.
   This is done to keep some seperation between two obviously different command 
groups.
   Personally it feels "messy" when all modulation types are considered common.
   


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