Messages in this thread |  | | Subject | Re: [PATCH v2 7/9] PCI: dwc: Replace lower into upper case characters | From | Gustavo Pimentel <> | Date | Mon, 9 Apr 2018 14:10:44 +0100 |
| |
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
|  |