Skip to content

Conversation

@LZhaoM
Copy link

@LZhaoM LZhaoM commented Sep 29, 2025

arm64: dts: rk3399: improve rock pi 4 emmc config

This change is based on this commit[0].

However, the property 'enable-strobe-pulldown' has already been set[1].
We just need to set 'mmc-hs400-1_8v' and 'mmc-hs400-enhanced-strobe' in
this change.

For testing compactability, the following eMMCs were tested on Rock
4B:

Foresee 16G, 32G, 64G
Samsung 16G, 32G, 128G
Sandisk 8G, 16G

[0]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f720dd9b8b6d8b2160beda789429d5489ce8a099
[1]: https://github.com/radxa/kernel/commit/b045498f5c40faf27485dcfa9429b8e73f9aa9ce

Signed-off-by: Zhaoming Luo <[email protected]>

Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

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

你有了解过这么改的原因么?
a25a68a
d71e69b
52d3607


&sdhci {
max-frequency = <150000000>;
max-frequency = <384000000>;
Copy link
Member

Choose a reason for hiding this comment

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

@LZhaoM
Copy link
Author

LZhaoM commented Sep 30, 2025

你有了解过这么改的原因么? a25a68a d71e69b 52d3607

@RadxaYuntian
这些 @RadxaStephen 昨天有跟我说,然后让我基于目前的修改提交一个eMMC兼容性测试,提测我已经提交了。接下来怎么处理好些?

@RadxaYuntian
Copy link
Member

至少速率要修复成正确的值,然后提测。如果没问题再合并

This change is based on this commit[0].

However, the property 'enable-strobe-pulldown' has already been set[1].
We just need to set 'mmc-hs400-1_8v' and 'mmc-hs400-enhanced-strobe' in
this change.

For testing compactability, the following eMMCs were tested on Rock
4B:

Foresee 16G, 32G, 64G
Samsung 16G, 32G, 128G
Sandisk 8G, 16G

[0]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f720dd9b8b6d8b2160beda789429d5489ce8a099
[1]: radxa@b045498

Signed-off-by: Zhaoming Luo <[email protected]>
@LZhaoM
Copy link
Author

LZhaoM commented Oct 13, 2025

兼容性测试结果0

EMMC品牌及容量 SD卡启动读写速度 作为系统盘读写速度
江波龙 16G 154 MB/s|84.4 MB/s 150MB/s|81.0 MB/s
江波龙 32G 150 MB/s|123 MB/s 149 MB/s|124 MB/s
江波龙 64G 154 MB/s|105 MB/s 158 MB/s|103 MB/s
三星 16G 153 MB/s|55.5 MB/s 150 MB/s|54.2 MB/s
三星 32G 163 MB/s|105 MB/s 149 MB/s|98.7 MB/s
三星 128G 164 MB/s|132 MB/s 148 MB/s|125 MB/s
闪迪 8G 166 MB/s|42.6 MB/s 容量小 无法刷入镜像
闪迪 16G 165 MB/s|29.0 MB/s 161 MB/s|27.6 MB/s
闪迪 32G NA NA

@LZhaoM LZhaoM requested a review from RadxaYuntian October 13, 2025 02:44
Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

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

另外8g emmc可以用cli镜像吧,不过应该不需要重新测试恶劣

@LZhaoM LZhaoM requested a review from RadxaYuntian October 13, 2025 06:38
@RadxaStephen
Copy link
Member

Merged.

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.

3 participants