On Thu, May 02, 2019 at 08:59:39PM -0700, Frederick Lawler wrote: > +#define dev_fmt(fmt) "DPC: " fmt > + > @@ -110,7 +111,7 @@ static int dpc_wait_rp_inactive(struct dpc_dev *dpc) > + pci_warn(pdev, "DPC root port still busy\n"); > @@ -229,18 +229,17 @@ static irqreturn_t dpc_handler(int irq, void *context) > + pci_warn(pdev, "DPC %s detected\n", > @@ -328,11 +327,11 @@ static int dpc_probe(struct pcie_device *dev) > + pci_info(pdev, "DPC error containment capabilities: Int Msg #%d, RPExt%c PoisonedTLP%c SwTrigger%c RP PIO Log %d, DL_ActiveErr%c\n", Since you've already prefixed each print with "DPC: ", the extra "DPC" in the above prints is redundant.
