[PATCH] mmc: dw_mmc-rockchip: Fix wrong internal phase calculate

[Ulf Hansson]

On Tue, 4 Nov 2025 at 04:51, Shawn Lin <shawn.lin at rock-chips.com> wrote:
>
> ciu clock is 2 times of io clock, but the sample clk used is
> derived from io clock provided to the card. So we should use
> io clock to calculate the phase.
>
> Fixes: 59903441f5e4 ("mmc: dw_mmc-rockchip: Add internal phase support")
> Signed-off-by: Shawn Lin <shawn.lin at rock-chips.com>

Applied for fixes and by adding a stable-tag, thanks!

Kind regards
Uffe


> ---
>
>  drivers/mmc/host/dw_mmc-rockchip.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/host/dw_mmc-rockchip.c b/drivers/mmc/host/dw_mmc-rockchip.c
> index 82dd906..6813549 100644
> --- a/drivers/mmc/host/dw_mmc-rockchip.c
> +++ b/drivers/mmc/host/dw_mmc-rockchip.c
> @@ -42,7 +42,7 @@ struct dw_mci_rockchip_priv_data {
>   */
>  static int rockchip_mmc_get_internal_phase(struct dw_mci *host, bool sample)
>  {
> -       unsigned long rate = clk_get_rate(host->ciu_clk);
> +       unsigned long rate = clk_get_rate(host->ciu_clk) / RK3288_CLKGEN_DIV;
>         u32 raw_value;
>         u16 degrees;
>         u32 delay_num = 0;
> @@ -85,7 +85,7 @@ static int rockchip_mmc_get_phase(struct dw_mci *host, bool sample)
>
>  static int rockchip_mmc_set_internal_phase(struct dw_mci *host, bool sample, int degrees)
>  {
> -       unsigned long rate = clk_get_rate(host->ciu_clk);
> +       unsigned long rate = clk_get_rate(host->ciu_clk) / RK3288_CLKGEN_DIV;
>         u8 nineties, remainder;
>         u8 delay_num;
>         u32 raw_value;
> --
> 2.7.4
>