Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add some patch to repair eMMC startup issues #7538

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

CodeChenL
Copy link

@CodeChenL CodeChenL commented Dec 3, 2024

Description

The main content is to modify the frequency of the SDIO and reset when the start failure is added to the uboot to repair a series of compatibility issues of Samsung eMMC, such as: the launch failure and IO error

uboot boot failure issue: https://forum.radxa.com/t/rzero-autoboot-from-emmc/24764

io error issue:

root@radxa-zero:~# sudo /usr/sbin/cryptsetup luksFormat --type luks2 --batch-mode /dev/mmcblk1
Enter passphrase for /dev/mmcblk1: 
Device wipe error, offset 32768.
Cannot wipe header on device /dev/mmcblk1.
root@radxa-zero:~# sudo dmesg | grep blk
......
[ 1546.780637] I/O error, dev mmcblk1, sector 64 op 0x1:(WRITE) flags 0xc800 phys_seg 64 prio class 0
[ 1546.781847] I/O error, dev mmcblk1, sector 575 op 0x1:(WRITE) flags 0xc800 phys_seg 65 prio class 0
[ 1546.782272] I/O error, dev mmcblk1, sector 1086 op 0x1:(WRITE) flags 0xc800 phys_seg 65 prio class 0
[ 1546.784501] I/O error, dev mmcblk1, sector 1597 op 0x1:(WRITE) flags 0xc800 phys_seg 65 prio class 0

How Has This Been Tested?

  • Performed 200 reboot tests, all rebooted successfully
  • Use echo -n "rock" | /usr/sbin/cryptsetup luksFormat --type luks2 --batch-mode /dev/{emmc_blk} pressing eMMC for 48 hours, the kernel is not reported to IO Error

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@github-actions github-actions bot added size/medium PR with more then 50 and less then 250 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Dec 3, 2024
@SteeManMI
Copy link
Contributor

I see that you are only patching the 6.6 current kernel. Does this issue not exist in the 6.12 edge kernel, which will likely become the new current version soon.

@CodeChenL
Copy link
Author

I see that you are only patching the 6.6 current kernel. Does this issue not exist in the 6.12 edge kernel, which will likely become the new current version soon.

I will test it

@CodeChenL
Copy link
Author

I see that you are only patching the 6.6 current kernel. Does this issue not exist in the 6.12 edge kernel, which will likely become the new current version soon.

There are also this problem after testing the edge kernel, and it also can be repaired through this patch

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 02 Milestone: First quarter release and removed Needs review Seeking for review labels Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
02 Milestone: First quarter release Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/medium PR with more then 50 and less then 250 lines
Development

Successfully merging this pull request may close these issues.

3 participants