Skip to content

Conversation

@ExaltZephyr
Copy link
Contributor

This PR fixes an issue in SDIO driver where HAL_SDIO_Init fails if the card identification sequence is skipped.

Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

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

Please document changes in each HAL README file, otherwise change is ok and will be delivered in HAL in next versions.

@ExaltZephyr
Copy link
Contributor Author

ExaltZephyr commented May 27, 2025

@erwango we have added the fix to the HAL SDIO driver for STM32H7 and STM32U5 series.
I see that we have new HAL SDIO added under stm32n6 and stm32h7rs, would you recommend applying the same fix to those as well ?

@erwango
Copy link
Member

erwango commented May 27, 2025

I see that we have new HAL SDIO added under stm32n6 and stm32h7rs, would you recommend applying the same fix to those as well ?

Sure go ahead, indeed. Thx

when skipping card identification sequence.

This commit fixes an issue in the STM32H7 SDIO driver
where HAL_SDIO_Init() fails if the card identification
sequence is skipped.

Signed-off-by: Sara Touqan <[email protected]>
when skipping card identification sequence.

Fixes an issue in the STM32U5 SDIO driver where HAL_SDIO_Init() fails if
the card identification sequence is skipped.

Signed-off-by: Sara Touqan <[email protected]>
when skipping card identification sequence.

This commit fixes an issue in the STM32H5 SDIO driver
where HAL_SDIO_Init() fails if the card identification
sequence is skipped.

Signed-off-by: Sara Touqan <[email protected]>
when skipping card identification sequence.

This commit fixes an issue in the STM32H7RS SDIO driver
where HAL_SDIO_Init() fails if the card identification
sequence is skipped.

Signed-off-by: Sara Touqan <[email protected]>
when skipping card identification sequence.

This commit fixes an issue in the STM32N6 SDIO driver
where HAL_SDIO_Init() fails if the card identification
sequence is skipped.

Signed-off-by: Sara Touqan <[email protected]>
@erwango erwango merged commit c83a9ef into zephyrproject-rtos:main Jun 6, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants