mirrors/u-boot
1
0
Fork 0
mirror of https://source.denx.de/u-boot/u-boot.git synced 2026-05-05 12:46:14 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

clk: mediatek: fix parent rate lookup for fixed PLL clocks

Browse Source 
The refactoring in commit 00d0ff7f81bf ("clk: mediatek: refactor parent
rate lookup functions") introduced a regression where fixed PLL clocks
using mtk_clk_fixed_pll_ops are not properly recognized as valid parents
in the CLK_PARENT_APMIXED case.

Fixed PLL clocks are implemented using mtk_clk_fixed_pll_ops instead of
mtk_clk_apmixedsys_ops, but they can also serve as parent clocks in the
APMIXED domain. The parent lookup function needs to check for both
driver ops to properly resolve the parent clock device.

Add mtk_clk_fixed_pll_ops checks alongside mtk_clk_apmixedsys_ops checks
in mtk_find_parent_rate() to restore support for fixed PLL parent clocks.

Fixes: 00d0ff7f81bf ("clk: mediatek: refactor parent rate lookup functions")
Signed-off-by: Sam Shih <sam.shih@mediatek.com>
Signed-off-by: Weijie Gao <weijie.gao@mediatek.com>
Link: https://patch.msgid.link/923e50db696d910803828cd26b0ca0fbbfe11570.1776326933.git.weijie.gao@mediatek.com
Signed-off-by: David Lechner <dlechner@baylibre.com>
This commit is contained in:
Sam Shih 2026-04-16 16:23:10 +08:00 committed by David Lechner
parent 5576522219
commit ad3ea453d8
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/clk/mediatek/clk-mtk.c
View File

@ -216,11 +216,14 @@ static ulong mtk_find_parent_rate(struct mtk_clk_priv *priv, struct clk *clk,
switch (flags & CLK_PARENT_MASK) {
case CLK_PARENT_APMIXED:
/* APMIXEDSYS can be parent or grandparent. */
if (dev_get_driver_ops(clk->dev) == &mtk_clk_apmixedsys_ops)
if (dev_get_driver_ops(clk->dev) == &mtk_clk_apmixedsys_ops ||
dev_get_driver_ops(clk->dev) == &mtk_clk_fixed_pll_ops)
parent_dev = clk->dev;
else if (dev_get_driver_ops(priv->parent) == &mtk_clk_apmixedsys_ops)
else if (dev_get_driver_ops(priv->parent) == &mtk_clk_apmixedsys_ops ||
dev_get_driver_ops(priv->parent) == &mtk_clk_fixed_pll_ops)
parent_dev = priv->parent;
else if (dev_get_driver_ops(dev_get_parent(priv->parent)) == &mtk_clk_apmixedsys_ops)
else if (dev_get_driver_ops(dev_get_parent(priv->parent)) == &mtk_clk_apmixedsys_ops ||
dev_get_driver_ops(dev_get_parent(priv->parent)) == &mtk_clk_fixed_pll_ops)
parent_dev = dev_get_parent(priv->parent);
else
return -EINVAL;