RIOT icon indicating copy to clipboard operation
RIOT copied to clipboard

{dev,net}/ieee802154_*: add initial Kconfig modeling

Open jia200x opened this issue 4 years ago • 8 comments

Contribution description

This PR add the initial Kconfig modeling for the lower layers of IEEE 802.15.4. This includes:

  • Adding feature symbols to indicate presence of Radio HAL or Netdev Driver based driver.
  • Modeling of SubMAC and netdev_ieee802154_submac
  • Adding tests/ieee802154_submac to the list of Kconfig test executed by Murdock

This PR also refactors the IEEE 802.15.4 menu, because MODULE_IEEE802154 just provides helper functions for processing IEEE 802.15.4 frames. Last but not least, it removes unused configurations in nrf802154.

Testing procedure

Murdock should be enough I guess.

Issues/PRs references

Depends on #16837

jia200x avatar Sep 13 '21 15:09 jia200x

I think this needs a rebase before review

MrKevinWeiss avatar Oct 18 '21 11:10 MrKevinWeiss

@jia200x this needs a rebase ~now~(sorry the now made it sound like an order)!

fjmolinas avatar Nov 02 '21 10:11 fjmolinas

is there anything missing here?

jia200x avatar Nov 10 '21 09:11 jia200x

is there anything missing here?

Yes, I have some unanswered comments, I think currently the submac test does not build

fjmolinas avatar Nov 10 '21 10:11 fjmolinas

I rebased and fixed some of the comments from @leandrolanzieri

MrKevinWeiss avatar Feb 15 '22 12:02 MrKevinWeiss

Still some issues @MrKevinWeiss (assuming you are taking over)

fjmolinas avatar Mar 04 '22 09:03 fjmolinas

I think @leandrolanzieri is working on a broader solution that will make this obsolete...

MrKevinWeiss avatar Mar 04 '22 10:03 MrKevinWeiss

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

stale[bot] avatar Sep 21 '22 05:09 stale[bot]

@MrKevinWeiss this now outdated, right? Can it be closed?

Teufelchen1 avatar Mar 26 '24 12:03 Teufelchen1

Yup

MrKevinWeiss avatar Apr 09 '24 11:04 MrKevinWeiss