On Tue, Jul 11, 2017 at 12:45:43PM -0400, Mark Salter wrote: > The function acpi_gsi_to_irq must return 0 on success as the caller > ghes_probe expects an 0 for success. This change also matches x86 > implementation. > > This patch was submitted around 4.5 timeframe but wasn't pushed because > it didn't fix a real problem. Now that RAS/GHES patches are in kernel, > this fixes an error seen on a Mustang (arm64) platform:
Nit: I think you can rephrase the log, ie it is a bug and you are fixing it, that's it. > GHES: Failed to map GSI to IRQ for generic hardware error source: 2 > GHES: probe of GHES.2 failed with error 81 > > Signed-off-by: Tuan Phan <tp...@apm.com> > Signed-off-by: Loc Ho <l...@apm.com> > [rebased to v4.12-rc] > Signed-off-by: Mark Salter <msal...@redhat.com> > --- > drivers/acpi/irq.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Acked-by: Lorenzo Pieralisi <lorenzo.pieral...@arm.com> > diff --git a/drivers/acpi/irq.c b/drivers/acpi/irq.c > index 830299a..7c352cb 100644 > --- a/drivers/acpi/irq.c > +++ b/drivers/acpi/irq.c > @@ -24,7 +24,7 @@ static struct fwnode_handle *acpi_gsi_domain_id; > * > * irq location updated with irq value [>0 on success, 0 on failure] > * > - * Returns: linux IRQ number on success (>0) > + * Returns: 0 on success > * -EINVAL on failure > */ > int acpi_gsi_to_irq(u32 gsi, unsigned int *irq) > @@ -37,7 +37,7 @@ int acpi_gsi_to_irq(u32 gsi, unsigned int *irq) > * *irq == 0 means no mapping, that should > * be reported as a failure > */ > - return (*irq > 0) ? *irq : -EINVAL; > + return (*irq > 0) ? 0 : -EINVAL; > } > EXPORT_SYMBOL_GPL(acpi_gsi_to_irq); > > -- > 2.9.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
