Le lun. 9 mars 2026 à 22:29, David Lechner <[email protected]> a écrit :
>
> The MediaTek clock driver currently have two ways to indicate the parent
> tree that contains the parent clock. This adds to code complexity and
> makes it more difficult to know the "right thing" to do in new drivers.
>
> The plan here is to convert all mux clocks to use the more robust option
> of using struct mtk_parent to specify parent clocks. Then we can remove
> dead and duplicate code and macros.
>
> The conversion to the static const struct mtk_parent arrays was done
> by script to minimize the chance of error (and save time).
>
> Signed-off-by: David Lechner <[email protected]>
> ---
> Changes in v2:
> - Rebased on updated dependencies.
> - Link to v1: 
> https://lore.kernel.org/r/20260302-clk-mtk-unify-mux-parents-v1-0-32f45085f...@baylibre.com
>
> ---
> David Lechner (13):
>       clk: mediatek: mt7622: convert to struct mtk_parent

Hello David,

Is there a reason to have 2 different series to convert to struct mtk_parent?
You are already doing this kind of cleanup in [1]. Can we merge into
one single series? Specifically we end up doing this cleanup into two
commits for mt7622..

[1]: 
https://patchwork.ozlabs.org/project/uboot/cover/20260306-clk-mtk-remove-clk-bypass-xtal-flag-v2-0-b253b49f1...@baylibre.com/

Thanks
Julien
>       clk: mediatek: mt7981: convert to struct mtk_parent
>       clk: mediatek: mt7987: convert to struct mtk_parent
>       clk: mediatek: mt7988: convert to struct mtk_parent
>       clk: mediatek: mt8183: convert to struct mtk_parent
>       clk: mediatek: mt8188: convert to struct mtk_parent
>       clk: mediatek: mt8195: convert to struct mtk_parent
>       clk: mediatek: mt8365: convert to struct mtk_parent
>       clk: mediatek: mt8512: convert to struct mtk_parent
>       clk: mediatek: mt8516: convert to struct mtk_parent
>       clk: mediatek: mt8518: convert to struct mtk_parent
>       clk: mediatek: remove CLK_PARENT_MIXED flag
>       clk: mediatek: refactor duplicate *_mux_get_rate()
>
>  drivers/clk/mediatek/clk-mt7622.c |   88 +-
>  drivers/clk/mediatek/clk-mt7623.c |  147 ++-
>  drivers/clk/mediatek/clk-mt7629.c |   71 +-
>  drivers/clk/mediatek/clk-mt7981.c |  231 ++--
>  drivers/clk/mediatek/clk-mt7986.c |   10 +-
>  drivers/clk/mediatek/clk-mt7987.c |   75 +-
>  drivers/clk/mediatek/clk-mt7988.c |   79 +-
>  drivers/clk/mediatek/clk-mt8183.c |  584 +++++------
>  drivers/clk/mediatek/clk-mt8188.c | 1522 +++++++++++++--------------
>  drivers/clk/mediatek/clk-mt8189.c |   12 +-
>  drivers/clk/mediatek/clk-mt8195.c | 1349 ++++++++++++------------
>  drivers/clk/mediatek/clk-mt8365.c |  450 ++++----
>  drivers/clk/mediatek/clk-mt8512.c |  620 +++++------
>  drivers/clk/mediatek/clk-mt8516.c |  704 ++++++-------
>  drivers/clk/mediatek/clk-mt8518.c | 2088 
> ++++++++++++++++++-------------------
>  drivers/clk/mediatek/clk-mtk.c    |  104 +-
>  drivers/clk/mediatek/clk-mtk.h    |   62 +-
>  17 files changed, 4106 insertions(+), 4090 deletions(-)
> ---
> base-commit: 36add050eea439f0b2a15e4ea0d3a8e21216f159
> change-id: 20260302-clk-mtk-unify-mux-parents-1bf35aeabf00
> prerequisite-message-id: 
> <[email protected]>
> prerequisite-patch-id: 81f2dd4fd7c1632736c08180d6b9f6aef5fe1e7f
> prerequisite-patch-id: 60f0509a5f36c2828316fa3e27dfd8f4d218eb27
> prerequisite-patch-id: 237c548edf19f9cf734b1ac3b9252a4f06d8cf29
> prerequisite-patch-id: 22de64f70e8d1ba5e17f8b31f461118b35e54f30
> prerequisite-patch-id: 3d5661f9e55292a30f57617848a7b10e67844402
> prerequisite-patch-id: ccbc73b2bc1dc41beb295604088d63773cea15ef
> prerequisite-patch-id: aa76b6b028374f20551955eafe56a1e0c1daa75d
> prerequisite-message-id: 
> <20260306-clk-mtk-remove-clk-bypass-xtal-flag-v2-0-b253b49f1...@baylibre.com>
> prerequisite-patch-id: 15ac11765ddddcf378e0e227a9fb5876636aa8e6
> prerequisite-patch-id: 90b4781a99ea5eb40eb6c943dee86dd8a9fabe06
> prerequisite-patch-id: 150153a03fa40f8f9d790526512f7341a7d95eba
> prerequisite-patch-id: bdc28e831956d0a487c62e11d1442a51b9548c25
> prerequisite-patch-id: 08b3b97b5870fa4c5eb435f5b19762481a94c058
> prerequisite-patch-id: 291f87bf8033440b0c79a03055cec7e1317216e2
> prerequisite-patch-id: d3ff487de22ca508a65613bebf8cbe0e61005a31
> prerequisite-patch-id: f0e2edcbe17f6bea4bd4e6ceea49cbf73b7568d7
> prerequisite-patch-id: 0700411359ab89ebd24c4c3d17a5864e41289547
>
> Best regards,
> --
> David Lechner <[email protected]>
>

Reply via email to