Re: [PATCH 3/5] PCI: cadence: Add host driver for Cadence PCIe controller

From: Bjorn Helgaas
Date: Wed Nov 29 2017 - 10:55:33 EST


On Wed, Nov 29, 2017 at 09:19:29AM +0100, Thomas Petazzoni wrote:
> Hello,
>
> On Tue, 28 Nov 2017 14:41:14 -0600, Bjorn Helgaas wrote:
>
> > > + * struct cdns_pcie_rc_data - hardware specific data
> >
> > "cdns" is a weird abbreviation for "Cadence", since "Cadence" doesn't
> > contain an "s".
>
> cdns is the official Device Tree binding vendor prefix for Cadence:
>
> $ grep Cadence Documentation/devicetree/bindings/vendor-prefixes.txt
> cdns Cadence Design Systems Inc.
>
> And it is already widely used throughout the kernel for Cadence
> drivers. See drivers/watchdog/cadence_wdt.c, drivers/spi/spi-cadence.c,
> drivers/i2c/busses/i2c-cadence.c, etc.

Hard to argue with that! Thanks!