]> Git Repo - linux.git/commitdiff
clk: at91: sama7g5: Allocate only the needed amount of memory for PLLs
authorClaudiu Beznea <[email protected]>
Sun, 14 Jul 2024 14:13:15 +0000 (17:13 +0300)
committerClaudiu Beznea <[email protected]>
Sat, 24 Aug 2024 14:44:11 +0000 (17:44 +0300)
The maximum number of PLL components on SAMA7G5 is 3 (one fractional
part and 2 dividers). Allocate the needed amount of memory for
sama7g5_plls 2d array. Previous code used to allocate 7 array entries for
each PLL. While at it, replace 3 with PLL_COMPID_MAX in the loop which
parses the sama7g5_plls 2d array.

Fixes: cb783bbbcf54 ("clk: at91: sama7g5: add clock support for sama7g5")
Acked-by: Stephen Boyd <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Claudiu Beznea <[email protected]>
drivers/clk/at91/sama7g5.c

index 6706d1305baaa557d112f228fcdbb171b8e4778a..8385badc1c70670a680b2475aa3803a9c0a920be 100644 (file)
@@ -51,6 +51,7 @@ enum pll_component_id {
        PLL_COMPID_FRAC,
        PLL_COMPID_DIV0,
        PLL_COMPID_DIV1,
+       PLL_COMPID_MAX,
 };
 
 /*
@@ -157,7 +158,7 @@ static struct sama7g5_pll {
        u8 t;
        u8 eid;
        u8 safe_div;
-} sama7g5_plls[][PLL_ID_MAX] = {
+} sama7g5_plls[][PLL_COMPID_MAX] = {
        [PLL_ID_CPU] = {
                [PLL_COMPID_FRAC] = {
                        .n = "cpupll_fracck",
@@ -1030,7 +1031,7 @@ static void __init sama7g5_pmc_setup(struct device_node *np)
        sama7g5_pmc->chws[PMC_MAIN] = hw;
 
        for (i = 0; i < PLL_ID_MAX; i++) {
-               for (j = 0; j < 3; j++) {
+               for (j = 0; j < PLL_COMPID_MAX; j++) {
                        struct clk_hw *parent_hw;
 
                        if (!sama7g5_plls[i][j].n)
This page took 0.055128 seconds and 4 git commands to generate.