Skip to content

Conversation

@curtdept
Copy link

@curtdept curtdept commented Jun 13, 2025

Modifies the configuration to enable the CYW43439 WiFi/BT module.

Build tested: cy8ckit_062s2_ai
Run tested: cy8ckit_062s2_ai

Signed-off-by: Curtis Deptuck curtdept@me.com

@github-actions
Copy link

Hello @curtdept, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@github-actions github-actions bot added the platform: Infineon Infineon Technologies AG label Jun 13, 2025
@curtdept curtdept force-pushed the main branch 2 times, most recently from d2fe4f7 to a1019ba Compare June 17, 2025 23:09
@ifyall
Copy link
Contributor

ifyall commented Jun 17, 2025

@curtdept, can you provide a log for the Wi-Fi and BLE running on the kit? That would help with the review.

Thank you,

Ian

@sonarqubecloud
Copy link

@curtdept
Copy link
Author

curtdept commented Jun 18, 2025

@ifyall not sure which log you are after but I modified the wifi shell sample to include wifi, wifi mgmt and if dbg and this is the result. Hope that helps.

wifi_log.txt

@curtdept
Copy link
Author

curtdept commented Jun 18, 2025

@ifyall BT log
bt_log.txt

that being said, there is def a bug in the HCI timeout or a race somewhere, I somewhat frequently encounter this is there isnt enough spacing in between the commands. Working theory is that the 0x0c03 is being lost due to a reset? after the fx download. I elongated the timer HCI_CMD_TIMEOUT and it would still fail.

[00:00:01.528,000] <dbg> cyw43xxx_driver: bt_firmware_download: Fw downloading complete
[00:00:01.779,000] <dbg> bt_hci_core: bt_hci_cmd_create: opcode 0x0c03 param_len 0
[00:00:01.779,000] <dbg> bt_hci_core: bt_hci_cmd_create: buf 0x80118ac
[00:00:01.779,000] <dbg> bt_hci_core: bt_hci_cmd_send_sync: buf 0x80118ac opcode 0x0c03 len 4
[00:00:01.779,000] <dbg> bt_hci_core: bt_tx_irq_raise: kick TX
[00:00:01.779,000] <dbg> bt_hci_core: tx_processor: TX process start
[00:00:01.779,000] <dbg> bt_hci_core: hci_core_send_cmd: fetch cmd
[00:00:01.779,000] <dbg> bt_hci_core: hci_core_send_cmd: Sending command 0x0c03 (buf 0x80118ac) to driver
[00:00:01.779,000] <dbg> bt_hci_core: bt_send: buf 0x80118ac len 4 type 1
[00:00:01.779,000] <dbg> bt_driver: h4_send: buf 0x80118ac type 1 len 4
[00:00:01.779,000] <dbg> bt_hci_core: bt_tx_irq_raise: kick TX
[00:00:01.779,000] <dbg> bt_hci_core: tx_processor: TX process start
ASSERTION FAIL [err == 0] @ WEST_TOPDIR/zephyr/subsys/bluetooth/host/hci_core.c:472
        Controller unresponsive, command opcode 0x0c03 timeout with err -11
[00:00:31.779,000] <err> os: r0/a1:  0x00000003  r1/a2:  0x00000000  r2/a3:  0x00000002
[00:00:31.779,000] <err> os: r3/a4:  0x00000003 r12/ip:  0x00007c23 r14/lr:  0x10006a63
[00:00:31.779,000] <err> os:  xpsr:  0x01000000
[00:00:31.779,000] <err> os: Faulting instruction address (r15/pc): 0x10006a72
[00:00:31.779,000] <err> os: >>> ZEPHYR FATAL ERROR 3: Kernel oops on CPU 0
[00:00:31.779,000] <err> os: Current thread: 0x8004940 (unknown)
[00:00:31.806,000] <err> os: Halting system

@curtdept
Copy link
Author

curtdept commented Jun 18, 2025

@ifyall ya that's what it is, if I

(void)k_msleep(1000u);

at the end of drivers/bluetooth/hci/h4_ifx_cyw43xxx.c

it will load fine. Not sure how you guys handle changes like that.

bt_sleep_log.txt

@sonarqubecloud
Copy link

@curtdept
Copy link
Author

@sreeramIfx ping

@kartben kartben assigned sreeramIfx and unassigned ifyall Aug 28, 2025
@sreeramIfx
Copy link
Contributor

Please drop the merge commit.

@curtdept curtdept closed this Oct 2, 2025
Copilot AI review requested due to automatic review settings October 2, 2025 15:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 2, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: Infineon Infineon Technologies AG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants