diff mbox series

[v2,26/34] dt-bindings: add broadcom (formerly plx technology) vendor prefix

Message ID 20180831163817.23970-27-marcel@ziswiler.com
State Accepted
Headers show
Series ARM: dts: tegra: apalis_t30: major revamp incl. v1.1 hardware support | expand

Commit Message

Marcel Ziswiler Aug. 31, 2018, 4:38 p.m. UTC 
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

PLX Technology meanwhile got bought by Broadcom Corporation but the
vendor prefix plx is still used in 8 current device trees. This silences
the following checkpatch.pl warning:

WARNING: DT compatible string vendor "plx" appears un-documented
 -- check ./Documentation/devicetree/bindings/vendor-prefixes.txt

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

---

Changes in v2:
- Replace "[PATCH 27/28] ARM: tegra: apalis_t30: fix pcie switch vendor
  compatible" with "[PATCH v2 26/34] dt-bindings: add broadcom (formerly
  plx technology) vendor prefix" as suggested by Stefan.

 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Andreas Färber Aug. 31, 2018, 6:01 p.m. UTC | #1 
Am 31.08.2018 um 18:38 schrieb Marcel Ziswiler:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> PLX Technology meanwhile got bought by Broadcom Corporation but the
> vendor prefix plx is still used in 8 current device trees. This silences
> the following checkpatch.pl warning:
> 
> WARNING: DT compatible string vendor "plx" appears un-documented
>  -- check ./Documentation/devicetree/bindings/vendor-prefixes.txt
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

Sounds sensible,

Reviewed-by: Andreas Färber <afaerber@suse.de>

You may want to tweak the subject though, it is slightly confusing as
Broadcom already has a vendor prefix.

Cheers,
Andreas
Rob Herring Sept. 25, 2018, 6:24 p.m. UTC | #2 
On Fri, Aug 31, 2018 at 08:01:01PM +0200, Andreas Färber wrote:
> Am 31.08.2018 um 18:38 schrieb Marcel Ziswiler:
> > From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> > 
> > PLX Technology meanwhile got bought by Broadcom Corporation but the
> > vendor prefix plx is still used in 8 current device trees. This silences
> > the following checkpatch.pl warning:
> > 
> > WARNING: DT compatible string vendor "plx" appears un-documented
> >  -- check ./Documentation/devicetree/bindings/vendor-prefixes.txt
> > 
> > Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> Sounds sensible,
> 
> Reviewed-by: Andreas Färber <afaerber@suse.de>
> 
> You may want to tweak the subject though, it is slightly confusing as
> Broadcom already has a vendor prefix.

Yes, please.

With that,

Acked-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 2c3fc512e746..e40312dcba7d 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -297,6 +297,7 @@  pine64	Pine64
 pixcir  PIXCIR MICROELECTRONICS Co., Ltd
 plathome	Plat'Home Co., Ltd.
 plda	PLDA
+plx	Broadcom Corporation (formerly PLX Technology)
 portwell	Portwell Inc.
 poslab	Poslab Technology Co., Ltd.
 powervr	PowerVR (deprecated, use img)