Re: [PATCH v5] ACPI/IORT: Fix PMCG node single ID mapping handling

From: Will Deacon
Date: Wed May 20 2020 - 13:44:03 EST


On Wed, May 20, 2020 at 06:27:36PM +0100, Lorenzo Pieralisi wrote:
> On Wed, May 20, 2020 at 10:13:07AM -0700, Tuan Phan wrote:
> > An IORT PMCG node can have no ID mapping if its overflow interrupt is
> > wire based therefore the code that parses the PMCG node can not assume
> > the node will always have a single mapping present at index 0.
> >
> > Fix iort_get_id_mapping_index() by checking for an overflow interrupt
> > and mapping count.
> >
> > Fixes: 24e516049360 ("ACPI/IORT: Add support for PMCG")
> >
> > Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
> > Reviewed-by: Hanjun Guo <guoahanjun@xxxxxxxxxx>
> > Signed-off-by: Tuan Phan <tuanphan@xxxxxxxxxxxxxxxxxxxxxx>
> > ---
> > v1 -> v2:
> > - Use pmcg node to detect wired base overflow interrupt.
> >
> > v2 -> v3:
> > - Address Hanjun and Robin's comments.
> >
> > v3 -> v4:
> > - Update the title and description as mentioned by Lorenzo.
> >
> > v4 -> v5:
> > - Remove period in the title and commit references.
> >
> > drivers/acpi/arm64/iort.c | 5 +++++
>
> Hi Will,
>
> is there a chance we can get this patch into v5.8 ? I understand
> we are very late in the cycle but I wanted to ask (it applies cleanly
> to for-next/acpi).

Sorry, Lorenzo -- I didn't notice that this had been Acked already. I was
waiting for somebody to chime in! I'll queue it ASAP for 5.8.

Will