[Skiboot] [PATCH] hw/imc: Use ARRAY_SIZE instead of static macro
Madhavan Srinivasan
maddy at linux.vnet.ibm.com
Wed Oct 11 14:40:04 AEDT 2017
disable_unavailable_units() loops through nest_pmus array
to filter out the unsupported nest units from the imc
catalog dtb. Current code use a static macro ('MAX_NEST_UNITS')
for array limit, instead use ARRAY_SIZE. This will avoid updates
to static macro when updating the nest_pmus array.
Fixes: 712837cedca06 ('skiboot/imc: Update the nest_pmus array with occ/gpe microcode uav updates')
Signed-off-by: Madhavan Srinivasan <maddy at linux.vnet.ibm.com>
---
hw/imc.c | 2 +-
include/imc.h | 2 --
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/hw/imc.c b/hw/imc.c
index 10505f69d4bd..4b2e94884198 100644
--- a/hw/imc.c
+++ b/hw/imc.c
@@ -321,7 +321,7 @@ static void disable_unavailable_units(struct dt_node *dev)
avl_vec = (0xffULL) << 56;
}
- for (i = 0; i < MAX_NEST_UNITS; i++) {
+ for (i = 0; i < ARRAY_SIZE(nest_pmus); i++) {
if (!(PPC_BITMASK(i, i) & avl_vec)) {
/* Check if the device node exists */
target = dt_find_by_name(dev, nest_pmus[i]);
diff --git a/include/imc.h b/include/imc.h
index 9eaf804ceb50..f3d906ee51c6 100644
--- a/include/imc.h
+++ b/include/imc.h
@@ -119,8 +119,6 @@ struct imc_chip_cb
#define NEST_IMC_ENABLE 0x1
#define NEST_IMC_DISABLE 0x2
-#define MAX_NEST_UNITS 48
-
/*
* Core IMC SCOMs
*/
--
2.7.4
More information about the Skiboot
mailing list