Skip to content
  • John Stultz's avatar
    arm64: dts: hikey: Fix eMMC corruption regression · 9c6d26df
    John Stultz authored
    This patch is a partial revert of
    commit abd7d097
    
     ("arm64: dts: hikey: Enable HS200 mode on eMMC")
    
    which has been causing eMMC corruption on my HiKey board.
    
    Symptoms usually looked like:
    
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    ...
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    mmc0: new HS200 MMC card at address 0001
    ...
    dwmmc_k3 f723d000.dwmmc0: Unexpected command timeout, state 3
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    print_req_error: I/O error, dev mmcblk0, sector 8810504
    Aborting journal on device mmcblk0p10-8.
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    mmc_host mmc0: Bus speed (slot 0) = 24800000Hz (slot req 400000Hz, actual 400000HZ div = 31)
    mmc_host mmc0: Bus speed (slot 0) = 148800000Hz (slot req 150000000Hz, actual 148800000HZ div = 0)
    EXT4-fs error (device mmcblk0p10): ext4_journal_check_start:61: Detected aborted journal
    EXT4-fs (mmcblk0p10): Remounting filesystem read-only
    
    And quite often this would result in a disk that wouldn't properly
    boot even with older kernels.
    
    It seems the max-frequency property added by the above patch is
    causing the problem, so remove it.
    
    Cc: Ryan Grachek <ryan@edited.us>
    Cc: Wei Xu <xuwei5@hisilicon.com>
    Cc: Arnd Bergmann <arnd@arndb.de>
    Cc: Ulf Hansson <ulf.hansson@linaro.org>
    Cc: YongQin Liu <yongqin.liu@linaro.org>
    Cc: Leo Yan <leo.yan@linaro.org>
    Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
    Tested-by: default avatarLeo Yan <leo.yan@linaro.org>
    Signed-off-by: default avatarWei Xu <xuwei04@gmail.com>
    9c6d26df