From: Trevor Gamblin <tgamblin@baylibre.com>
To: Iker Pedrosa <ikerpedrosam@gmail.com>,
Ulf Hansson <ulf.hansson@linaro.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Adrian Hunter <adrian.hunter@intel.com>,
Paul Walmsley <pjw@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>,
Alexandre Ghiti <alex@ghiti.fr>, Yixun Lan <dlan@kernel.org>
Cc: Michael Opdenacker <michael.opdenacker@rootcommit.com>,
Javier Martinez Canillas <javierm@redhat.com>,
linux-mmc@vger.kernel.org, devicetree@vger.kernel.org,
linux-riscv@lists.infradead.org, spacemit@lists.linux.dev,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 7/7] riscv: dts: spacemit: k1-orangepi-rv2: add SD card support with UHS modes
Date: Fri, 13 Mar 2026 09:54:49 -0400 [thread overview]
Message-ID: <f2eeddcb-712f-4a67-9687-e217f46f9ea8@baylibre.com> (raw)
In-Reply-To: <20260309-orangepi-sd-card-uhs-v2-7-5bb2b574df5d@gmail.com>
On 2026-03-09 07:40, Iker Pedrosa wrote:
> Add complete SD card controller support with UHS high-speed modes.
>
> - Enable sdhci0 controller with 4-bit bus width
> - Configure card detect GPIO with inversion
> - 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
>
> This enables full SD card functionality including high-speed UHS modes
> for improved performance.
>
> Signed-off-by: Iker Pedrosa <ikerpedrosam@gmail.com>
> ---
> arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts b/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts
> index 414b03f5e6480f05f5d7eeaaa0afb4e86425ae36..361135269801f436703b6f1d768c91325a52f07f 100644
> --- a/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts
> +++ b/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts
> @@ -140,3 +140,22 @@ sd_vqmmc: aldo1 {
> };
> };
> };
> +
> +&sdhci0 {
> + pinctrl-names = "default", "state_uhs";
> + pinctrl-0 = <&mmc1_cfg>;
> + pinctrl-1 = <&mmc1_uhs_cfg>;
> + bus-width = <4>;
> + cd-gpios = <&gpio K1_GPIO(80) GPIO_ACTIVE_HIGH>;
> + cd-inverted;
> + no-mmc;
> + no-sdio;
> + disable-wp;
> + cap-sd-highspeed;
> + vmmc-supply = <&sd_vmmc>;
> + vqmmc-supply = <&sd_vqmmc>;
> + sd-uhs-sdr25;
> + sd-uhs-sdr50;
> + sd-uhs-sdr104;
> + status = "okay";
> +};
>
Hello again,
Thanks for doing this work. My colleagues and I have been testing this
on an OrangePi RV2, and also a Muse Pi Pro board (making a similar tweak
to this patch in k1-musepi-pro.dts). To do the testing on my end, I'm
applying the patch series on top of 6.19 in a Yocto BSP.
I've been finding that I see issues with the UHS support on my side
(this log comes from an image built and flashed to an SD card):
[ 1.072417] sdhci-spacemit d4280000.mmc: Got CD GPIO
[ 1.109741] mmc0: SDHCI controller on d4280000.mmc [d4280000.mmc] using
ADMA
[ 1.114589] clk: Disabling unused clocks
[ 1.118421] PM: genpd: Disabling unused power domains
[ 1.123249] ALSA device list:
[ 1.126[ 1.129949] check access for rdinit=/init failed: -2, ignoring
[ 1.133284] Waiting for root device
PARTUUID=e94bfdd7-a36f-4315-a480-476e2a12403d...
[ 1.176569] mmc0: new UHS-I speed DDR50 SDHC card at address aaaa
[ 1.180732] mmcblk0: mmc0:aaaa SS16G 14.8 GiB
[ 1.276268] mmcblk0: recovery failed!
[ 1.277351] I/O error, dev mmcblk0, sector 0 op 0x0:(READ) flags
0x800000 phys_seg 1 prio class 2
[ 1.286346] Buffer I/O error on dev mmcblk0, logical block 0, async page
read
[ 1.293891] mmcblk0: recovery failed!
[ 1.297289] I/O error, dev mmcblk0, sector 0 op 0x0:(READ) flags
0x800000 phys_seg 1 prio class 2
[ 1.306269] Buffer I/O error on dev mmcblk0, logical block 0, async page
read
[ 1.313522] mmcblk0: unable to read partition table
At first I wondered if maybe there was an issue with the card itself,
but by changing the sdhci0 section to this, I'm able to read the SD just
fine:
&sdhci0 {
pinctrl-names = "default";
pinctrl-0 = <&mmc1_cfg>;
bus-width = <4>;
cd-gpios = <&gpio K1_GPIO(80) GPIO_ACTIVE_HIGH>;
cd-inverted;
no-mmc;
no-sdio;
disable-wp;
vmmc-supply = <&sd_vmmc>;
vqmmc-supply = <&sd_vqmmc>;
no-1-8-v;
status = "okay";
};
Here's the boot log, for comparison:
[ 1.068399] sdhci-spacemit d4280000.mmc: Got CD GPIO
[ 1.105952] mmc0: SDHCI controller on d4280000.mmc [d4280000.mmc]
using ADMA
[ 1.110807] clk: Disabling unused clocks
[ 1.114624] PM: genpd: Disabling unused power domains
[ 1.119456] ALSA device list:
[ 1.122[ 1.126159] check access for rdinit=/init failed: -2, ignoring
[ 1.129491] Waiting for root device
PARTUUID=e94bfdd7-a36f-4315-a480-476e2a12403d...
[ 1.155817] mmc0: new high speed SDHC card at address aaaa
[ 1.159367] mmcblk0: mmc0:aaaa SS16G 14.8 GiB
[ 1.173666] GPT:Primary header thinks Alt. header is not at the end
of the disk.
[ 1.178530] GPT:1135739 != 31116287
[ 1.182045] GPT:Alternate GPT header not at the end of the disk.
[ 1.188124] GPT:1135739 != 31116287
[ 1.191650] GPT: Use GNU Parted to correct GPT errors.
[ 1.196894] mmcblk0: p1 p2 p3
[ 1.375482] EXT4-fs (mmcblk0p3): recovery complete
[ 1.379459] EXT4-fs (mmcblk0p3): mounted filesystem
ad761819-1030-44fe-9267-f59c53c4b5c4 r/w with ordered data mode. Quota
mode: disabled.
[ 1.390400] VFS: Mounted root (ext4 filesystem) on device 179:3.
To be sure, I tried a different SDXC card with your version and saw:
[ 1.064528] sdhci-spacemit d4280000.mmc: Got CD GPIO
[ 1.102113] mmc0: SDHCI controller on d4280000.mmc [d4280000.mmc]
using ADMA
[ 1.106978] clk: Disabling unused clocks
[ 1.110792] PM: genpd: Disabling unused power domains
[ 1.115622] ALSA device list:
[ 1.118[ 1.122325] check access for rdinit=/init failed: -2, ignoring
[ 1.125656] Waiting for root device
PARTUUID=e94bfdd7-a36f-4315-a480-476e2a12403d...
[ 1.215840] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 1.220520] mmc0: tuning execution failed: -5
[ 1.224931] mmc0: error -5 whilst initialising SD card
[ 1.378228] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 1.382901] mmc0: tuning execution failed: -5
[ 1.506443] mmc0: new high speed SDXC card at address 1388
[ 1.510009] mmcblk0: mmc0:1388 SATC 57.7 GiB
[ 1.645467] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 1.650158] mmc0: tuning execution failed: -5
[ 1.772856] GPT:Primary header thinks Alt. header is not at the end
of the disk.
[ 1.777718] GPT:1135725 != 120930303
[ 1.781317] GPT:Alternate GPT header not at the end of the disk.
[ 1.787397] GPT:1135725 != 120930303
[ 1.791022] GPT: Use GNU Parted to correct GPT errors.
[ 1.796238] mmcblk0: p1 p2 p3
[ 1.917789] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 1.922483] mmc0: tuning execution failed: -5
[ 2.174518] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 2.179193] mmc0: tuning execution failed: -5
[ 2.337900] EXT4-fs (mmcblk0p3): mounted filesystem
d1f9e739-cc74-4c48-a774-390717721c53 r/w with ordered data mode. Quota
mode: disabled.
[ 2.348031] VFS: Mounted root (ext4 filesystem) on device 179:3.
[ 2.492089] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 2.496771] mmc0: tuning execution failed: -5
[ 2.617641] devtmpfs: mounted
[ 2.618771] Freeing unused kernel image (initmem) memory: 2432K
[ 2.624085] Run /sbin/init as init process
[ 2.749813] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 2.754492] mmc0: tuning execution failed: -5
[ 3.013665] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 3.018357] mmc0: tuning execution failed: -5
[ 3.279459] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 3.284139] mmc0: tuning execution failed: -5
[ 3.541764] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 3.546450] mmc0: tuning execution failed: -5
[ 3.665967] I/O error, dev mmcblk0, sector 259162 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.672894] I/O error, dev mmcblk0, sector 259163 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.683275] I/O error, dev mmcblk0, sector 259170 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.691878] I/O error, dev mmcblk0, sector 259172 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.701516] I/O error, dev mmcblk0, sector 259175 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.711067] I/O error, dev mmcblk0, sector 259178 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.720183] I/O error, dev mmcblk0, sector 259180 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.730004] I/O error, dev mmcblk0, sector 259183 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.739010] I/O error, dev mmcblk0, sector 259184 op 0x0:(READ) flags
0x880700 phys_seg 1 prio class 2
[ 3.888355] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 3.893045] mmc0: tuning execution failed: -5
[ 4.149999] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 4.154686] mmc0: tuning execution failed: -5
[ 4.283933] I/O error, dev mmcblk0, sector 209865 op 0x0:(READ) flags
0x803000 phys_seg 1 prio class 2
[ 4.290750] EXT4-fs warning (device mmcblk0p3):
ext4_dx_find_entry:1726: inode #768: lblock 1: comm swapper/0: error -5
reading directory block
[ 4.303879] Starting init: /sbin/init exists but couldn't execute it
(error -5)
[ 4.311194] Run /etc/init as init process
[ 4.316220] Run /bin/init as init process
[ 4.319996] Run /bin/sh as init process
[ 4.457522] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 4.462200] mmc0: tuning execution failed: -5
[ 4.719271] sdhci-spacemit d4280000.mmc: Tuning failed: no stable
window found
[ 4.723945] mmc0: tuning execution failed: -5
[ 4.842487] EXT4-fs warning (device mmcblk0p3):
ext4_dx_find_entry:1726: inode #768: lblock 1: comm swapper/0: error -5
reading directory block
[ 4.852959] Starting init: /bin/sh exists but couldn't execute it
(error -5)
[ 4.860044] Kernel panic - not syncing: No working init found. Try
passing init= option to kernel. See Linux
Documentation/admin-guide/init.rst for guidance.
[ 4.874394] CPU: 4 UID: 0 PID: 1 Comm: swapper/0 Not tainted
6.19.0-00008-ga23b3fad2d87 #1 NONE
[ 4.883295] Hardware name: OrangePi RV2 (DT)
I don't think I've missed anything between 6.19 and 7.x, but it's
possible. It seems like there's a timing issue at play?
Let me know if I can help with further testing. I'm also in #spacemit on
IRC.
next prev parent reply other threads:[~2026-03-13 13:54 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-09 11:40 [PATCH v2 0/7] riscv: spacemit: enable SD card support with UHS modes for OrangePi RV2 Iker Pedrosa
2026-03-09 11:40 ` [PATCH v2 1/7] mmc: sdhci-of-k1: enable essential clock infrastructure for SD operation Iker Pedrosa
2026-03-13 13:04 ` Adrian Hunter
2026-03-16 9:04 ` Iker Pedrosa
2026-03-16 9:34 ` Adrian Hunter
2026-03-09 11:40 ` [PATCH v2 2/7] mmc: sdhci-of-k1: add regulator and pinctrl voltage switching support Iker Pedrosa
2026-03-09 13:22 ` Yixun Lan
2026-03-12 9:38 ` Iker Pedrosa
2026-03-13 13:04 ` Adrian Hunter
2026-03-09 11:40 ` [PATCH v2 3/7] mmc: sdhci-of-k1: add SDR tuning infrastructure Iker Pedrosa
2026-03-13 13:04 ` Adrian Hunter
2026-03-09 11:40 ` [PATCH v2 4/7] mmc: sdhci-of-k1: add comprehensive SDR tuning support Iker Pedrosa
2026-03-13 13:04 ` Adrian Hunter
2026-03-13 14:15 ` Yao Zi
2026-03-09 11:40 ` [PATCH v2 5/7] riscv: dts: spacemit: k1: add SD card controller and pinctrl support Iker Pedrosa
2026-03-09 11:40 ` [PATCH v2 6/7] riscv: dts: spacemit: k1-orangepi-rv2: add PMIC and power infrastructure Iker Pedrosa
2026-03-11 18:27 ` Trevor Gamblin
2026-03-13 0:19 ` Yixun Lan
2026-03-13 9:42 ` Iker Pedrosa
2026-03-13 11:11 ` Yixun Lan
2026-03-13 15:06 ` Iker Pedrosa
2026-03-09 11:40 ` [PATCH v2 7/7] riscv: dts: spacemit: k1-orangepi-rv2: add SD card support with UHS modes Iker Pedrosa
2026-03-13 11:20 ` Anand Moon
2026-03-13 13:54 ` Trevor Gamblin [this message]
2026-03-13 14:42 ` Anand Moon
2026-03-13 17:08 ` Trevor Gamblin
2026-03-09 18:49 ` [PATCH v2 0/7] riscv: spacemit: enable SD card support with UHS modes for OrangePi RV2 Anand Moon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f2eeddcb-712f-4a67-9687-e217f46f9ea8@baylibre.com \
--to=tgamblin@baylibre.com \
--cc=adrian.hunter@intel.com \
--cc=alex@ghiti.fr \
--cc=aou@eecs.berkeley.edu \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dlan@kernel.org \
--cc=ikerpedrosam@gmail.com \
--cc=javierm@redhat.com \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=michael.opdenacker@rootcommit.com \
--cc=palmer@dabbelt.com \
--cc=pjw@kernel.org \
--cc=robh@kernel.org \
--cc=spacemit@lists.linux.dev \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox