On Mon, Jan 04, 2021 at 06:42:58PM +0000, Kelley, Sean V wrote: > > On Dec 17, 2020, at 9:14 AM, Keith Busch <kbusch@xxxxxxxxxx> wrote: > > rc = pci_bus_error_reset(dev); > > - pci_info(dev, "Root Port link has been reset (%d)\n", rc); > > + pci_info(dev, "%s Port link has been reset (%d)\n", rc, > > Perhaps … "%s%s Port I am not sure I understand. What should the second string in this format say? I have to resend this patch anyway because I messed up the parmeter order and build bot caught me. I'll split it out from this patch since it is really a separate issue. > > + pci_is_root_bus(dev->bus) ? "Root" : "Downstream");