From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
Cc: Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Ludovic Desroches <ludovic.desroches@microchip.com>,
Rob Herring <robh+dt@kernel.org>,
linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v6 3/3] clk: at91: allow setting all PMC clock parents via DT
Date: Mon, 4 May 2020 23:44:57 +0200 [thread overview]
Message-ID: <20200504214457.GN34497@piout.net> (raw)
In-Reply-To: <5f41e1872b1a9a7f0f50f9bb47e4378e67e55e74.1588623391.git.mirq-linux@rere.qmqm.pl>
On 04/05/2020 22:19:18+0200, Michał Mirosław wrote:
> We need to have clocks accessible via phandle to select them
> as peripheral clock parent using assigned-clock-parents in DT.
> Add support for PLLACK/PLLBCK/AUDIOPLLCK clocks where available.
>
> Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> ---
> v2: rebase to clk/clk-at91 branch
> v3: no changes
> v4: extend to whole family
> v5: fix copy-and-paste bugs for SAMA5D3/4
> v6: make AUDIOPLL's id separate to PLLB
> ---
> drivers/clk/at91/at91rm9200.c | 6 +++++-
> drivers/clk/at91/at91sam9260.c | 6 +++++-
> drivers/clk/at91/at91sam9g45.c | 4 +++-
> drivers/clk/at91/at91sam9n12.c | 6 +++++-
> drivers/clk/at91/at91sam9rl.c | 4 +++-
> drivers/clk/at91/at91sam9x5.c | 4 +++-
> drivers/clk/at91/sam9x60.c | 4 +++-
> drivers/clk/at91/sama5d2.c | 6 +++++-
> drivers/clk/at91/sama5d3.c | 4 +++-
> drivers/clk/at91/sama5d4.c | 4 +++-
> include/dt-bindings/clock/at91.h | 3 +++
> 11 files changed, 41 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/clk/at91/at91rm9200.c b/drivers/clk/at91/at91rm9200.c
> index 8da88e9a95d8..38bdb4981315 100644
> --- a/drivers/clk/at91/at91rm9200.c
> +++ b/drivers/clk/at91/at91rm9200.c
> @@ -98,7 +98,7 @@ static void __init at91rm9200_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91rm9200_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91rm9200_pmc = pmc_data_allocate(PMC_PLLBCK + 1,
> nck(at91rm9200_systemck),
> nck(at91rm9200_periphck), 0, 4);
> if (!at91rm9200_pmc)
> @@ -123,12 +123,16 @@ static void __init at91rm9200_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91rm9200_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_pll(regmap, "pllbck", "mainck", 1,
> &at91rm9200_pll_layout,
> &rm9200_pll_characteristics);
> if (IS_ERR(hw))
> goto err_free;
>
> + at91rm9200_pmc->chws[PMC_PLLBCK] = hw;
> +
> parent_names[0] = slowxtal_name;
> parent_names[1] = "mainck";
> parent_names[2] = "pllack";
> diff --git a/drivers/clk/at91/at91sam9260.c b/drivers/clk/at91/at91sam9260.c
> index 7e5ff252fffc..6d0723aa8b13 100644
> --- a/drivers/clk/at91/at91sam9260.c
> +++ b/drivers/clk/at91/at91sam9260.c
> @@ -352,7 +352,7 @@ static void __init at91sam926x_pmc_setup(struct device_node *np,
> if (IS_ERR(regmap))
> return;
>
> - at91sam9260_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9260_pmc = pmc_data_allocate(PMC_PLLBCK + 1,
> ndck(data->sck, data->num_sck),
> ndck(data->pck, data->num_pck),
> 0, data->num_progck);
> @@ -399,12 +399,16 @@ static void __init at91sam926x_pmc_setup(struct device_node *np,
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9260_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_pll(regmap, "pllbck", "mainck", 1,
> data->pllb_layout,
> data->pllb_characteristics);
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9260_pmc->chws[PMC_PLLBCK] = hw;
> +
> parent_names[0] = slck_name;
> parent_names[1] = "mainck";
> parent_names[2] = "pllack";
> diff --git a/drivers/clk/at91/at91sam9g45.c b/drivers/clk/at91/at91sam9g45.c
> index 5d18eb04c218..9873b583c260 100644
> --- a/drivers/clk/at91/at91sam9g45.c
> +++ b/drivers/clk/at91/at91sam9g45.c
> @@ -115,7 +115,7 @@ static void __init at91sam9g45_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91sam9g45_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9g45_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(at91sam9g45_systemck),
> nck(at91sam9g45_periphck), 0, 2);
> if (!at91sam9g45_pmc)
> @@ -143,6 +143,8 @@ static void __init at91sam9g45_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9g45_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/at91sam9n12.c b/drivers/clk/at91/at91sam9n12.c
> index 3a2564c2f724..630dc5d87171 100644
> --- a/drivers/clk/at91/at91sam9n12.c
> +++ b/drivers/clk/at91/at91sam9n12.c
> @@ -128,7 +128,7 @@ static void __init at91sam9n12_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91sam9n12_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9n12_pmc = pmc_data_allocate(PMC_PLLBCK + 1,
> nck(at91sam9n12_systemck), 31, 0, 2);
> if (!at91sam9n12_pmc)
> return;
> @@ -162,11 +162,15 @@ static void __init at91sam9n12_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9n12_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_pll(regmap, "pllbck", "mainck", 1,
> &at91rm9200_pll_layout, &pllb_characteristics);
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9n12_pmc->chws[PMC_PLLBCK] = hw;
> +
> parent_names[0] = slck_name;
> parent_names[1] = "mainck";
> parent_names[2] = "plladivck";
> diff --git a/drivers/clk/at91/at91sam9rl.c b/drivers/clk/at91/at91sam9rl.c
> index bcf07f6a0e0e..0d1cc44b056f 100644
> --- a/drivers/clk/at91/at91sam9rl.c
> +++ b/drivers/clk/at91/at91sam9rl.c
> @@ -87,7 +87,7 @@ static void __init at91sam9rl_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91sam9rl_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9rl_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(at91sam9rl_systemck),
> nck(at91sam9rl_periphck), 0, 2);
> if (!at91sam9rl_pmc)
> @@ -105,6 +105,8 @@ static void __init at91sam9rl_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9rl_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/at91sam9x5.c b/drivers/clk/at91/at91sam9x5.c
> index f13756b407e2..0ce3da080287 100644
> --- a/drivers/clk/at91/at91sam9x5.c
> +++ b/drivers/clk/at91/at91sam9x5.c
> @@ -150,7 +150,7 @@ static void __init at91sam9x5_pmc_setup(struct device_node *np,
> if (IS_ERR(regmap))
> return;
>
> - at91sam9x5_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9x5_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(at91sam9x5_systemck), 31, 0, 2);
> if (!at91sam9x5_pmc)
> return;
> @@ -184,6 +184,8 @@ static void __init at91sam9x5_pmc_setup(struct device_node *np,
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9x5_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/sam9x60.c b/drivers/clk/at91/sam9x60.c
> index db14e0427c7f..3e20aa68259f 100644
> --- a/drivers/clk/at91/sam9x60.c
> +++ b/drivers/clk/at91/sam9x60.c
> @@ -182,7 +182,7 @@ static void __init sam9x60_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sam9x60_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + sam9x60_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(sam9x60_systemck),
> nck(sam9x60_periphck),
> nck(sam9x60_gck), 8);
> @@ -214,6 +214,8 @@ static void __init sam9x60_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sam9x60_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = sam9x60_clk_register_pll(regmap, &pmc_pll_lock, "upllck",
> "main_osc", 1, &upll_characteristics);
> if (IS_ERR(hw))
> diff --git a/drivers/clk/at91/sama5d2.c b/drivers/clk/at91/sama5d2.c
> index ae5e83cadb3d..b3fa2291ccd8 100644
> --- a/drivers/clk/at91/sama5d2.c
> +++ b/drivers/clk/at91/sama5d2.c
> @@ -166,7 +166,7 @@ static void __init sama5d2_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sama5d2_pmc = pmc_data_allocate(PMC_I2S1_MUX + 1,
> + sama5d2_pmc = pmc_data_allocate(PMC_AUDIOPLLCK + 1,
> nck(sama5d2_systemck),
> nck(sama5d2_periph32ck),
> nck(sama5d2_gck), 3);
> @@ -202,6 +202,8 @@ static void __init sama5d2_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d2_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_audio_pll_frac(regmap, "audiopll_fracck",
> "mainck");
> if (IS_ERR(hw))
> @@ -217,6 +219,8 @@ static void __init sama5d2_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d2_pmc->chws[PMC_AUDIOPLLCK] = hw;
> +
> regmap_sfr = syscon_regmap_lookup_by_compatible("atmel,sama5d2-sfr");
> if (IS_ERR(regmap_sfr))
> regmap_sfr = NULL;
> diff --git a/drivers/clk/at91/sama5d3.c b/drivers/clk/at91/sama5d3.c
> index 507eef6797f1..5e4e44dd4c37 100644
> --- a/drivers/clk/at91/sama5d3.c
> +++ b/drivers/clk/at91/sama5d3.c
> @@ -125,7 +125,7 @@ static void __init sama5d3_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sama5d3_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + sama5d3_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(sama5d3_systemck),
> nck(sama5d3_periphck), 0, 3);
> if (!sama5d3_pmc)
> @@ -158,6 +158,8 @@ static void __init sama5d3_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d3_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/sama5d4.c b/drivers/clk/at91/sama5d4.c
> index 80692902b4e4..662ff5fa6e98 100644
> --- a/drivers/clk/at91/sama5d4.c
> +++ b/drivers/clk/at91/sama5d4.c
> @@ -140,7 +140,7 @@ static void __init sama5d4_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sama5d4_pmc = pmc_data_allocate(PMC_MCK2 + 1,
> + sama5d4_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(sama5d4_systemck),
> nck(sama5d4_periph32ck), 0, 3);
> if (!sama5d4_pmc)
> @@ -173,6 +173,8 @@ static void __init sama5d4_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d4_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/include/dt-bindings/clock/at91.h b/include/dt-bindings/clock/at91.h
> index c3f4aa6a2d29..eba17106608b 100644
> --- a/include/dt-bindings/clock/at91.h
> +++ b/include/dt-bindings/clock/at91.h
> @@ -21,6 +21,9 @@
> #define PMC_MCK2 4
> #define PMC_I2S0_MUX 5
> #define PMC_I2S1_MUX 6
> +#define PMC_PLLACK 7
> +#define PMC_PLLBCK 8
> +#define PMC_AUDIOPLLCK 9
>
> #ifndef AT91_PMC_MOSCS
> #define AT91_PMC_MOSCS 0 /* MOSCS Flag */
> --
> 2.20.1
>
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
WARNING: multiple messages have this Message-ID (diff)
From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
Cc: devicetree@vger.kernel.org, Stephen Boyd <sboyd@kernel.org>,
Michael Turquette <mturquette@baylibre.com>,
linux-kernel@vger.kernel.org,
Ludovic Desroches <ludovic.desroches@microchip.com>,
Rob Herring <robh+dt@kernel.org>,
linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 3/3] clk: at91: allow setting all PMC clock parents via DT
Date: Mon, 4 May 2020 23:44:57 +0200 [thread overview]
Message-ID: <20200504214457.GN34497@piout.net> (raw)
In-Reply-To: <5f41e1872b1a9a7f0f50f9bb47e4378e67e55e74.1588623391.git.mirq-linux@rere.qmqm.pl>
On 04/05/2020 22:19:18+0200, Michał Mirosław wrote:
> We need to have clocks accessible via phandle to select them
> as peripheral clock parent using assigned-clock-parents in DT.
> Add support for PLLACK/PLLBCK/AUDIOPLLCK clocks where available.
>
> Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> ---
> v2: rebase to clk/clk-at91 branch
> v3: no changes
> v4: extend to whole family
> v5: fix copy-and-paste bugs for SAMA5D3/4
> v6: make AUDIOPLL's id separate to PLLB
> ---
> drivers/clk/at91/at91rm9200.c | 6 +++++-
> drivers/clk/at91/at91sam9260.c | 6 +++++-
> drivers/clk/at91/at91sam9g45.c | 4 +++-
> drivers/clk/at91/at91sam9n12.c | 6 +++++-
> drivers/clk/at91/at91sam9rl.c | 4 +++-
> drivers/clk/at91/at91sam9x5.c | 4 +++-
> drivers/clk/at91/sam9x60.c | 4 +++-
> drivers/clk/at91/sama5d2.c | 6 +++++-
> drivers/clk/at91/sama5d3.c | 4 +++-
> drivers/clk/at91/sama5d4.c | 4 +++-
> include/dt-bindings/clock/at91.h | 3 +++
> 11 files changed, 41 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/clk/at91/at91rm9200.c b/drivers/clk/at91/at91rm9200.c
> index 8da88e9a95d8..38bdb4981315 100644
> --- a/drivers/clk/at91/at91rm9200.c
> +++ b/drivers/clk/at91/at91rm9200.c
> @@ -98,7 +98,7 @@ static void __init at91rm9200_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91rm9200_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91rm9200_pmc = pmc_data_allocate(PMC_PLLBCK + 1,
> nck(at91rm9200_systemck),
> nck(at91rm9200_periphck), 0, 4);
> if (!at91rm9200_pmc)
> @@ -123,12 +123,16 @@ static void __init at91rm9200_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91rm9200_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_pll(regmap, "pllbck", "mainck", 1,
> &at91rm9200_pll_layout,
> &rm9200_pll_characteristics);
> if (IS_ERR(hw))
> goto err_free;
>
> + at91rm9200_pmc->chws[PMC_PLLBCK] = hw;
> +
> parent_names[0] = slowxtal_name;
> parent_names[1] = "mainck";
> parent_names[2] = "pllack";
> diff --git a/drivers/clk/at91/at91sam9260.c b/drivers/clk/at91/at91sam9260.c
> index 7e5ff252fffc..6d0723aa8b13 100644
> --- a/drivers/clk/at91/at91sam9260.c
> +++ b/drivers/clk/at91/at91sam9260.c
> @@ -352,7 +352,7 @@ static void __init at91sam926x_pmc_setup(struct device_node *np,
> if (IS_ERR(regmap))
> return;
>
> - at91sam9260_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9260_pmc = pmc_data_allocate(PMC_PLLBCK + 1,
> ndck(data->sck, data->num_sck),
> ndck(data->pck, data->num_pck),
> 0, data->num_progck);
> @@ -399,12 +399,16 @@ static void __init at91sam926x_pmc_setup(struct device_node *np,
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9260_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_pll(regmap, "pllbck", "mainck", 1,
> data->pllb_layout,
> data->pllb_characteristics);
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9260_pmc->chws[PMC_PLLBCK] = hw;
> +
> parent_names[0] = slck_name;
> parent_names[1] = "mainck";
> parent_names[2] = "pllack";
> diff --git a/drivers/clk/at91/at91sam9g45.c b/drivers/clk/at91/at91sam9g45.c
> index 5d18eb04c218..9873b583c260 100644
> --- a/drivers/clk/at91/at91sam9g45.c
> +++ b/drivers/clk/at91/at91sam9g45.c
> @@ -115,7 +115,7 @@ static void __init at91sam9g45_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91sam9g45_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9g45_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(at91sam9g45_systemck),
> nck(at91sam9g45_periphck), 0, 2);
> if (!at91sam9g45_pmc)
> @@ -143,6 +143,8 @@ static void __init at91sam9g45_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9g45_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/at91sam9n12.c b/drivers/clk/at91/at91sam9n12.c
> index 3a2564c2f724..630dc5d87171 100644
> --- a/drivers/clk/at91/at91sam9n12.c
> +++ b/drivers/clk/at91/at91sam9n12.c
> @@ -128,7 +128,7 @@ static void __init at91sam9n12_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91sam9n12_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9n12_pmc = pmc_data_allocate(PMC_PLLBCK + 1,
> nck(at91sam9n12_systemck), 31, 0, 2);
> if (!at91sam9n12_pmc)
> return;
> @@ -162,11 +162,15 @@ static void __init at91sam9n12_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9n12_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_pll(regmap, "pllbck", "mainck", 1,
> &at91rm9200_pll_layout, &pllb_characteristics);
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9n12_pmc->chws[PMC_PLLBCK] = hw;
> +
> parent_names[0] = slck_name;
> parent_names[1] = "mainck";
> parent_names[2] = "plladivck";
> diff --git a/drivers/clk/at91/at91sam9rl.c b/drivers/clk/at91/at91sam9rl.c
> index bcf07f6a0e0e..0d1cc44b056f 100644
> --- a/drivers/clk/at91/at91sam9rl.c
> +++ b/drivers/clk/at91/at91sam9rl.c
> @@ -87,7 +87,7 @@ static void __init at91sam9rl_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - at91sam9rl_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9rl_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(at91sam9rl_systemck),
> nck(at91sam9rl_periphck), 0, 2);
> if (!at91sam9rl_pmc)
> @@ -105,6 +105,8 @@ static void __init at91sam9rl_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9rl_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/at91sam9x5.c b/drivers/clk/at91/at91sam9x5.c
> index f13756b407e2..0ce3da080287 100644
> --- a/drivers/clk/at91/at91sam9x5.c
> +++ b/drivers/clk/at91/at91sam9x5.c
> @@ -150,7 +150,7 @@ static void __init at91sam9x5_pmc_setup(struct device_node *np,
> if (IS_ERR(regmap))
> return;
>
> - at91sam9x5_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + at91sam9x5_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(at91sam9x5_systemck), 31, 0, 2);
> if (!at91sam9x5_pmc)
> return;
> @@ -184,6 +184,8 @@ static void __init at91sam9x5_pmc_setup(struct device_node *np,
> if (IS_ERR(hw))
> goto err_free;
>
> + at91sam9x5_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/sam9x60.c b/drivers/clk/at91/sam9x60.c
> index db14e0427c7f..3e20aa68259f 100644
> --- a/drivers/clk/at91/sam9x60.c
> +++ b/drivers/clk/at91/sam9x60.c
> @@ -182,7 +182,7 @@ static void __init sam9x60_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sam9x60_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + sam9x60_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(sam9x60_systemck),
> nck(sam9x60_periphck),
> nck(sam9x60_gck), 8);
> @@ -214,6 +214,8 @@ static void __init sam9x60_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sam9x60_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = sam9x60_clk_register_pll(regmap, &pmc_pll_lock, "upllck",
> "main_osc", 1, &upll_characteristics);
> if (IS_ERR(hw))
> diff --git a/drivers/clk/at91/sama5d2.c b/drivers/clk/at91/sama5d2.c
> index ae5e83cadb3d..b3fa2291ccd8 100644
> --- a/drivers/clk/at91/sama5d2.c
> +++ b/drivers/clk/at91/sama5d2.c
> @@ -166,7 +166,7 @@ static void __init sama5d2_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sama5d2_pmc = pmc_data_allocate(PMC_I2S1_MUX + 1,
> + sama5d2_pmc = pmc_data_allocate(PMC_AUDIOPLLCK + 1,
> nck(sama5d2_systemck),
> nck(sama5d2_periph32ck),
> nck(sama5d2_gck), 3);
> @@ -202,6 +202,8 @@ static void __init sama5d2_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d2_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_audio_pll_frac(regmap, "audiopll_fracck",
> "mainck");
> if (IS_ERR(hw))
> @@ -217,6 +219,8 @@ static void __init sama5d2_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d2_pmc->chws[PMC_AUDIOPLLCK] = hw;
> +
> regmap_sfr = syscon_regmap_lookup_by_compatible("atmel,sama5d2-sfr");
> if (IS_ERR(regmap_sfr))
> regmap_sfr = NULL;
> diff --git a/drivers/clk/at91/sama5d3.c b/drivers/clk/at91/sama5d3.c
> index 507eef6797f1..5e4e44dd4c37 100644
> --- a/drivers/clk/at91/sama5d3.c
> +++ b/drivers/clk/at91/sama5d3.c
> @@ -125,7 +125,7 @@ static void __init sama5d3_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sama5d3_pmc = pmc_data_allocate(PMC_MAIN + 1,
> + sama5d3_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(sama5d3_systemck),
> nck(sama5d3_periphck), 0, 3);
> if (!sama5d3_pmc)
> @@ -158,6 +158,8 @@ static void __init sama5d3_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d3_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/drivers/clk/at91/sama5d4.c b/drivers/clk/at91/sama5d4.c
> index 80692902b4e4..662ff5fa6e98 100644
> --- a/drivers/clk/at91/sama5d4.c
> +++ b/drivers/clk/at91/sama5d4.c
> @@ -140,7 +140,7 @@ static void __init sama5d4_pmc_setup(struct device_node *np)
> if (IS_ERR(regmap))
> return;
>
> - sama5d4_pmc = pmc_data_allocate(PMC_MCK2 + 1,
> + sama5d4_pmc = pmc_data_allocate(PMC_PLLACK + 1,
> nck(sama5d4_systemck),
> nck(sama5d4_periph32ck), 0, 3);
> if (!sama5d4_pmc)
> @@ -173,6 +173,8 @@ static void __init sama5d4_pmc_setup(struct device_node *np)
> if (IS_ERR(hw))
> goto err_free;
>
> + sama5d4_pmc->chws[PMC_PLLACK] = hw;
> +
> hw = at91_clk_register_utmi(regmap, NULL, "utmick", "mainck");
> if (IS_ERR(hw))
> goto err_free;
> diff --git a/include/dt-bindings/clock/at91.h b/include/dt-bindings/clock/at91.h
> index c3f4aa6a2d29..eba17106608b 100644
> --- a/include/dt-bindings/clock/at91.h
> +++ b/include/dt-bindings/clock/at91.h
> @@ -21,6 +21,9 @@
> #define PMC_MCK2 4
> #define PMC_I2S0_MUX 5
> #define PMC_I2S1_MUX 6
> +#define PMC_PLLACK 7
> +#define PMC_PLLBCK 8
> +#define PMC_AUDIOPLLCK 9
>
> #ifndef AT91_PMC_MOSCS
> #define AT91_PMC_MOSCS 0 /* MOSCS Flag */
> --
> 2.20.1
>
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-05-04 21:46 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-04 20:19 [PATCH v6 0/3] clk: at91: support configuring more clocks via DT Michał Mirosław
2020-05-04 20:19 ` Michał Mirosław
2020-05-04 20:19 ` [PATCH v6 2/3] clk: at91: allow setting PCKx parent " Michał Mirosław
2020-05-04 20:19 ` Michał Mirosław
2020-05-04 20:44 ` Alexandre Belloni
2020-05-04 20:44 ` Alexandre Belloni
2020-05-04 22:36 ` Michał Mirosław
2020-05-04 22:36 ` Michał Mirosław
2020-05-04 20:19 ` [PATCH v6 1/3] clk: at91: optimize pmc data allocation Michał Mirosław
2020-05-04 20:19 ` Michał Mirosław
2020-05-04 20:19 ` [PATCH v6 3/3] clk: at91: allow setting all PMC clock parents via DT Michał Mirosław
2020-05-04 20:19 ` Michał Mirosław
2020-05-04 21:44 ` Alexandre Belloni [this message]
2020-05-04 21:44 ` Alexandre Belloni
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=20200504214457.GN34497@piout.net \
--to=alexandre.belloni@bootlin.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ludovic.desroches@microchip.com \
--cc=mirq-linux@rere.qmqm.pl \
--cc=mturquette@baylibre.com \
--cc=nicolas.ferre@microchip.com \
--cc=robh+dt@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.