From: Claudiu Beznea <claudiu.beznea@tuxon.dev>
To: Ryan.Wanner@microchip.com, mturquette@baylibre.com,
sboyd@kernel.org, alexandre.belloni@bootlin.com,
nicolas.ferre@microchip.com
Cc: linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, varshini.rajendran@microchip.com
Subject: Re: [PATCH v4 03/31] clk: at91: sam9x75: switch to parent_hw and parent_data
Date: Mon, 20 Oct 2025 22:38:26 +0300 [thread overview]
Message-ID: <63bcb444-7baf-4332-bca5-9582818691b3@tuxon.dev> (raw)
In-Reply-To: <42d01c533eecf4018174ab5c3e0a6130e3dc34f0.1758226719.git.Ryan.Wanner@microchip.com>
Hi, Ryan,
On 9/19/25 00:15, Ryan.Wanner@microchip.com wrote:
> From: Ryan Wanner <Ryan.Wanner@microchip.com>
>
> Switch SAM9X75 clocks to use parent_hw and parent_data. Having
> parent_hw instead of parent names improves to clock registration
> speed and re-parenting.
>
> The USBCLK will be updated in subsequent patches that update the clock
> registration functions to use parent_hw and parent_data.
>
> __clk_get_hw() will be removed in subsequent patches in this series.
>
> Signed-off-by: Ryan Wanner <Ryan.Wanner@microchip.com>
> ---
> drivers/clk/at91/sam9x7.c | 308 +++++++++++++++++++++-----------------
> 1 file changed, 173 insertions(+), 135 deletions(-)
>
> diff --git a/drivers/clk/at91/sam9x7.c b/drivers/clk/at91/sam9x7.c
> index 89868a0aeaba..cb5849da494f 100644
> --- a/drivers/clk/at91/sam9x7.c
> +++ b/drivers/clk/at91/sam9x7.c
> @@ -33,10 +33,22 @@ enum pll_ids {
> PLL_ID_UPLL,
> PLL_ID_AUDIO,
> PLL_ID_LVDS,
> - PLL_ID_PLLA_DIV2,
> PLL_ID_MAX,
> };
>
> +/*
> + * PLL component identifier
> + * @PLL_COMPID_FRAC: Fractional PLL component identifier
> + * @PLL_COMPID_DIV0: 1st PLL divider component identifier
> + * @PLL_COMPID_DIV1: 2nd PLL divider component identifier
> + */
> +enum pll_component_id {
> + PLL_COMPID_FRAC,
> + PLL_COMPID_DIV0,
> + PLL_COMPID_DIV1,
> + PLL_COMPID_MAX,
> +};
> +
> /**
> * enum pll_type - PLL type identifiers
> * @PLL_TYPE_FRAC: fractional PLL identifier
> @@ -185,6 +197,18 @@ static const struct clk_pll_layout pll_divio_layout = {
> .endiv_shift = 30,
> };
>
> +/*
> + * SAM9X7 PLL possible parents
> + * @SAM9X7_PLL_PARENT_MAINCK: MAINCK is PLL a parent
> + * @SAM9X7_PLL_PARENT_MAIN_XTAL: MAIN XTAL is a PLL parent
> + * @SAM9X7_PLL_PARENT_FRACCK: Frac PLL is a PLL parent (for PLL dividers)
> + */
> +enum sam9x7_pll_parent {
> + SAM9X7_PLL_PARENT_MAINCK,
> + SAM9X7_PLL_PARENT_MAIN_XTAL,
> + SAM9X7_PLL_PARENT_FRACCK
> +};
> +
> /*
> * PLL clocks description
> * @n: clock name
> @@ -192,22 +216,24 @@ static const struct clk_pll_layout pll_divio_layout = {
> * @l: clock layout
> * @t: clock type
> * @c: pll characteristics
> + * @hw: pointer to clk_hw
> * @f: clock flags
> * @eid: export index in sam9x7->chws[] array
> */
> -static const struct {
> +static struct {
> const char *n;
> - const char *p;
> const struct clk_pll_layout *l;
> u8 t;
> const struct clk_pll_characteristics *c;
> + struct clk_hw *hw;
> unsigned long f;
> + enum sam9x7_pll_parent p;
> u8 eid;
> -} sam9x7_plls[][3] = {
> +} sam9x7_plls[][PLL_COMPID_MAX] = {
> [PLL_ID_PLLA] = {
> - {
> + [PLL_COMPID_FRAC] = {
> .n = "plla_fracck",
> - .p = "mainck",
> + .p = SAM9X7_PLL_PARENT_MAINCK,
> .l = &plla_frac_layout,
> .t = PLL_TYPE_FRAC,
> /*
> @@ -218,9 +244,9 @@ static const struct {
> .c = &plla_characteristics,
> },
>
> - {
> + [PLL_COMPID_DIV0] = {
> .n = "plla_divpmcck",
> - .p = "plla_fracck",
> + .p = SAM9X7_PLL_PARENT_FRACCK,
> .l = &pll_divpmc_layout,
> .t = PLL_TYPE_DIV,
> /* This feeds CPU. It should not be disabled */
> @@ -228,21 +254,35 @@ static const struct {
> .eid = PMC_PLLACK,
> .c = &plla_characteristics,
> },
> +
> + [PLL_COMPID_DIV1] = {
> + .n = "plla_div2pmcck",
> + .p = SAM9X7_PLL_PARENT_FRACCK,
> + .l = &plladiv2_divpmc_layout,
> + /*
> + * This may feed critical parts of the system like timers.
> + * It should not be disabled.
> + */
> + .f = CLK_IS_CRITICAL | CLK_SET_RATE_GATE,
> + .c = &plladiv2_characteristics,
> + .eid = PMC_PLLADIV2,
> + .t = PLL_TYPE_DIV,
> + },
> },
>
> [PLL_ID_UPLL] = {
> - {
> + [PLL_COMPID_FRAC] = {
> .n = "upll_fracck",
> - .p = "main_osc",
> + .p = SAM9X7_PLL_PARENT_MAIN_XTAL,
> .l = &pll_frac_layout,
> .t = PLL_TYPE_FRAC,
> .f = CLK_SET_RATE_GATE,
> .c = &upll_characteristics,
> },
>
> - {
> + [PLL_COMPID_DIV0] = {
> .n = "upll_divpmcck",
> - .p = "upll_fracck",
> + .p = SAM9X7_PLL_PARENT_FRACCK,
> .l = &pll_divpmc_layout,
> .t = PLL_TYPE_DIV,
> .f = CLK_SET_RATE_GATE | CLK_SET_PARENT_GATE |
> @@ -253,18 +293,18 @@ static const struct {
> },
>
> [PLL_ID_AUDIO] = {
> - {
> + [PLL_COMPID_FRAC] = {
> .n = "audiopll_fracck",
> - .p = "main_osc",
> + .p = SAM9X7_PLL_PARENT_MAIN_XTAL,
> .l = &pll_frac_layout,
> .f = CLK_SET_RATE_GATE,
> .c = &audiopll_characteristics,
> .t = PLL_TYPE_FRAC,
> },
>
> - {
> + [PLL_COMPID_DIV0] = {
> .n = "audiopll_divpmcck",
> - .p = "audiopll_fracck",
> + .p = SAM9X7_PLL_PARENT_FRACCK,
> .l = &pll_divpmc_layout,
> .f = CLK_SET_RATE_GATE | CLK_SET_PARENT_GATE |
> CLK_SET_RATE_PARENT,
> @@ -273,9 +313,9 @@ static const struct {
> .t = PLL_TYPE_DIV,
> },
>
> - {
> + [PLL_COMPID_DIV1] = {
> .n = "audiopll_diviock",
> - .p = "audiopll_fracck",
> + .p = SAM9X7_PLL_PARENT_FRACCK,
> .l = &pll_divio_layout,
> .f = CLK_SET_RATE_GATE | CLK_SET_PARENT_GATE |
> CLK_SET_RATE_PARENT,
> @@ -286,18 +326,18 @@ static const struct {
> },
>
> [PLL_ID_LVDS] = {
> - {
> + [PLL_COMPID_FRAC] = {
> .n = "lvdspll_fracck",
> - .p = "main_osc",
> + .p = SAM9X7_PLL_PARENT_MAIN_XTAL,
> .l = &pll_frac_layout,
> .f = CLK_SET_RATE_GATE,
> .c = &lvdspll_characteristics,
> .t = PLL_TYPE_FRAC,
> },
>
> - {
> + [PLL_COMPID_DIV0] = {
> .n = "lvdspll_divpmcck",
> - .p = "lvdspll_fracck",
> + .p = SAM9X7_PLL_PARENT_FRACCK,
> .l = &pll_divpmc_layout,
> .f = CLK_SET_RATE_GATE | CLK_SET_PARENT_GATE |
> CLK_SET_RATE_PARENT,
> @@ -306,22 +346,6 @@ static const struct {
> .t = PLL_TYPE_DIV,
> },
> },
> -
> - [PLL_ID_PLLA_DIV2] = {
> - {
> - .n = "plla_div2pmcck",
> - .p = "plla_fracck",
> - .l = &plladiv2_divpmc_layout,
> - /*
> - * This may feed critical parts of the system like timers.
> - * It should not be disabled.
> - */
> - .f = CLK_IS_CRITICAL | CLK_SET_RATE_GATE,
> - .c = &plladiv2_characteristics,
> - .eid = PMC_PLLADIV2,
> - .t = PLL_TYPE_DIV,
> - },
> - },
> };
>
> static const struct clk_programmable_layout sam9x7_programmable_layout = {
> @@ -339,9 +363,9 @@ static const struct clk_pcr_layout sam9x7_pcr_layout = {
> .pid_mask = GENMASK(6, 0),
> };
>
> -static const struct {
> +static struct {
> char *n;
> - char *p;
> + struct clk_hw *parent_hw;
> u8 id;
> unsigned long flags;
> } sam9x7_systemck[] = {
> @@ -349,10 +373,10 @@ static const struct {
> * ddrck feeds DDR controller and is enabled by bootloader thus we need
> * to keep it enabled in case there is no Linux consumer for it.
> */
> - { .n = "ddrck", .p = "masterck_div", .id = 2, .flags = CLK_IS_CRITICAL },
> - { .n = "uhpck", .p = "usbck", .id = 6 },
> - { .n = "pck0", .p = "prog0", .id = 8 },
> - { .n = "pck1", .p = "prog1", .id = 9 },
> + { .n = "ddrck", .id = 2, .flags = CLK_IS_CRITICAL },
> + { .n = "uhpck", .id = 6 },
> + { .n = "pck0", .id = 8 },
> + { .n = "pck1", .id = 9 },
> };
>
> /*
> @@ -426,7 +450,8 @@ static const struct {
> /*
> * Generic clock description
> * @n: clock name
> - * @pp: PLL parents
> + * @pp: PLL parents (entry formed by PLL components identifiers
> + * (see enum pll_component_id))
> * @pp_mux_table: PLL parents mux table
> * @r: clock output range
> * @pp_chg_id: id in parent array of changeable PLL parent
> @@ -435,7 +460,10 @@ static const struct {
> */
> static const struct {
> const char *n;
> - const char *pp[8];
> + struct {
> + int pll_id;
> + int pll_compid;
2 tabs here should be enough.
> + } pp[8];
> const char pp_mux_table[8];
> struct clk_range r;
> int pp_chg_id;
> @@ -445,7 +473,7 @@ static const struct {
> {
> .n = "flex0_gclk",
> .id = 5,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -454,7 +482,7 @@ static const struct {
> {
> .n = "flex1_gclk",
> .id = 6,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -463,7 +491,7 @@ static const struct {
> {
> .n = "flex2_gclk",
> .id = 7,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -472,7 +500,7 @@ static const struct {
> {
> .n = "flex3_gclk",
> .id = 8,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -481,7 +509,7 @@ static const struct {
> {
> .n = "flex6_gclk",
> .id = 9,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -490,7 +518,7 @@ static const struct {
> {
> .n = "flex7_gclk",
> .id = 10,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -499,7 +527,7 @@ static const struct {
> {
> .n = "flex8_gclk",
> .id = 11,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -509,7 +537,7 @@ static const struct {
> .n = "sdmmc0_gclk",
> .id = 12,
> .r = { .max = 105000000 },
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -518,7 +546,7 @@ static const struct {
> {
> .n = "flex4_gclk",
> .id = 13,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -527,7 +555,7 @@ static const struct {
> {
> .n = "flex5_gclk",
> .id = 14,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -536,7 +564,7 @@ static const struct {
> {
> .n = "flex9_gclk",
> .id = 15,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -545,7 +573,7 @@ static const struct {
> {
> .n = "flex10_gclk",
> .id = 16,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -554,7 +582,7 @@ static const struct {
> {
> .n = "tcb0_gclk",
> .id = 17,
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -563,7 +591,7 @@ static const struct {
> {
> .n = "adc_gclk",
> .id = 19,
> - .pp = { "upll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(UPLL, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 5, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -573,7 +601,7 @@ static const struct {
> .n = "lcd_gclk",
> .id = 25,
> .r = { .max = 75000000 },
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -583,7 +611,7 @@ static const struct {
> .n = "sdmmc1_gclk",
> .id = 26,
> .r = { .max = 105000000 },
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -593,7 +621,7 @@ static const struct {
> .n = "mcan0_gclk",
> .id = 29,
> .r = { .max = 80000000 },
> - .pp = { "upll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(UPLL, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 5, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -603,7 +631,7 @@ static const struct {
> .n = "mcan1_gclk",
> .id = 30,
> .r = { .max = 80000000 },
> - .pp = { "upll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(UPLL, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 5, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -612,7 +640,7 @@ static const struct {
> {
> .n = "flex11_gclk",
> .id = 32,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -621,7 +649,7 @@ static const struct {
> {
> .n = "flex12_gclk",
> .id = 33,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -631,7 +659,7 @@ static const struct {
> .n = "i2s_gclk",
> .id = 34,
> .r = { .max = 100000000 },
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -641,7 +669,7 @@ static const struct {
> .n = "qspi_gclk",
> .id = 35,
> .r = { .max = 200000000 },
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -650,7 +678,7 @@ static const struct {
> {
> .n = "pit64b0_gclk",
> .id = 37,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -660,7 +688,7 @@ static const struct {
> .n = "classd_gclk",
> .id = 42,
> .r = { .max = 100000000 },
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -669,7 +697,7 @@ static const struct {
> {
> .n = "tcb1_gclk",
> .id = 45,
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -678,7 +706,7 @@ static const struct {
> {
> .n = "dbgu_gclk",
> .id = 47,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -688,7 +716,7 @@ static const struct {
> .n = "mipiphy_gclk",
> .id = 55,
> .r = { .max = 27000000 },
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -697,7 +725,7 @@ static const struct {
> {
> .n = "pit64b1_gclk",
> .id = 58,
> - .pp = { "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 8, },
> .pp_count = 1,
> .pp_chg_id = INT_MIN,
> @@ -706,7 +734,7 @@ static const struct {
> {
> .n = "gmac_gclk",
> .id = 67,
> - .pp = { "audiopll_divpmcck", "plla_div2pmcck", },
> + .pp = { PLL_IDS_TO_ARR_ENTRY(AUDIO, DIV0), PLL_IDS_TO_ARR_ENTRY(PLLA, DIV1), },
> .pp_mux_table = { 6, 8, },
> .pp_count = 2,
> .pp_chg_id = INT_MIN,
> @@ -716,33 +744,25 @@ static const struct {
> static void __init sam9x7_pmc_setup(struct device_node *np)
> {
> struct clk_range range = CLK_RANGE(0, 0);
> - const char *td_slck_name, *md_slck_name, *mainxtal_name;
> + const char *main_xtal_name;
> struct pmc_data *sam9x7_pmc;
> const char *parent_names[9];
> void **clk_mux_buffer = NULL;
> int clk_mux_buffer_size = 0;
> - struct clk_hw *main_osc_hw;
> struct regmap *regmap;
> - struct clk_hw *hw;
> + struct clk_hw *hw, *main_rc_hw, *main_osc_hw, *main_xtal_hw;
> + struct clk_hw *td_slck_hw, *md_slck_hw, *usbck_hw;
> + struct clk_hw *parent_hws[9];
> int i, j;
>
> - i = of_property_match_string(np, "clock-names", "td_slck");
> - if (i < 0)
> - return;
> -
> - td_slck_name = of_clk_get_parent_name(np, i);
> -
> - i = of_property_match_string(np, "clock-names", "md_slck");
> - if (i < 0)
> - return;
> -
> - md_slck_name = of_clk_get_parent_name(np, i);
> -
> + td_slck_hw = __clk_get_hw(of_clk_get_by_name(np, "td_slck"));
> + md_slck_hw = __clk_get_hw(of_clk_get_by_name(np, "md_slck"));
> i = of_property_match_string(np, "clock-names", "main_xtal");
Could you please move this above:
main_xtal_name = of_clk_get_parent_name(np, i);
and have it like:
i = of_property_match_string(np, "clock-names", "main_xtal");
if (i < 0)
return;
mainxtal_name = of_clk_get_parent_name(np, i);
> - if (i < 0)
> +
> + if (!td_slck_hw || !md_slck_hw || !i)
I would keep the checking of i close to of_property_match_string(). Also,
you changed check only zero as invalid value. Is that right?
of_property_match_string() can return negative error numbers as well.
> return;
> - mainxtal_name = of_clk_get_parent_name(np, i);
>
> + main_xtal_name = of_clk_get_parent_name(np, i);
> regmap = device_node_to_regmap(np);
> if (IS_ERR(regmap))
> return;
> @@ -760,26 +780,25 @@ static void __init sam9x7_pmc_setup(struct device_node *np)
> if (!clk_mux_buffer)
> goto err_free;
>
> - hw = at91_clk_register_main_rc_osc(regmap, "main_rc_osc", 12000000,
> - 50000000);
> - if (IS_ERR(hw))
> + main_rc_hw = at91_clk_register_main_rc_osc(regmap, "main_rc_osc", 12000000,
> + 50000000);
> + if (IS_ERR(main_rc_hw))
> goto err_free;
>
> - hw = at91_clk_register_main_osc(regmap, "main_osc", mainxtal_name, NULL, 0);
> - if (IS_ERR(hw))
> + main_osc_hw = at91_clk_register_main_osc(regmap, "main_osc", NULL, &AT91_CLK_PD_NAME(main_xtal_name), 0);
Could you please add a line break before &AT91_CLK_PD_NAME() ?
next prev parent reply other threads:[~2025-10-20 19:38 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-18 21:15 [PATCH v4 00/31] clk: at91: add support for parent_data and Ryan.Wanner
2025-09-18 21:15 ` [PATCH v4 01/31] clk: at91: pmc: add macros for clk_parent_data Ryan.Wanner
2025-10-20 19:40 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 02/31] clk: at91: pmc: Move macro to header file Ryan.Wanner
2025-09-18 21:15 ` [PATCH v4 03/31] clk: at91: sam9x75: switch to parent_hw and parent_data Ryan.Wanner
2025-10-20 19:38 ` Claudiu Beznea [this message]
2025-09-18 21:15 ` [PATCH v4 04/31] clk: at91: clk-sam9x60-pll: use clk_parent_data Ryan.Wanner
2025-10-20 19:41 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 05/31] clk: at91: clk-peripheral: switch to clk_parent_data Ryan.Wanner
2025-10-20 19:41 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 06/31] clk: at91: clk-main: switch to clk parent data Ryan.Wanner
2025-10-20 19:42 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 07/31] clk: at91: clk-utmi: use clk_parent_data Ryan.Wanner
2025-10-20 19:43 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 08/31] clk: at91: clk-master: " Ryan.Wanner
2025-10-20 19:44 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 09/31] clk: at91: clk-programmable: " Ryan.Wanner
2025-09-18 21:15 ` [PATCH v4 10/31] clk: at91: clk-generated: " Ryan.Wanner
2025-10-20 19:39 ` Claudiu Beznea
2025-10-20 19:45 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 11/31] clk: at91: clk-usb: add support for clk_parent_data Ryan.Wanner
2025-10-20 19:17 ` Claudiu Beznea
2025-12-18 16:23 ` Ryan.Wanner
2025-12-23 14:00 ` claudiu beznea
2026-01-05 17:58 ` Ryan.Wanner
2026-01-10 15:03 ` Claudiu Beznea
2026-01-12 21:25 ` Ryan Wanner
2026-01-16 6:57 ` claudiu beznea
2025-09-18 21:15 ` [PATCH v4 12/31] clk: at91: clk-system: use clk_parent_data Ryan.Wanner
2025-09-18 21:15 ` [PATCH v4 13/31] clk: at91: sama7d65: switch to parent_hw and parent_data Ryan.Wanner
2025-10-20 19:14 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 14/31] clk: at91: clk-pll: add support for parent_hw Ryan.Wanner
2025-09-18 21:15 ` [PATCH v4 15/31] clk: at91: clk-audio-pll: " Ryan.Wanner
2025-09-18 21:15 ` [PATCH v4 16/31] clk: at91: clk-plldiv: " Ryan.Wanner
2025-10-20 19:12 ` Claudiu Beznea
2025-09-18 21:15 ` [PATCH v4 17/31] clk: at91: clk-h32mx: " Ryan.Wanner
2025-10-20 19:12 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 18/31] clk: at91: clk-i2s-mux: " Ryan.Wanner
2025-10-20 19:13 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 19/31] clk: at91: clk-smd: add support for clk_parent_data Ryan.Wanner
2025-10-20 19:14 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 20/31] clk: at91: clk-slow: add support for parent_hw Ryan.Wanner
2025-10-20 19:17 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 21/31] clk: at91: dt-compat: switch to parent_hw and parent_data Ryan.Wanner
2025-10-20 19:15 ` Claudiu Beznea
2026-01-09 17:03 ` Ryan Wanner
2026-01-10 15:07 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 22/31] clk: at91: sam9x60: " Ryan.Wanner
2025-09-18 21:16 ` [PATCH v4 23/31] clk: at91: sama5d2: " Ryan.Wanner
2025-10-20 19:19 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 24/31] clk: at91: sama5d3: " Ryan.Wanner
2025-09-18 21:16 ` [PATCH v4 25/31] clk: at91: sama5d4: " Ryan.Wanner
2025-09-18 21:16 ` [PATCH v4 26/31] clk: at91: at91sam9x5: " Ryan.Wanner
2025-10-20 19:19 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 27/31] clk: at91: at91rm9200: " Ryan.Wanner
2025-09-18 21:16 ` [PATCH v4 28/31] clk: at91: at91sam9260: " Ryan.Wanner
2025-10-20 19:35 ` Claudiu Beznea
2025-09-18 21:16 ` [PATCH v4 29/31] clk: at91: at91sam9g45: " Ryan.Wanner
2025-09-18 21:16 ` [PATCH v4 30/31] clk: at91: at91sam9n12: " Ryan.Wanner
2025-09-18 21:16 ` [PATCH v4 31/31] clk: at91: at91sam9rl: switch to clk_parent_data Ryan.Wanner
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=63bcb444-7baf-4332-bca5-9582818691b3@tuxon.dev \
--to=claudiu.beznea@tuxon.dev \
--cc=Ryan.Wanner@microchip.com \
--cc=alexandre.belloni@bootlin.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=nicolas.ferre@microchip.com \
--cc=sboyd@kernel.org \
--cc=varshini.rajendran@microchip.com \
/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