[Backport v4.4-branch] dts: arm: stm32wba: add interrupts for bt_hci_wba and ieee802154 nodes#108220
[Backport v4.4-branch] dts: arm: stm32wba: add interrupts for bt_hci_wba and ieee802154 nodes#108220github-actions[bot] wants to merge 4 commits into
Conversation
Add interrupt-parent and interrupt-names into the bt_hci_wba and ieee802154 nodes. Signed-off-by: Vincent Tardy <vincent.tardy@st.com> (cherry picked from commit 7dc5a0f)
Define IRQs numbers for bt_hci_wba and ieee802154 nodes. The IRQs are the same for both nodes. Signed-off-by: Vincent Tardy <vincent.tardy@st.com> (cherry picked from commit 4028255)
Add radio interrupt and radio SW Low interrupt for the nucleo wba board. The radio SW Low interrupt is mapped on the unused HASH interrupt. In case of HASH interrupt is used, this setting shall be changed. Signed-off-by: Vincent Tardy <vincent.tardy@st.com> (cherry picked from commit 7910e27)
…odes Use the interrupts properties from bt and ieee802154 nodes. Signed-off-by: Vincent Tardy <vincent.tardy@st.com> (cherry picked from commit 49337c6)
zephyrbot
requested review from
FRASTM,
GeorgeCGV,
HoZHel,
asm5878,
djiatsaf-st,
erwango,
etienne-lms,
gautierg-st,
juickar and
mathieuchopstm
|
|
I've added a Issue reference (#108246). |
|
@etienne-lms could you please add your +1 review? |
| &bt_hci_wba { | ||
| /* Use HASH IRQ to allocate Radio SW Low Process (Warning HASH is not used) */ | ||
| interrupts = <60 0>, <55 14>; | ||
| interrupt-names = "radio", "radio-sw-low"; | ||
| }; | ||
|
|
There was a problem hiding this comment.
Bluetooth, or radio in general, is not supported on WBA2 in Zephyr v4.4. This information may be misleading.
There was a problem hiding this comment.
@HoZHel :should I create new commit including the remove of this part of code or rework the commit including this change ?
There was a problem hiding this comment.
I prefer the latter, and I also suggest disabling the node with a proper comment.
You can also mention this change in the PR description as it differs from the original PR.
|
|
||
| bt_hci_wba: bt_hci_wba { | ||
| interrupts = <60 0>; | ||
| }; | ||
|
|
||
| ieee802154: ieee802154 { | ||
| interrupts = <60 0>; | ||
| }; |
There was a problem hiding this comment.
Please, see above comment.
|
Closed in favor of #110341 |
7 participants
Backport 49337c6~4..49337c6 from #107832.
Fixes: #108246
Original PR description:
This PR adds 'radio' and 'radio-sw-low' interrupts properties in the bt_hci_wba and ieee802154 nodes for STM32WBAx.
These interrupts properties are used during radio IRQ management.
The 'radio" interrupt is mapped on the dedicated RADIO IRQ while the 'radio-sw-low' interrupt is mapped on an available IRQ (HASH IRQ by default).
This PR is implemented to fix issue observed during compilation based on bluetooth or ieee802154 protocols on STM32WBA55 soc due to CONFIG_NUM_IRQS uncompliant with wireless radio use on STM32WBA55
FYI : @asm5878 , @erwango , @etienne-lms