On 14/07/2020 19:56, Bjorn Helgaas wrote: > [+cc Luc, linux-pci] > > I think what's necessary here is to replace "enum pci_channel_state > state" with "pci_channel_state_t state", as in 16d79cd4e23b ("PCI: Use > 'pci_channel_state_t' instead of 'enum pci_channel_state'"): > > https://git.kernel.org/cgit/linux/kernel/git/helgaas/pci.git/commit/?id=16d79cd4e23b > > This change should be safe even without 16d79cd4e23b, so there won't > be a revlock if you make it in the renesas-drivers tree. This is presumably goingto happen in any tree that pulls both net-next and pci/next. So shouldn't we make the change in net-next, rather than in a renesas tree that really has little to do with it other than being the first place the problem showed up? Apart from that question, the fix you describe sounds reasonable afaict, and I'll happily Ack a patch. -ed