Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-05 Thread via GitHub
xiaoxiang781216 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2007357442 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. *

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-05 Thread via GitHub
keever50 commented on PR #16023: URL: https://github.com/apache/nuttx/pull/16023#issuecomment-2737695968 I will squash too -- 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. T

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-05 Thread via GitHub
keever50 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2005199938 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. /

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-05 Thread via GitHub
keever50 commented on PR #16024: URL: https://github.com/apache/nuttx/pull/16024#issuecomment-2739784987 > Please remember to fill in the Commit Message. Just copy from the PR Summary above. Thanks :-) > > * [8696cd8](https://github.com/apache/nuttx/commit/8696cd8f0a52351054594961b6d

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-05 Thread via GitHub
keever50 commented on PR #16023: URL: https://github.com/apache/nuttx/pull/16023#issuecomment-2737710461 huh looks like this PR did not go completely as planned. It is also bringing in "[Merge remote-tracking branch 'upstream/master']" I think i have to redo this PR -- This is an automat

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-04 Thread via GitHub
keever50 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2005192904 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. /

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-04-04 Thread via GitHub
xiaoxiang781216 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2007357442 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. *

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-23 Thread via GitHub
xiaoxiang781216 merged PR #16024: URL: https://github.com/apache/nuttx/pull/16024 -- 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...@nu

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-22 Thread via GitHub
keever50 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2008752782 ## Documentation/components/drivers/character/wireless/index.rst: ## @@ -6,3 +6,34 @@ Wireless character drivers :maxdepth: 1 lpwan/index.rst + +IOCTL Interfac

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-22 Thread via GitHub
keever50 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2007083751 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. /

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-21 Thread via GitHub
raiden00pl commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2008689091 ## Documentation/components/drivers/character/wireless/index.rst: ## @@ -6,3 +6,34 @@ Wireless character drivers :maxdepth: 1 lpwan/index.rst + +IOCTL Interf

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-20 Thread via GitHub
hartmannathan commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2006472694 ## Documentation/components/drivers/character/wireless/index.rst: ## @@ -6,3 +6,34 @@ Wireless character drivers :maxdepth: 1 lpwan/index.rst + +IOCTL Int

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-20 Thread via GitHub
xiaoxiang781216 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2006712477 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. *

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-20 Thread via GitHub
keever50 commented on PR #16024: URL: https://github.com/apache/nuttx/pull/16024#issuecomment-2741746752 Thank you for the spelling checks. These are now corrected -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-20 Thread via GitHub
lupyuen commented on PR #16024: URL: https://github.com/apache/nuttx/pull/16024#issuecomment-2739800941 Yep we just updated the guidelines here :-) https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
xiaoxiang781216 commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2004843090 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. *

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
lupyuen commented on PR #16024: URL: https://github.com/apache/nuttx/pull/16024#issuecomment-2738829040 Please remember to fill in the Commit Message. Just copy from the PR Summary above. Thanks :-) - https://github.com/apache/nuttx/pull/16024/commits/8696cd8f0a52351054594961b6d76c051ad5

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
lupyuen commented on code in PR #16024: URL: https://github.com/apache/nuttx/pull/16024#discussion_r2004628671 ## include/nuttx/wireless/ioctl.h: ## @@ -48,67 +48,214 @@ * the open() interface. / -

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
keever50 commented on PR #16024: URL: https://github.com/apache/nuttx/pull/16024#issuecomment-2737892731 will fix the conflict. That error did not pop up for me. Ill make sure to clean build double check everything. -- This is an automated message from the Apache Git Service. To respo

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
keever50 closed pull request #16023: nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies URL: https://github.com/apache/nuttx/pull/16023 -- 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 g

Re: [PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
nuttxpr commented on PR #16024: URL: https://github.com/apache/nuttx/pull/16024#issuecomment-2737758343 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR generally meets the NuttX requirements, bu

[PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
keever50 opened a new pull request, #16024: URL: https://github.com/apache/nuttx/pull/16024 ### Summary This PR is a follow-up of https://github.com/apache/nuttx/issues/15856 and the mailing list proposal "Proposal: Common IOCTL API for RF Modulation Technologies" Currently, t

[PR] nuttx/wireless/ioctl: Common IOCTL API for RF Modulation Technologies [nuttx]

2025-03-19 Thread via GitHub
keever50 opened a new pull request, #16023: URL: https://github.com/apache/nuttx/pull/16023 **### Summary** This PR is a follow-up of https://github.com/apache/nuttx/issues/15856 and the mailing list proposal "Proposal: Common IOCTL API for RF Modulation Technologies" Currentl