Re: [PATCH] arm64, irqchip/gic-v3, ACPI: Move MADT GICC enabled check into a helper

From: Sudeep Holla <sudeep.holla@arm.com>
To: "Rafael J. Wysocki" <rafael@kernel.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>
Cc: Russell King <rmk+kernel@armlinux.org.uk>,
	Marc Zyngier <maz@kernel.org>, Len Brown <lenb@kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux-arm-kernel@lists.infradead.org, linux-acpi@vger.kernel.org
Subject: Re: [PATCH] arm64, irqchip/gic-v3, ACPI: Move MADT GICC enabled check into a helper
Date: Tue, 24 Oct 2023 15:07:47 +0100	[thread overview]
Message-ID: <ZTfPsyUbWCB0u96R@bogus> (raw)
In-Reply-To: <CAJZ5v0j+h9M4qhcdRW7_sOxbwdVqjyyrZWw14JsM1h1JoUL5+Q@mail.gmail.com>

On Tue, Oct 24, 2023 at 02:06:55PM +0200, Rafael J. Wysocki wrote:
> On Mon, Oct 23, 2023 at 3:35 PM Russell King <rmk+kernel@armlinux.org.uk> wrote:
> >
> > From: James Morse <james.morse@arm.com>
> >
> > ACPI, irqchip and the architecture code all inspect the MADT
> > enabled bit for a GICC entry in the MADT.
> >
> > The addition of an 'online capable' bit means all these sites need
> > updating.
> >
> > Move the current checks behind a helper to make future updates easier.
> >
> > Signed-off-by: James Morse <james.morse@arm.com>
> > Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > Reviewed-by: Gavin Shan <gshan@redhat.com>
> > Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
> 
> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> for the generic ACPI changes in this patch, but it is knod of
> ARM-specific, so I'd prefer ARM64 to pick it up (CC Sudeep).

Thanks Rafael.

The changes looks good to me.

Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>

Catalin,

Can you please up this patch for v6.7 if not too late ? It is simple
refactoring and must not have too much impact.

-- 
Regards,
Sudeep