Bjorn Helgaas <helgaas@xxxxxxxxxx> writes: > On Fri, Feb 09, 2018 at 12:07:41PM -0600, Bjorn Helgaas wrote: >> On Fri, Feb 09, 2018 at 05:23:58PM +1100, Alexey Kardashevskiy wrote: >> > Commit 59f47eff03a0 ("powerpc/pci: Use of_irq_parse_and_map_pci() helper") >> > replaced of_irq_parse_pci() + irq_create_of_mapping() with >> > of_irq_parse_and_map_pci() but this change lost virq returned by >> > irq_create_of_mapping() so virq remained zero causing INTx >> > misconfiguration. >> > >> > This fixes pci_read_irq_line() not to loose a virq returned by >> > of_irq_parse_and_map_pci(). >> >> s/not to loose a/to not lose the/ >> >> > Fixes: 59f47eff03a0 "powerpc/pci: Use of_irq_parse_and_map_pci() helper" >> > Signed-off-by: Alexey Kardashevskiy <aik@xxxxxxxxx> >> >> I'm fine with this version. >> >> Since you started applying a previous version, Michael, I'll assume >> you will handle this unless you tell me otherwise. One way or another >> it would be good to get this in before -rc1. > > I went ahead and applied this and asked Linus to pull it. Thanks. cheers