[PATCH] arm: mach-k3: *: dev-data: Update to use ARRAY_SIZE

Instead of hard-coding the count of entries manually, use ARRAY_SIZE to keep the count updates appropriately.
Cc: Bryan Brattlof bb@ti.com Suggested-by: Ravi Gunasekaran r-gunasekaran@ti.com Signed-off-by: Nishanth Menon nm@ti.com ---
Link to discussion that triggered this: https://lore.kernel.org/all/20230713185927.3h6vqap4x3h62zd5@bryanbrattlof.co...
arch/arm/mach-k3/am62ax/dev-data.c | 8 ++++---- arch/arm/mach-k3/am62x/dev-data.c | 8 ++++---- arch/arm/mach-k3/j7200/dev-data.c | 8 ++++---- arch/arm/mach-k3/j721e/dev-data.c | 8 ++++---- arch/arm/mach-k3/j721s2/dev-data.c | 8 ++++---- 5 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/arch/arm/mach-k3/am62ax/dev-data.c b/arch/arm/mach-k3/am62ax/dev-data.c index 74739c638524..abf5d8e91aa2 100644 --- a/arch/arm/mach-k3/am62ax/dev-data.c +++ b/arch/arm/mach-k3/am62ax/dev-data.c @@ -66,8 +66,8 @@ const struct ti_k3_pd_platdata am62ax_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list, - .num_psc = 2, - .num_pd = 4, - .num_lpsc = 14, - .num_devs = 19, + .num_psc = ARRAY_SIZE(soc_psc_list), + .num_pd = ARRAY_SIZE(soc_pd_list), + .num_lpsc = ARRAY_SIZE(soc_lpsc_list), + .num_devs = ARRAY_SIZE(soc_dev_list), }; diff --git a/arch/arm/mach-k3/am62x/dev-data.c b/arch/arm/mach-k3/am62x/dev-data.c index 616d0650b9c0..1a6f9e2ca03e 100644 --- a/arch/arm/mach-k3/am62x/dev-data.c +++ b/arch/arm/mach-k3/am62x/dev-data.c @@ -71,8 +71,8 @@ const struct ti_k3_pd_platdata am62x_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list, - .num_psc = 2, - .num_pd = 5, - .num_lpsc = 16, - .num_devs = 21, + .num_psc = ARRAY_SIZE(soc_psc_list), + .num_pd = ARRAY_SIZE(soc_pd_list), + .num_lpsc = ARRAY_SIZE(soc_lpsc_list), + .num_devs = ARRAY_SIZE(soc_dev_list), }; diff --git a/arch/arm/mach-k3/j7200/dev-data.c b/arch/arm/mach-k3/j7200/dev-data.c index c1a4dab6941a..4ddc34210ebb 100644 --- a/arch/arm/mach-k3/j7200/dev-data.c +++ b/arch/arm/mach-k3/j7200/dev-data.c @@ -75,8 +75,8 @@ const struct ti_k3_pd_platdata j7200_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list, - .num_psc = 2, - .num_pd = 6, - .num_lpsc = 17, - .num_devs = 23, + .num_psc = ARRAY_SIZE(soc_psc_list), + .num_pd = ARRAY_SIZE(soc_pd_list), + .num_lpsc = ARRAY_SIZE(soc_lpsc_list), + .num_devs = ARRAY_SIZE(soc_dev_list), }; diff --git a/arch/arm/mach-k3/j721e/dev-data.c b/arch/arm/mach-k3/j721e/dev-data.c index f0afa3552b7f..97f017f8af50 100644 --- a/arch/arm/mach-k3/j721e/dev-data.c +++ b/arch/arm/mach-k3/j721e/dev-data.c @@ -73,8 +73,8 @@ const struct ti_k3_pd_platdata j721e_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list, - .num_psc = 2, - .num_pd = 5, - .num_lpsc = 16, - .num_devs = 23, + .num_psc = ARRAY_SIZE(soc_psc_list), + .num_pd = ARRAY_SIZE(soc_pd_list), + .num_lpsc = ARRAY_SIZE(soc_lpsc_list), + .num_devs = ARRAY_SIZE(soc_dev_list), }; diff --git a/arch/arm/mach-k3/j721s2/dev-data.c b/arch/arm/mach-k3/j721s2/dev-data.c index 35e8b17eb1e7..8c999a3c5a8b 100644 --- a/arch/arm/mach-k3/j721s2/dev-data.c +++ b/arch/arm/mach-k3/j721s2/dev-data.c @@ -79,8 +79,8 @@ const struct ti_k3_pd_platdata j721s2_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list, - .num_psc = 2, - .num_pd = 6, - .num_lpsc = 19, - .num_devs = 25, + .num_psc = ARRAY_SIZE(soc_psc_list), + .num_pd = ARRAY_SIZE(soc_pd_list), + .num_lpsc = ARRAY_SIZE(soc_lpsc_list), + .num_devs = ARRAY_SIZE(soc_dev_list), };

On July 13, 2023 thus sayeth Nishanth Menon:
Instead of hard-coding the count of entries manually, use ARRAY_SIZE to keep the count updates appropriately.
Cc: Bryan Brattlof bb@ti.com Suggested-by: Ravi Gunasekaran r-gunasekaran@ti.com Signed-off-by: Nishanth Menon nm@ti.com
That looks good to me :)
Reviewed-by: Bryan Brattlof bb@ti.com
Thanks for the quick fix ~Bryan

