* [PATCH v11] riscv: dts: spacemit: k1-musepi-pro: add SD card support with UHS modes
@ 2026-05-20 13:06 Andre Heider
2026-05-20 13:21 ` sashiko-bot
0 siblings, 1 reply; 2+ messages in thread
From: Andre Heider @ 2026-05-20 13:06 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Yixun Lan,
Paul Walmsley, Palmer Dabbelt, Albert Ou, Alexandre Ghiti
Cc: Trevor Gamblin, Iker Pedrosa, devicetree, linux-riscv, spacemit,
linux-kernel
From: Trevor Gamblin <tgamblin@baylibre.com>
Update the Muse Pi Pro devicetree with SD card support to match what
was done for the OrangePi RV2 in [1]. More precisely:
- Enable sdhci0 controller with 4-bit bus width
- Configure card detect GPIO with internal pull-up support
- Connect vmmc-supply to buck4 for 3.3V card power
- Connect vqmmc-supply to aldo1 for 1.8V/3.3V I/O switching
- Add dual pinctrl states for voltage-dependent pin configuration
- Support UHS-I SDR25, SDR50, and SDR104 modes
- Add stable MMC device aliases (mmc0 = eMMC, mmc1 = SD card)
[1] https://lore.kernel.org/linux-riscv/20260316-orangepi-sd-card-uhs-v3-0-aefd3b7832df@gmail.com/T/#
Tested-by: Andre Heider <a.heider@gmail.com>
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Iker Pedrosa <ikerpedrosam@gmail.com>
Signed-off-by: Andre Heider <a.heider@gmail.com>
---
Hi,
this is just a rebase of Iker's v10 4/4 patch[0] on top of my merged
musepi-pro series[1].
[0] https://lore.kernel.org/all/20260515-orangepi-sd-card-uhs-v10-4-094af27e310d@gmail.com/
[1] https://lore.kernel.org/all/20260513071958.29574-1-a.heider@gmail.com/
.../riscv/boot/dts/spacemit/k1-musepi-pro.dts | 22 ++++++++++++++++++-
1 file changed, 21 insertions(+), 1 deletion(-)
diff --git a/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts b/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts
index 4aef287353cab..0d809e4ad3b1f 100644
--- a/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts
+++ b/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts
@@ -19,6 +19,8 @@ aliases {
serial0 = &uart0;
i2c2 = &i2c2;
i2c8 = &i2c8;
+ mmc0 = &emmc;
+ mmc1 = &sdhci0;
};
chosen {
@@ -218,7 +220,7 @@ buck6 {
regulator-always-on;
};
- aldo1 {
+ aldo1: aldo1 {
regulator-min-microvolt = <500000>;
regulator-max-microvolt = <3400000>;
regulator-boot-on;
@@ -384,3 +386,21 @@ hub_3_0: hub@2 {
reset-gpios = <&gpio K1_GPIO(123) GPIO_ACTIVE_LOW>; /* HUB_RST */
};
};
+
+&sdhci0 {
+ pinctrl-names = "default", "uhs";
+ pinctrl-0 = <&mmc1_cfg>;
+ pinctrl-1 = <&mmc1_uhs_cfg>;
+ bus-width = <4>;
+ cd-gpios = <&gpio K1_GPIO(80) (GPIO_ACTIVE_HIGH | GPIO_PULL_UP)>;
+ no-mmc;
+ no-sdio;
+ disable-wp;
+ cap-sd-highspeed;
+ vmmc-supply = <&buck4_3v3>;
+ vqmmc-supply = <&aldo1>;
+ sd-uhs-sdr25;
+ sd-uhs-sdr50;
+ sd-uhs-sdr104;
+ status = "okay";
+};
--
2.53.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v11] riscv: dts: spacemit: k1-musepi-pro: add SD card support with UHS modes
2026-05-20 13:06 [PATCH v11] riscv: dts: spacemit: k1-musepi-pro: add SD card support with UHS modes Andre Heider
@ 2026-05-20 13:21 ` sashiko-bot
0 siblings, 0 replies; 2+ messages in thread
From: sashiko-bot @ 2026-05-20 13:21 UTC (permalink / raw)
To: Andre Heider; +Cc: devicetree, conor+dt, robh
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [High] Inverted SD card detect polarity in `cd-gpios` property.
--
commit b8da2f3ea83cbd0820dd4246075e26e687bac678
Author: Trevor Gamblin <tgamblin@baylibre.com>
riscv: dts: spacemit: k1-musepi-pro: add SD card support with UHS modes
This commit updates the Muse Pi Pro device tree to add SD card support,
enabling the sdhci0 controller with a 4-bit bus, UHS-I modes, and voltage
switching.
Link: https://lore.kernel.org/linux-riscv/20260316-orangepi-sd-card-uhs-v3-0-aefd3b7832df@gmail.com/T/#
> diff --git a/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts b/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts
> --- a/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts
> +++ b/arch/riscv/boot/dts/spacemit/k1-musepi-pro.dts
[ ... ]
> +&sdhci0 {
> + pinctrl-names = "default", "uhs";
> + pinctrl-0 = <&mmc1_cfg>;
> + pinctrl-1 = <&mmc1_uhs_cfg>;
> + bus-width = <4>;
> + cd-gpios = <&gpio K1_GPIO(80) (GPIO_ACTIVE_HIGH | GPIO_PULL_UP)>;
[Severity: High]
Should this cd-gpios property use GPIO_ACTIVE_LOW?
With an internal pull-up enabled, the physical card-detect switch usually
connects the pin to ground when a card is inserted, driving the signal low.
Using GPIO_ACTIVE_HIGH might cause the MMC subsystem to assume a card is
inserted when the slot is empty, and report a removal event when a card is
actually inserted.
The commit message mentions matching the OrangePi RV2 configuration, which
would typically use GPIO_ACTIVE_LOW for a pulled-up card detect signal.
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260520130624.1763487-1-a.heider@gmail.com?part=1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-05-20 13:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-20 13:06 [PATCH v11] riscv: dts: spacemit: k1-musepi-pro: add SD card support with UHS modes Andre Heider
2026-05-20 13:21 ` sashiko-bot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox