Skip to content

Commit

Permalink
phy/rockchip: inno-hdmi: round fractal pixclock in rk3328 recalc_rate
Browse files Browse the repository at this point in the history
[ Upstream commit d5ef343 ]

inno_hdmi_phy_rk3328_clk_recalc_rate() is returning a rate not found
in the pre pll config table when the fractal divider is used.
This can prevent proper power_on because a tmdsclock for the new rate
is not found in the pre pll config table.

Fix this by saving and returning a rounded pixel rate that exist
in the pre pll config table.

Fixes: 53706a1 ("phy: add Rockchip Innosilicon hdmi phy")
Signed-off-by: Zheng Yang <zhengyang@rock-chips.com>
Signed-off-by: Jonas Karlman <jonas@kwiboo.se>
Link: https://lore.kernel.org/r/20230615171005.2251032-3-jonas@kwiboo.se
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
Zheng Yang authored and gregkh committed Sep 13, 2023
1 parent 0d86292 commit 8f0f545
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions drivers/phy/rockchip/phy-rockchip-inno-hdmi.c
Expand Up @@ -745,10 +745,12 @@ unsigned long inno_hdmi_phy_rk3328_clk_recalc_rate(struct clk_hw *hw,
do_div(vco, (nd * (no_a == 1 ? no_b : no_a) * no_d * 2));
}

inno->pixclock = vco;
dev_dbg(inno->dev, "%s rate %lu\n", __func__, inno->pixclock);
inno->pixclock = DIV_ROUND_CLOSEST((unsigned long)vco, 1000) * 1000;

return vco;
dev_dbg(inno->dev, "%s rate %lu vco %llu\n",
__func__, inno->pixclock, vco);

return inno->pixclock;
}

static long inno_hdmi_phy_rk3328_clk_round_rate(struct clk_hw *hw,
Expand Down

0 comments on commit 8f0f545

Please sign in to comment.