[PATCH v2 00/24] clk: Add clk_resolve_parent_clk() and fix up iMX clock drivers

[Peng Fan]

On Sun, Mar 23, 2025 at 04:58:29PM +0100, Marek Vasut wrote:
>Add clk_resolve_parent_clk() to resolve parent clock udevice name
>based on clock-names DT property. This is used in SoC clock drivers
>to look up the clock name in clock tables, which matches a clock
>name in DT clock-names property, and convert it into udevice name
>which is used by U-Boot clock framework to look up parent clock in
>e.g. clk_register() using uclass_get_device_by_name(UCLASS_CLK,
>parent_name, &parent);
>
>Pass struct udevice pointer through the various iMX clock drivers.

Thanks for the effort. 

Acked-by: Peng Fan <peng.fan at nxp.com>


BTW: this could only fix partial of the current clk code.

The current clock framework is kind of a bit mess. There is
more issue is not support multiple clk provider, name should be unique.

I was thinking to drop using device for clk, following linux clk
driver to use a clk_hw structure for each clk.

Regards,
Peng

>
>Marek Vasut (24):
>  clk: Add clk_resolve_parent_clk()
>  clk: clk-mux: Fold clk_register_mux()
>  clk: clk-mux: Use struct udevice instead of struct device
>  clk: clk-mux: Resolve parent clock by name
>  clk: imx: Pass struct udevice into imx_clk_mux*()
>  clk: imx: Pass struct udevice to clk_register_mux()
>  clk: clk-gate: Use struct udevice instead of struct device
>  clk: clk-gate: Resolve parent clock by name
>  clk: imx: gate2: Use struct udevice instead of struct device
>  clk: imx: gate2: Resolve parent clock by name
>  clk: imx: Pass struct udevice into imx_clk_gate*()
>  clk: imx: Pass struct udevice to clk_register_gate*()
>  clk: clk-composite: Use struct udevice instead of struct device
>  clk: clk-composite: Resolve parent clock by name
>  clk: imx: Pass struct udevice into imx_clk_composite*()
>  clk: imx: Convert clock-osc-* back to osc_*
>  clk: imx: Pass struct udevice into imx_clk_pllv3*()
>  clk: imx: pllv3: Resolve parent clock by name
>  clk: clk-divider: Use struct udevice instead of struct device
>  clk: imx: Pass struct udevice into imx_clk_divider*()
>  clk: clk-divider: Resolve parent clock by name
>  clk: clk-fixed-factor: Use struct udevice instead of struct device
>  clk: clk-fixed-factor: Resolve parent clock by name
>  clk: imx: Pass struct udevice into imx_clk_fixed_factor*()
>
> drivers/clk/clk-composite.c        |   4 +-
> drivers/clk/clk-divider.c          |   7 +-
> drivers/clk/clk-fixed-factor.c     |   6 +-
> drivers/clk/clk-gate.c             |   5 +-
> drivers/clk/clk-mux.c              |  47 +---
> drivers/clk/clk-uclass.c           |  18 ++
> drivers/clk/imx/clk-composite-8m.c |   4 +-
> drivers/clk/imx/clk-gate2.c        |   5 +-
> drivers/clk/imx/clk-imx6q.c        |  80 +++----
> drivers/clk/imx/clk-imx8mm.c       | 254 +++++++++++----------
> drivers/clk/imx/clk-imx8mn.c       | 246 ++++++++++----------
> drivers/clk/imx/clk-imx8mp.c       | 348 ++++++++++++++---------------
> drivers/clk/imx/clk-imx8mq.c       | 226 +++++++++----------
> drivers/clk/imx/clk-imx93.c        |   8 +-
> drivers/clk/imx/clk-imxrt1020.c    |  42 ++--
> drivers/clk/imx/clk-imxrt1050.c    |  78 +++----
> drivers/clk/imx/clk-imxrt1170.c    |  30 +--
> drivers/clk/imx/clk-pllv3.c        |   9 +-
> drivers/clk/imx/clk.h              | 116 +++++-----
> include/clk.h                      |   9 +
> include/linux/clk-provider.h       |  10 +-
> 21 files changed, 783 insertions(+), 769 deletions(-)
>
>---
>Cc: Adam Ford <aford173 at gmail.com>
>Cc: Christoph Niedermaier <cniedermaier at dh-electronics.com>
>Cc: Dong Aisheng <aisheng.dong at nxp.com>
>Cc: Fabio Estevam <festevam at denx.de>
>Cc: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
>Cc: Michael Trimarchi <michael at amarulasolutions.com>
>Cc: Peng Fan <peng.fan at nxp.com>
>Cc: Tim Harvey <tharvey at gateworks.com>
>Cc: Tom Rini <trini at konsulko.com>
>Cc: u-boot at lists.denx.de
>Cc: uboot-imx at nxp.com
>
>-- 
>2.47.2
>