From: Yao Zi <ziyao@disroot.org>
To: Jonas Karlman <jonas@kwiboo.se>, Heiko Stuebner <heiko@sntech.de>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>
Cc: Chukun Pan <amadeus@jmu.edu.cn>,
linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/6] arm64: dts: rockchip: Add Radxa ROCK 2A/2F
Date: Wed, 9 Jul 2025 04:25:34 +0000 [thread overview]
Message-ID: <aG3vPsUd-FPkhi-S@pie.lan> (raw)
In-Reply-To: <20250708224921.2254116-3-jonas@kwiboo.se>
On Tue, Jul 08, 2025 at 10:48:52PM +0000, Jonas Karlman wrote:
> The ROCK 2A and ROCK 2F is a high-performance single board computer
> developed by Radxa, based on the Rockchip RK3528A SoC.
>
> Add initial device tree for the Radxa ROCK 2A and ROCK 2F boards.
>
> Signed-off-by: Jonas Karlman <jonas@kwiboo.se>
> ---
> Schematics:
> - https://dl.radxa.com/rock2/2a/v1.2/radxa_rock_2a_v1.2_schematic.pdf
> - https://dl.radxa.com/rock2/2f/radxa_rock2f_v1.01_schematic.pdf
> ---
> arch/arm64/boot/dts/rockchip/Makefile | 2 +
> .../boot/dts/rockchip/rk3528-rock-2.dtsi | 292 ++++++++++++++++++
> .../boot/dts/rockchip/rk3528-rock-2a.dts | 82 +++++
> .../boot/dts/rockchip/rk3528-rock-2f.dts | 10 +
> 4 files changed, 386 insertions(+)
> create mode 100644 arch/arm64/boot/dts/rockchip/rk3528-rock-2.dtsi
> create mode 100644 arch/arm64/boot/dts/rockchip/rk3528-rock-2a.dts
> create mode 100644 arch/arm64/boot/dts/rockchip/rk3528-rock-2f.dts
While testing the patch on my Rock 2A board, I noticed one of my SDcard
that works perfectly on Radxa E20C and NanoPi Zero 2 cannot be correctly
read out under UHS-125-SDR mode,
# dd if=/dev/mmcblk1 of=/dev/null bs=4M count=4
[ 18.616828] mmc_host mmc1: Bus speed (slot 0) = 400000Hz (slot req 400000Hz, actual 400000HZ div = 0)
[ 19.193315] mmc1: Skipping voltage switch
[ 19.202046] mmc1: tried to HW reset card, got error -110
[ 19.213312] mmcblk1: recovery failed!
[ 19.213709] I/O error, dev mmcblk1, sector 0 op 0x0:(READ) flags 0x80700 phys_seg 256 prio class 0
[ 19.225201] mmcblk1: recovery failed!
[ 19.225530] I/O error, dev mmcblk1, sector 0 op 0x0:(READ) flags 0x0 phys_seg 1 prio class 0
[ 19.226283] Buffer I/O error on dev mmcblk1, logical block 0, async page read
dd: /dev/mmcblk1: I/O error
which could be reproduced stably.
the SDMMC controller issued interesting messages during the tuning
process,
[ 0.665246] mmc_host mmc1: Bus speed (slot 0) = 148500000Hz (slot req 150000000Hz, actual 148500000HZ div = 0)
[ 0.851940] dwmmc_rockchip ffc30000.mmc: All phases work, using default phase 90.
but actually it doesn't work with phase = 90. If the frequency is
limited to 100MHz with max-frequency = <100000000> instead of the
default 150MHz, tuning results in a very different phase,
[ 0.665483] mmc_host mmc1: Bus speed (slot 0) = 99600000Hz (slot req 100000000Hz, actual 99600000HZ div = 0)
[ 1.166340] dwmmc_rockchip ffc30000.mmc: Successfully tuned phase to 141
and the card works, too. If I set rockchip,default-sample-phase to 141
in devicetree, the card could work at full 150MHz as well.
I think there's something wrong with the tuning process, or the board's
design cannot always run reliably at 150MHz.
Could you reproduce similar failures on Radxa 2A? If so, it may be
necessary to lower the SDMMC's maximum frequency for the board.
Regards,
Yao Zi
next prev parent reply other threads:[~2025-07-09 4:32 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-08 22:48 [PATCH 0/6] arm64: dts: rockchip: Add ROCK 2A/2F, Sige1 and NanoPi Zero2 Jonas Karlman
2025-07-08 22:48 ` [PATCH 1/6] dt-bindings: arm: rockchip: Add Radxa ROCK 2A/2F Jonas Karlman
2025-07-08 22:48 ` [PATCH 2/6] arm64: dts: " Jonas Karlman
2025-07-09 4:25 ` Yao Zi [this message]
2025-07-09 23:56 ` Jonas Karlman
2025-07-10 16:41 ` Yao Zi
2025-07-10 20:07 ` Jonas Karlman
2025-07-10 2:45 ` Rob Herring (Arm)
2025-07-08 22:48 ` [PATCH 3/6] dt-bindings: arm: rockchip: Add ArmSoM Sige1 Jonas Karlman
2025-07-08 22:48 ` [PATCH 4/6] arm64: dts: " Jonas Karlman
2025-07-09 7:00 ` Chukun Pan
2025-07-10 0:00 ` Jonas Karlman
2025-07-08 22:48 ` [PATCH 5/6] dt-bindings: arm: rockchip: Add FriendlyElec NanoPi Zero2 Jonas Karlman
2025-07-08 22:48 ` [PATCH 6/6] arm64: dts: " Jonas Karlman
2025-07-09 3:30 ` Yao Zi
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=aG3vPsUd-FPkhi-S@pie.lan \
--to=ziyao@disroot.org \
--cc=amadeus@jmu.edu.cn \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=heiko@sntech.de \
--cc=jonas@kwiboo.se \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=robh@kernel.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;
as well as URLs for NNTP newsgroup(s).