From: Chuan Liu <chuan.liu@amlogic.com>
To: Jerome Brunet <jbrunet@baylibre.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Kevin Hilman <khilman@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 17/26] clk: amlogic: use probe helper in mmio based controllers
Date: Thu, 3 Jul 2025 14:29:05 +0800 [thread overview]
Message-ID: <c9489be5-b48a-43dd-8785-b4db91659bc2@amlogic.com> (raw)
In-Reply-To: <20250702-meson-clk-cleanup-24-v1-17-e163c9a1fc21@baylibre.com>
Hi Jerome:
I have no objections for C3 and S4. Thanks!
Reviewed-by: Chuan Liu <chuan.liu@amlogic.com>
On 7/2/2025 11:26 PM, Jerome Brunet wrote:
> [ EXTERNAL EMAIL ]
>
> Factorize the probe function of the mmio based amlogic clock controllers
> using the newly introduced probe helper. This removes a fair amount
> of duplicated code.
>
> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
> ---
> drivers/clk/meson/a1-peripherals.c | 52 +++++++--------------------------
> drivers/clk/meson/a1-pll.c | 52 +++++++--------------------------
> drivers/clk/meson/c3-peripherals.c | 51 +++++---------------------------
> drivers/clk/meson/c3-pll.c | 49 +++++--------------------------
> drivers/clk/meson/meson8-ddr.c | 57 +++++++++---------------------------
> drivers/clk/meson/s4-peripherals.c | 49 +++++--------------------------
> drivers/clk/meson/s4-pll.c | 60 +++++++-------------------------------
> 7 files changed, 66 insertions(+), 304 deletions(-)
>
> diff --git a/drivers/clk/meson/a1-peripherals.c b/drivers/clk/meson/a1-peripherals.c
> index 9e352dba54775c22126ee8bf7861ee1d981d6c88..b2feb8fe4775e38a17d8aa9ce9b992b3e1fb2bb8 100644
> --- a/drivers/clk/meson/a1-peripherals.c
> +++ b/drivers/clk/meson/a1-peripherals.c
> @@ -2057,54 +2057,24 @@ static struct clk_hw *a1_peripherals_hw_clks[] = {
> [CLKID_DMC_SEL2] = &a1_dmc_sel2.hw,
> };
>
> -static const struct regmap_config a1_peripherals_regmap_cfg = {
> - .reg_bits = 32,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = DMC_CLK_CTRL,
> -};
> -
> -static struct meson_clk_hw_data a1_peripherals_clks = {
> - .hws = a1_peripherals_hw_clks,
> - .num = ARRAY_SIZE(a1_peripherals_hw_clks),
> -};
> -
> -static int a1_peripherals_clkc_probe(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - void __iomem *base;
> - struct regmap *map;
> - int clkid, err;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return dev_err_probe(dev, PTR_ERR(base),
> - "can't ioremap resource\n");
> -
> - map = devm_regmap_init_mmio(dev, base, &a1_peripherals_regmap_cfg);
> - if (IS_ERR(map))
> - return dev_err_probe(dev, PTR_ERR(map),
> - "can't init regmap mmio region\n");
> -
> - for (clkid = 0; clkid < a1_peripherals_clks.num; clkid++) {
> - err = devm_clk_hw_register(dev, a1_peripherals_clks.hws[clkid]);
> - if (err)
> - return dev_err_probe(dev, err,
> - "clock[%d] registration failed\n",
> - clkid);
> - }
> -
> - return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, &a1_peripherals_clks);
> -}
> +static const struct meson_clkc_data a1_peripherals_clkc_data = {
> + .hw_clks = {
> + .hws = a1_peripherals_hw_clks,
> + .num = ARRAY_SIZE(a1_peripherals_hw_clks),
> + },
> +};
>
> static const struct of_device_id a1_peripherals_clkc_match_table[] = {
> - { .compatible = "amlogic,a1-peripherals-clkc", },
> + {
> + .compatible = "amlogic,a1-peripherals-clkc",
> + .data = &a1_peripherals_clkc_data,
> + },
> {}
> };
> MODULE_DEVICE_TABLE(of, a1_peripherals_clkc_match_table);
>
> static struct platform_driver a1_peripherals_clkc_driver = {
> - .probe = a1_peripherals_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "a1-peripherals-clkc",
> .of_match_table = a1_peripherals_clkc_match_table,
> diff --git a/drivers/clk/meson/a1-pll.c b/drivers/clk/meson/a1-pll.c
> index 79ef4cbe955326ecedceb68cda7f59bb8882b165..1f82e9c7c14ebeae5d43cf2623c3ab69427a8504 100644
> --- a/drivers/clk/meson/a1-pll.c
> +++ b/drivers/clk/meson/a1-pll.c
> @@ -295,56 +295,24 @@ static struct clk_hw *a1_pll_hw_clks[] = {
> [CLKID_HIFI_PLL] = &a1_hifi_pll.hw,
> };
>
> -static const struct regmap_config a1_pll_regmap_cfg = {
> - .reg_bits = 32,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = ANACTRL_HIFIPLL_STS,
> -};
> -
> -static struct meson_clk_hw_data a1_pll_clks = {
> - .hws = a1_pll_hw_clks,
> - .num = ARRAY_SIZE(a1_pll_hw_clks),
> +static const struct meson_clkc_data a1_pll_clkc_data = {
> + .hw_clks = {
> + .hws = a1_pll_hw_clks,
> + .num = ARRAY_SIZE(a1_pll_hw_clks),
> + },
> };
>
> -static int a1_pll_clkc_probe(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - void __iomem *base;
> - struct regmap *map;
> - int clkid, err;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return dev_err_probe(dev, PTR_ERR(base),
> - "can't ioremap resource\n");
> -
> - map = devm_regmap_init_mmio(dev, base, &a1_pll_regmap_cfg);
> - if (IS_ERR(map))
> - return dev_err_probe(dev, PTR_ERR(map),
> - "can't init regmap mmio region\n");
> -
> - /* Register clocks */
> - for (clkid = 0; clkid < a1_pll_clks.num; clkid++) {
> - err = devm_clk_hw_register(dev, a1_pll_clks.hws[clkid]);
> - if (err)
> - return dev_err_probe(dev, err,
> - "clock[%d] registration failed\n",
> - clkid);
> - }
> -
> - return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get,
> - &a1_pll_clks);
> -}
> -
> static const struct of_device_id a1_pll_clkc_match_table[] = {
> - { .compatible = "amlogic,a1-pll-clkc", },
> + {
> + .compatible = "amlogic,a1-pll-clkc",
> + .data = &a1_pll_clkc_data,
> + },
> {}
> };
> MODULE_DEVICE_TABLE(of, a1_pll_clkc_match_table);
>
> static struct platform_driver a1_pll_clkc_driver = {
> - .probe = a1_pll_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "a1-pll-clkc",
> .of_match_table = a1_pll_clkc_match_table,
> diff --git a/drivers/clk/meson/c3-peripherals.c b/drivers/clk/meson/c3-peripherals.c
> index a09cb1435ab108b2dcc209c6557bcd1988c4ba1a..e9c1ef99be13d0542b8a972ceffe69c8a9977118 100644
> --- a/drivers/clk/meson/c3-peripherals.c
> +++ b/drivers/clk/meson/c3-peripherals.c
> @@ -2091,52 +2091,17 @@ static struct clk_hw *c3_peripherals_hw_clks[] = {
> [CLKID_VAPB] = &c3_vapb.hw,
> };
>
> -static const struct regmap_config c3_peripherals_regmap_cfg = {
> - .reg_bits = 32,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = NNA_CLK_CTRL,
> -};
> -
> -static struct meson_clk_hw_data c3_peripherals_clks = {
> - .hws = c3_peripherals_hw_clks,
> - .num = ARRAY_SIZE(c3_peripherals_hw_clks),
> -};
> -
> -static int c3_peripherals_clkc_probe(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - struct regmap *regmap;
> - void __iomem *base;
> - int clkid, ret;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return PTR_ERR(base);
> -
> - regmap = devm_regmap_init_mmio(dev, base, &c3_peripherals_regmap_cfg);
> - if (IS_ERR(regmap))
> - return PTR_ERR(regmap);
> -
> - for (clkid = 0; clkid < c3_peripherals_clks.num; clkid++) {
> - /* array might be sparse */
> - if (!c3_peripherals_clks.hws[clkid])
> - continue;
> -
> - ret = devm_clk_hw_register(dev, c3_peripherals_clks.hws[clkid]);
> - if (ret) {
> - dev_err(dev, "Clock registration failed\n");
> - return ret;
> - }
> - }
> -
> - return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get,
> - &c3_peripherals_clks);
> -}
> +static const struct meson_clkc_data c3_peripherals_clkc_data = {
> + .hw_clks = {
> + .hws = c3_peripherals_hw_clks,
> + .num = ARRAY_SIZE(c3_peripherals_hw_clks),
> + },
> +};
>
> static const struct of_device_id c3_peripherals_clkc_match_table[] = {
> {
> .compatible = "amlogic,c3-peripherals-clkc",
> + .data = &c3_peripherals_clkc_data,
> },
> { /* sentinel */ }
> };
> @@ -2144,7 +2109,7 @@ static const struct of_device_id c3_peripherals_clkc_match_table[] = {
> MODULE_DEVICE_TABLE(of, c3_peripherals_clkc_match_table);
>
> static struct platform_driver c3_peripherals_clkc_driver = {
> - .probe = c3_peripherals_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "c3-peripherals-clkc",
> .of_match_table = c3_peripherals_clkc_match_table,
> diff --git a/drivers/clk/meson/c3-pll.c b/drivers/clk/meson/c3-pll.c
> index ccfcd4b5be8996592c27df31fa62d4871c826926..dd047d17488c1309dcc4607dfb55582ea978528d 100644
> --- a/drivers/clk/meson/c3-pll.c
> +++ b/drivers/clk/meson/c3-pll.c
> @@ -653,59 +653,24 @@ static struct clk_hw *c3_pll_hw_clks[] = {
> [CLKID_MCLK1] = &c3_mclk1.hw
> };
>
> -static const struct regmap_config c3_pll_regmap_cfg = {
> - .reg_bits = 32,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = ANACTRL_MPLL_CTRL4,
> -};
> -
> -static struct meson_clk_hw_data c3_pll_clks = {
> - .hws = c3_pll_hw_clks,
> - .num = ARRAY_SIZE(c3_pll_hw_clks),
> +static const struct meson_clkc_data c3_pll_clkc_data = {
> + .hw_clks = {
> + .hws = c3_pll_hw_clks,
> + .num = ARRAY_SIZE(c3_pll_hw_clks),
> + },
> };
>
> -static int c3_pll_clkc_probe(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - struct regmap *regmap;
> - void __iomem *base;
> - int clkid, ret;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return PTR_ERR(base);
> -
> - regmap = devm_regmap_init_mmio(dev, base, &c3_pll_regmap_cfg);
> - if (IS_ERR(regmap))
> - return PTR_ERR(regmap);
> -
> - for (clkid = 0; clkid < c3_pll_clks.num; clkid++) {
> - /* array might be sparse */
> - if (!c3_pll_clks.hws[clkid])
> - continue;
> -
> - ret = devm_clk_hw_register(dev, c3_pll_clks.hws[clkid]);
> - if (ret) {
> - dev_err(dev, "Clock registration failed\n");
> - return ret;
> - }
> - }
> -
> - return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get,
> - &c3_pll_clks);
> -}
> -
> static const struct of_device_id c3_pll_clkc_match_table[] = {
> {
> .compatible = "amlogic,c3-pll-clkc",
> + .data = &c3_pll_clkc_data,
> },
> {}
> };
> MODULE_DEVICE_TABLE(of, c3_pll_clkc_match_table);
>
> static struct platform_driver c3_pll_clkc_driver = {
> - .probe = c3_pll_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "c3-pll-clkc",
> .of_match_table = c3_pll_clkc_match_table,
> diff --git a/drivers/clk/meson/meson8-ddr.c b/drivers/clk/meson/meson8-ddr.c
> index 6a9efde9b570d8a2609c118d7d38334a3b4a5dcc..0f93774f73718d29afca87b4c601bd32b9745d30 100644
> --- a/drivers/clk/meson/meson8-ddr.c
> +++ b/drivers/clk/meson/meson8-ddr.c
> @@ -83,57 +83,26 @@ static struct clk_hw *meson8_ddr_hw_clks[] = {
> [DDR_CLKID_DDR_PLL] = &meson8_ddr_pll.hw,
> };
>
> -static struct meson_clk_hw_data meson8_ddr_clks = {
> - .hws = meson8_ddr_hw_clks,
> - .num = ARRAY_SIZE(meson8_ddr_hw_clks),
> -};
> -
> -static const struct regmap_config meson8_ddr_regmap_cfg = {
> - .reg_bits = 8,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = DDR_CLK_STS,
> +static const struct meson_clkc_data meson8_ddr_clkc_data = {
> + .hw_clks = {
> + .hws = meson8_ddr_hw_clks,
> + .num = ARRAY_SIZE(meson8_ddr_hw_clks),
> + },
> };
>
> -static int meson8_ddr_clkc_probe(struct platform_device *pdev)
> -{
> - struct regmap *regmap;
> - void __iomem *base;
> - struct clk_hw *hw;
> - int ret, i;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return PTR_ERR(base);
> -
> - regmap = devm_regmap_init_mmio(&pdev->dev, base,
> - &meson8_ddr_regmap_cfg);
> - if (IS_ERR(regmap))
> - return PTR_ERR(regmap);
> -
> - /* Register all clks */
> - for (i = 0; i < meson8_ddr_clks.num; i++) {
> - hw = meson8_ddr_clks.hws[i];
> -
> - ret = devm_clk_hw_register(&pdev->dev, hw);
> - if (ret) {
> - dev_err(&pdev->dev, "Clock registration failed\n");
> - return ret;
> - }
> - }
> -
> - return devm_of_clk_add_hw_provider(&pdev->dev, meson_clk_hw_get,
> - &meson8_ddr_clks);
> -}
> -
> static const struct of_device_id meson8_ddr_clkc_match_table[] = {
> - { .compatible = "amlogic,meson8-ddr-clkc" },
> - { .compatible = "amlogic,meson8b-ddr-clkc" },
> + {
> + .compatible = "amlogic,meson8-ddr-clkc",
> + .data = &meson8_ddr_clkc_data,
> + }, {
> + .compatible = "amlogic,meson8b-ddr-clkc",
> + .data = &meson8_ddr_clkc_data,
> + },
> { /* sentinel */ }
> };
>
> static struct platform_driver meson8_ddr_clkc_driver = {
> - .probe = meson8_ddr_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "meson8-ddr-clkc",
> .of_match_table = meson8_ddr_clkc_match_table,
> diff --git a/drivers/clk/meson/s4-peripherals.c b/drivers/clk/meson/s4-peripherals.c
> index 9bcd35f12836de5e318fd1ad9c9ae15a2bfc3dd7..fc1500df926d056ce17252987dd91095a8399b55 100644
> --- a/drivers/clk/meson/s4-peripherals.c
> +++ b/drivers/clk/meson/s4-peripherals.c
> @@ -3449,59 +3449,24 @@ static struct clk_hw *s4_peripherals_hw_clks[] = {
> [CLKID_HDCP22_SKPCLK] = &s4_hdcp22_skpclk.hw,
> };
>
> -static const struct regmap_config s4_peripherals_regmap_cfg = {
> - .reg_bits = 32,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = CLKCTRL_DEMOD_CLK_CTRL,
> -};
> -
> -static struct meson_clk_hw_data s4_peripherals_clks = {
> - .hws = s4_peripherals_hw_clks,
> - .num = ARRAY_SIZE(s4_peripherals_hw_clks),
> +static const struct meson_clkc_data s4_peripherals_clkc_data = {
> + .hw_clks = {
> + .hws = s4_peripherals_hw_clks,
> + .num = ARRAY_SIZE(s4_peripherals_hw_clks),
> + },
> };
>
> -static int s4_peripherals_clkc_probe(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - struct regmap *regmap;
> - void __iomem *base;
> - int ret, i;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return dev_err_probe(dev, PTR_ERR(base),
> - "can't ioremap resource\n");
> -
> - regmap = devm_regmap_init_mmio(dev, base, &s4_peripherals_regmap_cfg);
> - if (IS_ERR(regmap))
> - return dev_err_probe(dev, PTR_ERR(regmap),
> - "can't init regmap mmio region\n");
> -
> - for (i = 0; i < s4_peripherals_clks.num; i++) {
> - /* array might be sparse */
> - if (!s4_peripherals_clks.hws[i])
> - continue;
> -
> - ret = devm_clk_hw_register(dev, s4_peripherals_clks.hws[i]);
> - if (ret)
> - return dev_err_probe(dev, ret,
> - "clock[%d] registration failed\n", i);
> - }
> -
> - return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, &s4_peripherals_clks);
> -}
> -
> static const struct of_device_id s4_peripherals_clkc_match_table[] = {
> {
> .compatible = "amlogic,s4-peripherals-clkc",
> + .data = &s4_peripherals_clkc_data,
> },
> {}
> };
> MODULE_DEVICE_TABLE(of, s4_peripherals_clkc_match_table);
>
> static struct platform_driver s4_peripherals_clkc_driver = {
> - .probe = s4_peripherals_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "s4-peripherals-clkc",
> .of_match_table = s4_peripherals_clkc_match_table,
> diff --git a/drivers/clk/meson/s4-pll.c b/drivers/clk/meson/s4-pll.c
> index 6a266bcafd6257937c1de50cbc5606dcc6f8207b..56ce6f566e537a26e932db245ede17d900d9f093 100644
> --- a/drivers/clk/meson/s4-pll.c
> +++ b/drivers/clk/meson/s4-pll.c
> @@ -798,66 +798,26 @@ static const struct reg_sequence s4_pll_init_regs[] = {
> { .reg = ANACTRL_MPLL_CTRL0, .def = 0x00000543 },
> };
>
> -static const struct regmap_config s4_pll_clkc_regmap_cfg = {
> - .reg_bits = 32,
> - .val_bits = 32,
> - .reg_stride = 4,
> - .max_register = ANACTRL_HDMIPLL_CTRL0,
> -};
> -
> -static struct meson_clk_hw_data s4_pll_clks = {
> - .hws = s4_pll_hw_clks,
> - .num = ARRAY_SIZE(s4_pll_hw_clks),
> -};
> -
> -static int s4_pll_clkc_probe(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - struct regmap *regmap;
> - void __iomem *base;
> - int ret, i;
> -
> - base = devm_platform_ioremap_resource(pdev, 0);
> - if (IS_ERR(base))
> - return dev_err_probe(dev, PTR_ERR(base),
> - "can't ioremap resource\n");
> -
> - regmap = devm_regmap_init_mmio(dev, base, &s4_pll_clkc_regmap_cfg);
> - if (IS_ERR(regmap))
> - return dev_err_probe(dev, PTR_ERR(regmap),
> - "can't init regmap mmio region\n");
> -
> - ret = regmap_multi_reg_write(regmap, s4_pll_init_regs, ARRAY_SIZE(s4_pll_init_regs));
> - if (ret)
> - return dev_err_probe(dev, ret,
> - "Failed to init registers\n");
> -
> - /* Register clocks */
> - for (i = 0; i < s4_pll_clks.num; i++) {
> - /* array might be sparse */
> - if (!s4_pll_clks.hws[i])
> - continue;
> -
> - ret = devm_clk_hw_register(dev, s4_pll_clks.hws[i]);
> - if (ret)
> - return dev_err_probe(dev, ret,
> - "clock[%d] registration failed\n", i);
> - }
> -
> - return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get,
> - &s4_pll_clks);
> -}
> +static const struct meson_clkc_data s4_pll_clkc_data = {
> + .hw_clks = {
> + .hws = s4_pll_hw_clks,
> + .num = ARRAY_SIZE(s4_pll_hw_clks),
> + },
> + .init_regs = s4_pll_init_regs,
> + .init_count = ARRAY_SIZE(s4_pll_init_regs),
> +};
>
> static const struct of_device_id s4_pll_clkc_match_table[] = {
> {
> .compatible = "amlogic,s4-pll-clkc",
> + .data = &s4_pll_clkc_data,
> },
> {}
> };
> MODULE_DEVICE_TABLE(of, s4_pll_clkc_match_table);
>
> static struct platform_driver s4_pll_clkc_driver = {
> - .probe = s4_pll_clkc_probe,
> + .probe = meson_clkc_mmio_probe,
> .driver = {
> .name = "s4-pll-clkc",
> .of_match_table = s4_pll_clkc_match_table,
>
> --
> 2.47.2
>
>
> _______________________________________________
> linux-amlogic mailing list
> linux-amlogic@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-amlogic
next prev parent reply other threads:[~2025-07-03 6:29 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-02 15:25 [PATCH 00/26] clk: amlogic: clock controllers clean-up and factorisation Jerome Brunet
2025-07-02 15:25 ` [PATCH 01/26] clk: amlogic: a1-peripherals: naming consistency alignment Jerome Brunet
2025-07-02 15:26 ` [PATCH 02/26] clk: amlogic: a1-pll: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 03/26] clk: amlogic: axg-ao: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 04/26] clk: amlogic: axg: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 05/26] clk: amlogic: c3-peripherals: " Jerome Brunet
2025-07-03 2:51 ` Chuan Liu
2025-07-03 7:48 ` Jerome Brunet
2025-07-03 8:31 ` Chuan Liu
2025-07-03 9:02 ` Jerome Brunet
2025-07-03 9:23 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 06/26] clk: amlogic: c3-pll: " Jerome Brunet
2025-07-03 2:57 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 07/26] clk: amlogic: g12a-ao: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 08/26] clk: amlogic: g12a: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 09/26] clk: amlogic: gxbb-ao: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 10/26] clk: amlogic: gxbb: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 11/26] clk: amlogic: meson8b: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 12/26] clk: amlogic: s4-peripherals: " Jerome Brunet
2025-07-03 3:18 ` Chuan Liu
2025-07-03 7:54 ` Jerome Brunet
2025-07-03 8:00 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 13/26] clk: amlogic: s4-pll: " Jerome Brunet
2025-07-03 3:19 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 14/26] clk: amlogic: meson8-ddr: " Jerome Brunet
2025-07-02 15:26 ` [PATCH 15/26] clk: amlogic: drop meson-clkcee Jerome Brunet
2025-07-02 15:26 ` [PATCH 16/26] clk: amlogic: add probe helper for mmio based controllers Jerome Brunet
2025-07-03 3:29 ` Chuan Liu
2025-07-03 8:35 ` Jerome Brunet
2025-07-02 15:26 ` [PATCH 17/26] clk: amlogic: use probe helper in " Jerome Brunet
2025-07-03 6:29 ` Chuan Liu [this message]
2025-07-02 15:26 ` [PATCH 18/26] clk: amlogic: aoclk: use clkc-utils syscon probe Jerome Brunet
2025-07-02 15:26 ` [PATCH 19/26] clk: amlogic: move PCLK definition to clkc-utils Jerome Brunet
2025-07-02 15:26 ` [PATCH 20/26] clk: amlogic: drop CLK_SET_RATE_PARENT from peripheral clocks Jerome Brunet
2025-07-02 15:26 ` [PATCH 21/26] clk: amlogic: pclk explicitly use CLK_IGNORE_UNUSED Jerome Brunet
2025-07-03 7:05 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 22/26] clk: amlogic: introduce a common pclk definition Jerome Brunet
2025-07-03 7:10 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 23/26] clk: amlogic: use the " Jerome Brunet
2025-07-03 7:16 ` Chuan Liu
2025-07-03 8:39 ` Jerome Brunet
2025-07-02 15:26 ` [PATCH 24/26] clk: amlogic: add composite clock helpers Jerome Brunet
2025-07-03 7:24 ` Chuan Liu
2025-07-03 8:39 ` Jerome Brunet
2025-07-02 15:26 ` [PATCH 25/26] clk: amlogic: align s4 and c3 pwm clock descriptions Jerome Brunet
2025-07-03 7:27 ` Chuan Liu
2025-07-02 15:26 ` [PATCH 26/26] clk: amlogic: c3-peripherals: use helper for basic composite clocks Jerome Brunet
2025-07-03 7:56 ` Chuan Liu
2025-07-03 8:44 ` Jerome Brunet
2025-08-06 7:10 ` [PATCH 00/26] clk: amlogic: clock controllers clean-up and factorisation Chuan Liu
2025-08-25 14:24 ` Jerome Brunet
2025-09-21 11:25 ` Mark Brown
2025-09-21 12:21 ` Martin Blumenstingl
2025-09-21 12:59 ` Jerome Brunet
2025-09-22 9:40 ` Mark Brown
2025-09-22 11:02 ` Martin Blumenstingl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c9489be5-b48a-43dd-8785-b4db91659bc2@amlogic.com \
--to=chuan.liu@amlogic.com \
--cc=jbrunet@baylibre.com \
--cc=khilman@baylibre.com \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=mturquette@baylibre.com \
--cc=neil.armstrong@linaro.org \
--cc=sboyd@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox