-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
IEEE 802.15.4 support for ESP32-C6 #82876
base: main
Are you sure you want to change the base?
IEEE 802.15.4 support for ESP32-C6 #82876
Conversation
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
29bcd1a
to
10168bf
Compare
Rebased and fixed some bugs so that the firmware doesn't get stuck anymore and we reach the shell. |
78413d0
to
a85597e
Compare
Hi @martinjaeger, I see you're actively working on this, thank you! My team is beginning development of an OpenThread application prototype on an ESP32-C6-Mini based custom board soon. Personally I would love for it to be Zephyr based. What hurdles are there still to take? Can we support you? |
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.
One more thing, please move west.yml changes as the first commit in this PR.
@@ -71,3 +72,7 @@ | |||
&wdt0 { | |||
status = "okay"; | |||
}; | |||
|
|||
&ieee802154 { |
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.
This 'enable' should go into a samples/.../shell/socs/esp32c6.overlay
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.
No, I don't think so. It is also enabled in the board file of nrf52840dk
and many other boards. The sample should just enable OpenThread via Kconfig.
@@ -21,6 +21,7 @@ | |||
zephyr,shell-uart = &uart0; | |||
zephyr,flash = &flash0; | |||
zephyr,code-partition = &slot0_partition; | |||
zephyr,ieee802154 = &ieee802154; |
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.
this should probably go to the sample overlay.
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.
I don't thinks so. See previous comment.
Why do you think that's necessary? The first commit in this PR is unrelated and tries to address the issues with the interrupt controller, so it will be removed for the final version of the PR and the Please ignore any style issues for now. This PR is still in draft mode. We should focus on getting the driver to work first. I will clean it up at the end. |
UART does not work reliably anymore if assigned to this IRQ. Signed-off-by: Martin Jäger <[email protected]>
PRELIMINARY: Pointing to related PR head. Signed-off-by: Martin Jäger <[email protected]>
Draft implementation of the IEEE 802.15.4 driver using Espressif HAL. Signed-off-by: Martin Jäger <[email protected]>
Enable IEEE 802.15.4 driver for esp32c6_devkitc and xiao_esp32c6 boards. Signed-off-by: Martin Jäger <[email protected]>
a85597e
to
627c41e
Compare
Hi @MaikVermeulen thanks for offering support. The driver is compiling and we have successfully received and transmitted some packets (verified with a sniffer based on a nRF52840 board). However, From the log output it seems that we are receiving packages from the scan after the scan has finished and In general there seems to be a major issue with the interrupt allocator for the ESP32C6. We have observed similar issues with interrupts being fired too late (sometimes several hundreds of milliseconds) in the UART driver. Modifying the reserved interrupts in You can try it out by using the branch of this PR and running |
Code builds for
xiao_esp32c6
, butot scan
on the shell does not yet succeed.