Skip to content

modules: hal_infineon: Fix SPI support for AIROC driver #90789

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MagpieEmbedded
Copy link
Contributor

Amends changes made in a recent update to the WHD driver to add conditional compilation of spi source file.

This support was originally added here: 37e39de#diff-13ce3c21bdef5739612dbf0b9d4eb3d007484a399d4e3eef0a19ee74da2414d4

Then in a recent change to update the WHD driver the ifdefs for CONFIG_AIROC_WIFI_BUS_SPI were removed: c9ffcac#diff-13ce3c21bdef5739612dbf0b9d4eb3d007484a399d4e3eef0a19ee74da2414d4
(L60-62 of modules/hal_infineon/wifi-host-driver/CMakeLists.txt)

@github-actions github-actions bot added the platform: Infineon Infineon Technologies AG label May 29, 2025
@MagpieEmbedded MagpieEmbedded force-pushed the hal_infineon_whd_spi_fix branch from 98e6c68 to 2f599ee Compare May 29, 2025 09:14
@MagpieEmbedded
Copy link
Contributor Author

This was preventing building networking code on the RPi Pico.

On head currently, there would be a linker error with a failure to find the whd_bus_spi_attach symbol. With this commit it builds and works.

I have been using this command: west build -p -b rpi_pico/rp2040/w samples/net/wifi/shell/

@MagpieEmbedded
Copy link
Contributor Author

Related: #90423? Although this adds the BLHS_SUPPORT definition which is not mentioned in that PR

@ThreeEights
Copy link
Collaborator

ThreeEights commented Jun 5, 2025

Duplicate of #90423. Update of the Infineon HAL layer is also required.

Amends changes made in a recent update to the WHD driver to add
conditional compilation of spi source file

Signed-off-by: Magpie Embedded <magpieembedded@gmail.com>
@MagpieEmbedded MagpieEmbedded force-pushed the hal_infineon_whd_spi_fix branch from 2f599ee to 1a94b6e Compare June 5, 2025 17:19
Copy link

sonarqubecloud bot commented Jun 5, 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