This eliminates the following coccicheck warning: ./arch/s390/tools/gen_facilities.c:154:37-38: WARNING: Use ARRAY_SIZE ./arch/s390/tools/gen_opcode_table.c:141:39-40: WARNING: Use ARRAY_SIZE Reported-by: Abaci Robot <abaci@xxxxxxxxxxxxxxxxx> Signed-off-by: Yang Li <yang.lee@xxxxxxxxxxxxxxxxx> --- arch/s390/tools/gen_facilities.c | 2 +- arch/s390/tools/gen_opcode_table.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/s390/tools/gen_facilities.c b/arch/s390/tools/gen_facilities.c index 61ce5b5..5366817 100644 --- a/arch/s390/tools/gen_facilities.c +++ b/arch/s390/tools/gen_facilities.c @@ -151,7 +151,7 @@ static void print_facility_lists(void) { unsigned int i; - for (i = 0; i < sizeof(facility_defs) / sizeof(facility_defs[0]); i++) + for (i = 0; i < ARRAY_SIZE(facility_defs); i++) print_facility_list(&facility_defs[i]); } diff --git a/arch/s390/tools/gen_opcode_table.c b/arch/s390/tools/gen_opcode_table.c index a1bc02b..468b70c 100644 --- a/arch/s390/tools/gen_opcode_table.c +++ b/arch/s390/tools/gen_opcode_table.c @@ -138,7 +138,7 @@ static struct insn_type *insn_format_to_type(char *format) strcpy(tmp, format); base_format = tmp; base_format = strsep(&base_format, "_"); - for (i = 0; i < sizeof(insn_type_table) / sizeof(insn_type_table[0]); i++) { + for (i = 0; i < ARRAY_SIZE(insn_type_table); i++) { ptr = insn_type_table[i].format; while (*ptr) { if (!strcmp(base_format, *ptr)) -- 1.8.3.1