Hi Joe, On 09/04/2018 11:25, Joe Perches wrote: > On Mon, 2018-04-09 at 10:41 +0100, Gustavo Pimentel wrote: >> Replaces lower into upper case characters in comments and debug printks. >> >> This is an attempt to keep the messages coherent within the designware >> driver. > > It'd be nice to make all the dwc drivers use the same > message wording. > > For instance: > > drivers/pci/dwc/pci-keystone.c: dev_err(dev, "phy link never came up\n"); > drivers/pci/dwc/pcie-designware.c: dev_err(pci->dev, "phy link never came up\n"); > drivers/pci/host/pcie-xilinx-nwl.c: dev_err(dev, "PHY link never came up\n"); I also agree, I added a task to my backlog to fix that. However, this task shouldn't be done in this patch series since it's a more broader task involving many drivers and different maintainers. Agree? > Regards, Gustavo