-
Notifications
You must be signed in to change notification settings - Fork 6.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
drivers: ieee802154: cc13xx_cc26xx: sub-ghz support #29598
drivers: ieee802154: cc13xx_cc26xx: sub-ghz support #29598
Conversation
Notes:
|
Furthermore, it requires zephyrproject-rtos/hal_ti#22 in order to pull in the multi-protocol RF backend. Will likely also require some whitespace corrections |
Until #29750 is resolved, SubGHz and 2.4 GHz cannot be used simultaneously, so to test this, please use the following: server: west build \
-p always \
-b cc1352r1_launchxl \
samples/net/sockets/echo_server \
-- \
-DOVERLAY_CONFIG=overlay-802154-subg.conf \
-DCONFIG_NET_CONFIG_IEEE802154_DEV_NAME="\"IEEE802154_1\"" \
-DCONFIG_IEEE802154_CC13XX_CC26XX=n \
-DCONFIG_IEEE802154_CC13XX_CC26XX_SUB_GHZ=y client: west build \
-p always \
-b cc1352r1_launchxl \
samples/net/sockets/echo_client \
-- \
-DOVERLAY_CONFIG=overlay-802154-subg.conf \
-DCONFIG_NET_CONFIG_IEEE802154_DEV_NAME="\"IEEE802154_1\"" \
-DCONFIG_IEEE802154_CC13XX_CC26XX=n \
-DCONFIG_IEEE802154_CC13XX_CC26XX_SUB_GHZ=y |
So odd that checkpatch.pl doesn't catch some long lines. I don't understand... was the 80-character limit silently extended? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small changes requested
@vanti - I've defaulted
For the second item, we're seeing things like the following:
Tried increasing |
Looks like the fix for this is in zephyrproject-rtos/hal_nxp#66 and #29797 Is there anything I can do to sidestep that at the moment, or am I blocked until that fix is merged into hal/nxp & zephyr? |
It's been merged now, can you rebase? |
This change adds IEEE 802.15.4g (Sub GHz) support for the cc1352r. The 2.4 GHz radio and the Sub GHz radio are capable of operating simultaneously. Fixes #26315 Signed-off-by: Christopher Friedt <[email protected]>
Hello, does this support mean one can use zephyr 6lowpan with CC1310? |
This change adds IEEE 802.15.4g (Sub GHz) support for the cc1352r.
The 2.4 GHz radio and the Sub GHz radio are capable of operating simultaneously.
Fixes #26315