On Wed, Feb 7, 2018 at 11:33 PM, Alexey Kardashevskiy <[email protected]> wrote: > Commit 59f47eff03a0 ("powerpc/pci: Use of_irq_parse_and_map_pci() helper") > correctly states that of_irq_parse_and_map_pci() does the same thing as > of_irq_parse_pci() does as it simply calls > of_irq_parse_pci() and irq_create_of_mapping(). > > However of_irq_parse_and_map_pci() not only returns 0 for success and > negative value for an error but also a positive virq value from > irq_create_of_mapping() which the mentioned commit ignores and > INTx config fails. > > This fixes the of_irq_parse_and_map_pci() return value handling. > > Fixes: 59f47eff03a0 "powerpc/pci: Use of_irq_parse_and_map_pci() helper" > Signed-off-by: Alexey Kardashevskiy <[email protected]> > --- > > Found it on POWER9 + powernv system - almost all devices suddenly lost > INTx support. > --- > arch/powerpc/kernel/pci-common.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-)
Thanks. Acked-by: Rob Herring <[email protected]>
