* [PATCH] ACPI: tables: Use string choice helpers
@ 2025-01-03 3:51 Sunil V L
2025-01-07 18:14 ` Rafael J. Wysocki
0 siblings, 1 reply; 2+ messages in thread
From: Sunil V L @ 2025-01-03 3:51 UTC (permalink / raw)
To: linux-acpi, linux-kernel
Cc: Rafael J . Wysocki, Len Brown, Palmer Dabbelt, Andrew Jones,
Conor Dooley, Anup Patel, Sunil V L, kernel test robot,
Julia Lawall
Use str_enabled_disabled string helpers for better readability and to
fix cocci warning.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/r/202501010947.0e3GVHNa-lkp@intel.com/
Reported-by: Julia Lawall <julia.lawall@inria.fr>
Closes: https://lore.kernel.org/r/202501010947.0e3GVHNa-lkp@intel.com/
Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
---
drivers/acpi/tables.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
index 9e1b01c35070..2295abbecd14 100644
--- a/drivers/acpi/tables.c
+++ b/drivers/acpi/tables.c
@@ -56,7 +56,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
(struct acpi_madt_local_apic *)header;
pr_debug("LAPIC (acpi_id[0x%02x] lapic_id[0x%02x] %s)\n",
p->processor_id, p->id,
- (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
+ str_enabled_disabled(p->lapic_flags & ACPI_MADT_ENABLED));
}
break;
@@ -66,7 +66,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
(struct acpi_madt_local_x2apic *)header;
pr_debug("X2APIC (apic_id[0x%02x] uid[0x%02x] %s)\n",
p->local_apic_id, p->uid,
- (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
+ str_enabled_disabled(p->lapic_flags & ACPI_MADT_ENABLED));
}
break;
@@ -160,7 +160,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
(struct acpi_madt_local_sapic *)header;
pr_debug("LSAPIC (acpi_id[0x%02x] lsapic_id[0x%02x] lsapic_eid[0x%02x] %s)\n",
p->processor_id, p->id, p->eid,
- (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
+ str_enabled_disabled(p->lapic_flags & ACPI_MADT_ENABLED));
}
break;
@@ -183,7 +183,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
pr_debug("GICC (acpi_id[0x%04x] address[%llx] MPIDR[0x%llx] %s)\n",
p->uid, p->base_address,
p->arm_mpidr,
- (p->flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
+ str_enabled_disabled(p->flags & ACPI_MADT_ENABLED));
}
break;
@@ -218,7 +218,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
pr_debug("CORE PIC (processor_id[0x%02x] core_id[0x%02x] %s)\n",
p->processor_id, p->core_id,
- (p->flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
+ str_enabled_disabled(p->flags & ACPI_MADT_ENABLED));
}
break;
@@ -228,7 +228,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
pr_debug("RISC-V INTC (acpi_uid[0x%04x] hart_id[0x%llx] %s)\n",
p->uid, p->hart_id,
- (p->flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
+ str_enabled_disabled(p->flags & ACPI_MADT_ENABLED));
}
break;
--
2.43.0
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] ACPI: tables: Use string choice helpers
2025-01-03 3:51 [PATCH] ACPI: tables: Use string choice helpers Sunil V L
@ 2025-01-07 18:14 ` Rafael J. Wysocki
0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2025-01-07 18:14 UTC (permalink / raw)
To: Sunil V L
Cc: linux-acpi, linux-kernel, Rafael J . Wysocki, Len Brown,
Palmer Dabbelt, Andrew Jones, Conor Dooley, Anup Patel,
kernel test robot, Julia Lawall
On Fri, Jan 3, 2025 at 4:51 AM Sunil V L <sunilvl@ventanamicro.com> wrote:
>
> Use str_enabled_disabled string helpers for better readability and to
> fix cocci warning.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/r/202501010947.0e3GVHNa-lkp@intel.com/
> Reported-by: Julia Lawall <julia.lawall@inria.fr>
> Closes: https://lore.kernel.org/r/202501010947.0e3GVHNa-lkp@intel.com/
> Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
> ---
> drivers/acpi/tables.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
> index 9e1b01c35070..2295abbecd14 100644
> --- a/drivers/acpi/tables.c
> +++ b/drivers/acpi/tables.c
> @@ -56,7 +56,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
> (struct acpi_madt_local_apic *)header;
> pr_debug("LAPIC (acpi_id[0x%02x] lapic_id[0x%02x] %s)\n",
> p->processor_id, p->id,
> - (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
> + str_enabled_disabled(p->lapic_flags & ACPI_MADT_ENABLED));
> }
> break;
>
> @@ -66,7 +66,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
> (struct acpi_madt_local_x2apic *)header;
> pr_debug("X2APIC (apic_id[0x%02x] uid[0x%02x] %s)\n",
> p->local_apic_id, p->uid,
> - (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
> + str_enabled_disabled(p->lapic_flags & ACPI_MADT_ENABLED));
> }
> break;
>
> @@ -160,7 +160,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
> (struct acpi_madt_local_sapic *)header;
> pr_debug("LSAPIC (acpi_id[0x%02x] lsapic_id[0x%02x] lsapic_eid[0x%02x] %s)\n",
> p->processor_id, p->id, p->eid,
> - (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
> + str_enabled_disabled(p->lapic_flags & ACPI_MADT_ENABLED));
> }
> break;
>
> @@ -183,7 +183,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
> pr_debug("GICC (acpi_id[0x%04x] address[%llx] MPIDR[0x%llx] %s)\n",
> p->uid, p->base_address,
> p->arm_mpidr,
> - (p->flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
> + str_enabled_disabled(p->flags & ACPI_MADT_ENABLED));
>
> }
> break;
> @@ -218,7 +218,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
>
> pr_debug("CORE PIC (processor_id[0x%02x] core_id[0x%02x] %s)\n",
> p->processor_id, p->core_id,
> - (p->flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
> + str_enabled_disabled(p->flags & ACPI_MADT_ENABLED));
> }
> break;
>
> @@ -228,7 +228,7 @@ void acpi_table_print_madt_entry(struct acpi_subtable_header *header)
>
> pr_debug("RISC-V INTC (acpi_uid[0x%04x] hart_id[0x%llx] %s)\n",
> p->uid, p->hart_id,
> - (p->flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled");
> + str_enabled_disabled(p->flags & ACPI_MADT_ENABLED));
> }
> break;
>
> --
Applied as 6.14 material, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-01-07 18:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-01-03 3:51 [PATCH] ACPI: tables: Use string choice helpers Sunil V L
2025-01-07 18:14 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome