> On Jan 4, 2021, at 2:05 PM, Keith Busch <kbusch@xxxxxxxxxx> wrote: > > 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. Never mind you only had one string, it was the order issue that the builedbot caught. Sean > > >>> + pci_is_root_bus(dev->bus) ? "Root" : "Downstream");
