add sdhc support on i.MX 91 FRDM board #97509
Open
+356
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
github-actions
bot
added
manifest
manifest-hal_nxp
DNM (manifest)
zephyrbot
added
area: Boards/SoCs
area: ARM64
zephyrbot
requested review from
EmilioCBen,
Laczen,
PetervdPerk-NXP,
ZhaoxiangJin,
Zhiqiang-Hou,
bperseghetti,
dbaluta,
de-nordic,
decsny,
dleach02,
iuliana-prodan,
manuargue,
mmahadevan108,
nashif and
yangbolu1991
JiafeiPan
force-pushed
the
develop/imx91-frdm-sdhc
branch
2 times, most recently
from
44700b3 to
903d97b
Compare
|
Rebased and update west.yml to include the hal_nxp update which has been merged. |
github-actions
bot
removed
the
DNM (manifest)
decsny
requested changes
Oct 15, 2025
| if IMX_USDHC | ||
|
|
||
| config GPIO | ||
| default y |
There was a problem hiding this comment.
could also just do default y if IMX_USDHC
Comment on lines
53
to
57
| sdmmc { | ||
| compatible = "zephyr,mmc-disk"; | ||
| disk-name = "SD2"; | ||
| status = "disabled"; | ||
| }; |
There was a problem hiding this comment.
why is the node name sdmmc but the compatible is mmc
Comment on lines
1
to
16
| /* | ||
| * Copyright 2025 NXP | ||
| * | ||
| * SPDX-License-Identifier: Apache-2.0 | ||
| */ | ||
|
|
||
| #ifndef _SOC_NXP_IMX_IMX91_SOC_H_ | ||
| #define _SOC_NXP_IMX_IMX91_SOC_H_ | ||
|
|
||
|
|
||
| #endif /* _SOC_NXP_IMX_IMX91_SOC_H_ */ |
There was a problem hiding this comment.
I found currently soc_common.h includes some macro definitions and not useful so far, but anymore I added it at here.
JiafeiPan
force-pushed
the
develop/imx91-frdm-sdhc
branch
from
903d97b to
05c3f5b
Compare
|
dbaluta
approved these changes
Oct 16, 2025
To include the hal_nxp update for i.MX 93 SDHC. Signed-off-by: Jiafei Pan <[email protected]>
Add device nodes for SDHC. Signed-off-by: Lei Xu <[email protected]> Signed-off-by: Jiafei Pan <[email protected]>
Some drivers need header file soc.h, according to Zephyr SoC Porting Guide soc.h must be provided for each SoC, so created an empty one. Signed-off-by: Lei Xu <[email protected]> Signed-off-by: Jiafei Pan <[email protected]>
Added pinctrl and dts nodes for uSDHC1 and uSDHC2, they are disabled by default, and provided overlay files to enable them if needed. Signed-off-by: Lei Xu <[email protected]> Signed-off-by: Jiafei Pan <[email protected]>
Increase CONFIG_MAIN_STACK_SIZE to 8192 to avoid possible crash. Signed-off-by: Lei Xu <[email protected]> Signed-off-by: Jiafei Pan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
add sdhc support on i.MX 91 FRDM board.
Depends on zephyrproject-rtos/hal_nxp#620 (has been merged)