On 14/07/23 01:06, Nishanth Menon wrote:
Instead of hard-coding the count of entries manually, use ARRAY_SIZE to keep the count updates appropriately.
Cc: Bryan Brattlof bb@ti.com Suggested-by: Ravi Gunasekaran r-gunasekaran@ti.com Signed-off-by: Nishanth Menon nm@ti.com
Reviewed-by: Siddharth Vadapalli s-vadapalli@ti.com
Link to discussion that triggered this: https://lore.kernel.org/all/20230713185927.3h6vqap4x3h62zd5@bryanbrattlof.co...
arch/arm/mach-k3/am62ax/dev-data.c | 8 ++++---- arch/arm/mach-k3/am62x/dev-data.c | 8 ++++---- arch/arm/mach-k3/j7200/dev-data.c | 8 ++++---- arch/arm/mach-k3/j721e/dev-data.c | 8 ++++---- arch/arm/mach-k3/j721s2/dev-data.c | 8 ++++---- 5 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/arch/arm/mach-k3/am62ax/dev-data.c b/arch/arm/mach-k3/am62ax/dev-data.c index 74739c638524..abf5d8e91aa2 100644 --- a/arch/arm/mach-k3/am62ax/dev-data.c +++ b/arch/arm/mach-k3/am62ax/dev-data.c @@ -66,8 +66,8 @@ const struct ti_k3_pd_platdata am62ax_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list,
- .num_psc = 2,
- .num_pd = 4,
- .num_lpsc = 14,
- .num_devs = 19,
- .num_psc = ARRAY_SIZE(soc_psc_list),
- .num_pd = ARRAY_SIZE(soc_pd_list),
- .num_lpsc = ARRAY_SIZE(soc_lpsc_list),
- .num_devs = ARRAY_SIZE(soc_dev_list),
}; diff --git a/arch/arm/mach-k3/am62x/dev-data.c b/arch/arm/mach-k3/am62x/dev-data.c index 616d0650b9c0..1a6f9e2ca03e 100644 --- a/arch/arm/mach-k3/am62x/dev-data.c +++ b/arch/arm/mach-k3/am62x/dev-data.c @@ -71,8 +71,8 @@ const struct ti_k3_pd_platdata am62x_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list,
- .num_psc = 2,
- .num_pd = 5,
- .num_lpsc = 16,
- .num_devs = 21,
- .num_psc = ARRAY_SIZE(soc_psc_list),
- .num_pd = ARRAY_SIZE(soc_pd_list),
- .num_lpsc = ARRAY_SIZE(soc_lpsc_list),
- .num_devs = ARRAY_SIZE(soc_dev_list),
}; diff --git a/arch/arm/mach-k3/j7200/dev-data.c b/arch/arm/mach-k3/j7200/dev-data.c index c1a4dab6941a..4ddc34210ebb 100644 --- a/arch/arm/mach-k3/j7200/dev-data.c +++ b/arch/arm/mach-k3/j7200/dev-data.c @@ -75,8 +75,8 @@ const struct ti_k3_pd_platdata j7200_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list,
- .num_psc = 2,
- .num_pd = 6,
- .num_lpsc = 17,
- .num_devs = 23,
- .num_psc = ARRAY_SIZE(soc_psc_list),
- .num_pd = ARRAY_SIZE(soc_pd_list),
- .num_lpsc = ARRAY_SIZE(soc_lpsc_list),
- .num_devs = ARRAY_SIZE(soc_dev_list),
}; diff --git a/arch/arm/mach-k3/j721e/dev-data.c b/arch/arm/mach-k3/j721e/dev-data.c index f0afa3552b7f..97f017f8af50 100644 --- a/arch/arm/mach-k3/j721e/dev-data.c +++ b/arch/arm/mach-k3/j721e/dev-data.c @@ -73,8 +73,8 @@ const struct ti_k3_pd_platdata j721e_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list,
- .num_psc = 2,
- .num_pd = 5,
- .num_lpsc = 16,
- .num_devs = 23,
- .num_psc = ARRAY_SIZE(soc_psc_list),
- .num_pd = ARRAY_SIZE(soc_pd_list),
- .num_lpsc = ARRAY_SIZE(soc_lpsc_list),
- .num_devs = ARRAY_SIZE(soc_dev_list),
}; diff --git a/arch/arm/mach-k3/j721s2/dev-data.c b/arch/arm/mach-k3/j721s2/dev-data.c index 35e8b17eb1e7..8c999a3c5a8b 100644 --- a/arch/arm/mach-k3/j721s2/dev-data.c +++ b/arch/arm/mach-k3/j721s2/dev-data.c @@ -79,8 +79,8 @@ const struct ti_k3_pd_platdata j721s2_pd_platdata = { .pd = soc_pd_list, .lpsc = soc_lpsc_list, .devs = soc_dev_list,
- .num_psc = 2,
- .num_pd = 6,
- .num_lpsc = 19,
- .num_devs = 25,
- .num_psc = ARRAY_SIZE(soc_psc_list),
- .num_pd = ARRAY_SIZE(soc_pd_list),
- .num_lpsc = ARRAY_SIZE(soc_lpsc_list),
- .num_devs = ARRAY_SIZE(soc_dev_list),
};

On Thu, Jul 13, 2023 at 02:36:34PM -0500, Nishanth Menon wrote:
Instead of hard-coding the count of entries manually, use ARRAY_SIZE to keep the count updates appropriately.
Cc: Bryan Brattlof bb@ti.com Suggested-by: Ravi Gunasekaran r-gunasekaran@ti.com Signed-off-by: Nishanth Menon nm@ti.com Reviewed-by: Bryan Brattlof bb@ti.com Reviewed-by: Siddharth Vadapalli s-vadapalli@ti.com
Applied to u-boot/master, thanks!
participants (4)
-
Bryan Brattlof
-
Nishanth Menon
-
Siddharth Vadapalli
-
Tom Rini