On Wed, Sep 13, 2023 at 04:38:18PM +0000, James Morse wrote: > static inline bool acpi_gicc_is_usable(struct acpi_madt_generic_interrupt *gicc) > { > - return (gicc->flags & ACPI_MADT_ENABLED); > + return ((gicc->flags & ACPI_MADT_ENABLED || > + gicc->flags & ACPI_MADT_GICC_CPU_CAPABLE)); ... and this starts getting silly with the number of parens. return gicc->flags & ACPI_MADT_ENABLED || gicc->flags & ACPI_MADT_GICC_CPU_CAPABLE; is entirely sufficient. Also: return gicc->flags & (ACPI_MADT_ENABLED | ACPI_MADT_GICC_CPU_CAPABLE); also works. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!