[PATCH v2] Revert "mmc: sdhci-cadence: trigger tuning for SD HS mode on SD6HC (v6) PHY"
Peng Fan
peng.fan at oss.nxp.com
Mon Jul 6 08:33:15 CEST 2026
On Sun, Jun 28, 2026 at 11:16:15PM -0700, Tanmay Kathpalia wrote:
>This reverts commit b42c67188c14 ("mmc: sdhci-cadence: trigger tuning
>for SD HS mode on SD6HC (v6) PHY").
>
>The reverted patch introduced several issues:
>
>1. Non-standard tuning trigger: The SD Physical Layer Specification
> only mandates execute_tuning for SDR50 and SDR104 UHS-I modes.
> Triggering tuning for SD High Speed mode is outside the spec and
> is handled via a non-standard set_ios_post callback rather than
> through the established SDHCI framework tuning path.
>
>2. Non-standard device tree property: The patch introduced a new
> "cdns,sd-hs-tuning" DT property to opt into SD HS tuning. This
> is not aligned with existing DT bindings and bypasses the standard
> MMC capability negotiation mechanism.
>
>3. Incorrect tunable mode allowlist: The sdhci_cdns6_mode_is_tuned()
> function includes SD_HS, UHS_SDR50, and MMC_HS_400_ES as tunable
> modes. According to the Cadence SD6HC IP User Guide (section 7.5.2,
> Figure 18), tuning is only required for UHS-I SDR104 (SD) and
> HS200 (eMMC). SD High Speed, UHS-I SDR50, and DDR50 only require
> a PHY settings update from the pre-calculation script, not the
> tuning procedure. HS400 transitions through HS200 and reuses its
> tuned DLL value with a partial settings update. HS400ES only
> requires a plain settings update from the calculation script with
> no dependency on HS200 tuning.
>
>4. Tuned state management outside the framework: The patch manually
> tracks tuned DLL state (tuned_mode, tuned_dll_slave_ctrl) and
> restores it across PHY reconfigurations. This duplicates
> responsibility that belongs in the core MMC tuning framework and
> adds unnecessary complexity to the driver.
>
>Reverting to realign the driver with the IP documentation and the SD
>Physical Layer Specification.
>
>Signed-off-by: Tanmay Kathpalia <tanmay.kathpalia at altera.com>
Applied.
Thanks
Peng
More information about the U-Boot
mailing list