[RESEND Patch 0/2] DT vendor/soc updates

[RESEND Patch 0/2] DT vendor/soc updates

[Andrew Lunn]

This is a resend because i've been told they did not make it to the
list properly.

Andrew Lunn (2):
  DT: Vendor: Add prefixes used by Kirkwood devices
  DT: bindings: add missing Marvell Kirkwood SoC documentation

 .../devicetree/bindings/arm/marvell,kirkwood.txt   | 97 ++++++++++++++++++++++
 .../devicetree/bindings/vendor-prefixes.txt        | 13 +++
 2 files changed, 110 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/arm/marvell,kirkwood.txt

-- 
1.9.0

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[RESEND Patch 1/2] DT: Vendor: Add prefixes used by Kirkwood devices

[Andrew Lunn]

Add a number of vendor prefixes by kirkwood devices. These are not all
stock tickers, but have been in use for a while so changing would not
be easy.

Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index c73b435f58f5..860c467374fe 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -20,2 +20,3 @@ bosch	Bosch Sensortec GmbH
 brcm	Broadcom Corporation
+buffalo	Buffalo, Inc.
 capella	Capella Microsystems, Inc
@@ -30,2 +31,3 @@ davicom	DAVICOM Semiconductor, Inc.
 denx	Denx Software Engineering
+dlink	D-Link Corporation
 edt	Emerging Display Technologies
@@ -35,2 +37,3 @@ epson	Seiko Epson Corp.
 est	ESTeem Wireless Modems
+excito	Excito
 fsl	Freescale Semiconductor
@@ -39,2 +42,3 @@ gef	GE Fanuc Intelligent Platforms Embedded Systems, Inc.
 gmt	Global Mixed-mode Technology, Inc.
+globalscale	GlobalScale Technologies, Inc.
 gumstix	Gumstix, Inc.
@@ -45,2 +49,3 @@ ibm	International Business Machines (IBM)
 idt	Integrated Device Technologies, Inc.
+iom	Iomega Corporation
 img	Imagination Technologies Ltd.
@@ -49,2 +54,4 @@ isl	Intersil
 karo	Ka-Ro electronics GmbH
+keymile	Keymile GmbH
+lacie	LaCie
 lg	LG Corporation
@@ -56,3 +63,5 @@ microchip	Microchip Technology Inc.
 mosaixtech	Mosaix Technologies, Inc.
+mpl	MPL AG
 national	National Semiconductor
+netgear	Netgear, Inc.
 nintendo	Nintendo
@@ -64,2 +73,3 @@ phytec	PHYTEC Messtechnik GmbH
 picochip	Picochip Ltd
+plathome	Plat'Home Co., Ltd.
 powervr	PowerVR (deprecated, use img)
@@ -68,2 +78,3 @@ qcom	Qualcomm, Inc.
 qnap	QNAP Systems, Inc.
+raidsonic	RaidSonic Technology GmbH
 ralink	Mediatek/Ralink Technology Corp.
@@ -77,2 +88,3 @@ sbs	Smart Battery System
 schindler	Schindler
+seagate	Seagate Technology PLC
 sil	Silicon Image
@@ -97 +109,2 @@ wm	Wondermedia Technologies, Inc.
 xlnx	Xilinx
+zyxel	ZyXEL Communications Corp.
-- 
1.9.0

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [RESEND Patch 1/2] DT: Vendor: Add prefixes used by Kirkwood devices

[Arnaud Ebalard]

Hi Andrew,

Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> writes:

> Add a number of vendor prefixes by kirkwood devices. These are not all
> stock tickers, but have been in use for a while so changing would not
> be easy.
>
> Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index c73b435f58f5..860c467374fe 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -20,2 +20,3 @@ bosch	Bosch Sensortec GmbH
>  brcm	Broadcom Corporation
> +buffalo	Buffalo, Inc.
>  capella	Capella Microsystems, Inc
> @@ -30,2 +31,3 @@ davicom	DAVICOM Semiconductor, Inc.
>  denx	Denx Software Engineering
> +dlink	D-Link Corporation
>  edt	Emerging Display Technologies
> @@ -35,2 +37,3 @@ epson	Seiko Epson Corp.
>  est	ESTeem Wireless Modems
> +excito	Excito
>  fsl	Freescale Semiconductor
> @@ -39,2 +42,3 @@ gef	GE Fanuc Intelligent Platforms Embedded Systems, Inc.
>  gmt	Global Mixed-mode Technology, Inc.
> +globalscale	GlobalScale Technologies, Inc.
>  gumstix	Gumstix, Inc.
> @@ -45,2 +49,3 @@ ibm	International Business Machines (IBM)
>  idt	Integrated Device Technologies, Inc.
> +iom	Iomega Corporation
>  img	Imagination Technologies Ltd.
> @@ -49,2 +54,4 @@ isl	Intersil
>  karo	Ka-Ro electronics GmbH
> +keymile	Keymile GmbH
> +lacie	LaCie
>  lg	LG Corporation
> @@ -56,3 +63,5 @@ microchip	Microchip Technology Inc.
>  mosaixtech	Mosaix Technologies, Inc.
> +mpl	MPL AG
>  national	National Semiconductor
> +netgear	Netgear, Inc.

I do not have a strong opinion on it but I think I read on LAKML there
was some kind of rule to use ticker symbol for a given vendor if it
exists e.g. ntgr for netgear (like fsl for freescale, brcm for broadcom,
etc).

Cheers,

a+


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [RESEND Patch 1/2] DT: Vendor: Add prefixes used by Kirkwood devices

[Jason Cooper]

On Thu, Mar 06, 2014 at 01:21:03PM +0100, Arnaud Ebalard wrote:
> Hi Andrew,
> 
> Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> writes:
> 
> > Add a number of vendor prefixes by kirkwood devices. These are not all
> > stock tickers, but have been in use for a while so changing would not
> > be easy.
> >
> > Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
> > ---
> >  Documentation/devicetree/bindings/vendor-prefixes.txt | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > index c73b435f58f5..860c467374fe 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > @@ -20,2 +20,3 @@ bosch	Bosch Sensortec GmbH
> >  brcm	Broadcom Corporation
> > +buffalo	Buffalo, Inc.
> >  capella	Capella Microsystems, Inc
> > @@ -30,2 +31,3 @@ davicom	DAVICOM Semiconductor, Inc.
> >  denx	Denx Software Engineering
> > +dlink	D-Link Corporation
> >  edt	Emerging Display Technologies
> > @@ -35,2 +37,3 @@ epson	Seiko Epson Corp.
> >  est	ESTeem Wireless Modems
> > +excito	Excito
> >  fsl	Freescale Semiconductor
> > @@ -39,2 +42,3 @@ gef	GE Fanuc Intelligent Platforms Embedded Systems, Inc.
> >  gmt	Global Mixed-mode Technology, Inc.
> > +globalscale	GlobalScale Technologies, Inc.
> >  gumstix	Gumstix, Inc.
> > @@ -45,2 +49,3 @@ ibm	International Business Machines (IBM)
> >  idt	Integrated Device Technologies, Inc.
> > +iom	Iomega Corporation
> >  img	Imagination Technologies Ltd.
> > @@ -49,2 +54,4 @@ isl	Intersil
> >  karo	Ka-Ro electronics GmbH
> > +keymile	Keymile GmbH
> > +lacie	LaCie
> >  lg	LG Corporation
> > @@ -56,3 +63,5 @@ microchip	Microchip Technology Inc.
> >  mosaixtech	Mosaix Technologies, Inc.
> > +mpl	MPL AG
> >  national	National Semiconductor
> > +netgear	Netgear, Inc.
> 
> I do not have a strong opinion on it but I think I read on LAKML there
> was some kind of rule to use ticker symbol for a given vendor if it
> exists e.g. ntgr for netgear (like fsl for freescale, brcm for broadcom,
> etc).

For new symbols, we do try to use the ticker when available.  However,
it's more important not to break compatibility with DT info already in
the field.

thx,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [RESEND Patch 1/2] DT: Vendor: Add prefixes used by Kirkwood devices

[Andrew Lunn]

On Thu, Mar 06, 2014 at 01:21:03PM +0100, Arnaud Ebalard wrote:
> Hi Andrew,
> 
> Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> writes:
> 
> > Add a number of vendor prefixes by kirkwood devices. These are not all
> > stock tickers, but have been in use for a while so changing would not
> > be easy.
> >
> > Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
> > ---
> >  Documentation/devicetree/bindings/vendor-prefixes.txt | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > index c73b435f58f5..860c467374fe 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > @@ -20,2 +20,3 @@ bosch	Bosch Sensortec GmbH
> >  brcm	Broadcom Corporation
> > +buffalo	Buffalo, Inc.
> >  capella	Capella Microsystems, Inc
> > @@ -30,2 +31,3 @@ davicom	DAVICOM Semiconductor, Inc.
> >  denx	Denx Software Engineering
> > +dlink	D-Link Corporation
> >  edt	Emerging Display Technologies
> > @@ -35,2 +37,3 @@ epson	Seiko Epson Corp.
> >  est	ESTeem Wireless Modems
> > +excito	Excito
> >  fsl	Freescale Semiconductor
> > @@ -39,2 +42,3 @@ gef	GE Fanuc Intelligent Platforms Embedded Systems, Inc.
> >  gmt	Global Mixed-mode Technology, Inc.
> > +globalscale	GlobalScale Technologies, Inc.
> >  gumstix	Gumstix, Inc.
> > @@ -45,2 +49,3 @@ ibm	International Business Machines (IBM)
> >  idt	Integrated Device Technologies, Inc.
> > +iom	Iomega Corporation
> >  img	Imagination Technologies Ltd.
> > @@ -49,2 +54,4 @@ isl	Intersil
> >  karo	Ka-Ro electronics GmbH
> > +keymile	Keymile GmbH
> > +lacie	LaCie
> >  lg	LG Corporation
> > @@ -56,3 +63,5 @@ microchip	Microchip Technology Inc.
> >  mosaixtech	Mosaix Technologies, Inc.
> > +mpl	MPL AG
> >  national	National Semiconductor
> > +netgear	Netgear, Inc.
> 
> I do not have a strong opinion on it but I think I read on LAKML there
> was some kind of rule to use ticker symbol for a given vendor if it
> exists e.g. ntgr for netgear (like fsl for freescale, brcm for broadcom,
> etc).

What you say is true, we try to use stock tickers. However, in the
changelog i say:

    Add a number of vendor prefixes by kirkwood devices. These are not
    all stock tickers, but have been in use for a while so changing
    would not be easy.

We have the problem that we have DTs in the field making use of these
vendor prefixes. They have effectively become part of the ABI. We have
to support them. Even if we do change to stock tickers, we still need
to support the old vendor prefix as well. So since it is just a rule
of thumb to use the stock ticker, not a strict rule, i've kept with
what we have.

     Andrew
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[RESEND Patch 2/2] DT: bindings: add missing Marvell Kirkwood SoC documentation

[Andrew Lunn]

Marvell Dove SoC binding was not documented, yet. Add the
documentation including the supported SoCs and boards.

Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
---
 .../devicetree/bindings/arm/marvell,kirkwood.txt   | 97 ++++++++++++++++++++++
 1 file changed, 97 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/arm/marvell,kirkwood.txt

diff --git a/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt b/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
new file mode 100644
index 000000000000..925ecbf6e7b7
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
@@ -0,0 +1,97 @@
+Marvell Kirkwood SoC Family Device Tree Bindings
+------------------------------------------------
+
+Boards with a SoC of the Marvell Kirkwook family, eg 88f6281
+
+* Required root node properties:
+compatible: must contain "marvell,kirkwood"
+
+In addition, the above compatible shall be extended with the specific
+SoC. Currently known SoC compatibles are:
+
+"marvell,kirkwood-88f6192"
+"marvell,kirkwood-88f6281"
+"marvell,kirkwood-88f6282"
+"marvell,kirkwood-88f6283"
+"marvell,kirkwood-88f6702"
+"marvell,kirkwood-98DX4122"
+
+And in addition, the compatible shall be extended with the specific
+board. Currently known boards are:
+
+"buffalo,lschlv2"
+"buffalo,lsxhl"
+"buffalo,lsxl"
+"dlink,dns-320"
+"dlink,dns-320-a1"
+"dlink,dns-325"
+"dlink,dns-325-a1"
+"dlink,dns-kirkwood"
+"excito,b3"
+"globalscale,dreamplug-003-ds2001"
+"globalscale,guruplug"
+"globalscale,guruplug-server-plus"
+"globalscale,sheevaplug"
+"globalscale,sheevaplug"
+"globalscale,sheevaplug-esata"
+"globalscale,sheevaplug-esata-rev13"
+"iom,iconnect"
+"iom,iconnect-1.1"
+"iom,ix2-200"
+"keymile,km_kirkwood"
+"lacie,cloudbox"
+"lacie,inetspace_v2"
+"lacie,laplug"
+"lacie,netspace_lite_v2"
+"lacie,netspace_max_v2"
+"lacie,netspace_mini_v2"
+"lacie,netspace_v2"
+"marvell,db-88f6281-bp"
+"marvell,db-88f6282-bp"
+"marvell,mv88f6281gtw-ge"
+"marvell,rd88f6281"
+"marvell,rd88f6281"
+"marvell,rd88f6281-a0"
+"marvell,rd88f6281-a1"
+"mpl,cec4"
+"mpl,cec4-10"
+"netgear,readynas"
+"netgear,readynas"
+"netgear,readynas-duo-v2"
+"netgear,readynas-nv+-v2"
+"plathome,openblocks-a6"
+"plathome,openblocks-a7"
+"raidsonic,ib-nas6210"
+"raidsonic,ib-nas6210-b"
+"raidsonic,ib-nas6220"
+"raidsonic,ib-nas6220-b"
+"raidsonic,ib-nas62x0"
+"seagate,dockstar"
+"seagate,goflexnet"
+"synology,ds109"
+"synology,ds110jv10"
+"synology,ds110jv20"
+"synology,ds110jv30"
+"synology,ds111"
+"synology,ds209"
+"synology,ds210jv10"
+"synology,ds210jv20"
+"synology,ds212"
+"synology,ds212jv10"
+"synology,ds212jv20"
+"synology,ds212pv10"
+"synology,ds409"
+"synology,ds409slim"
+"synology,ds410j"
+"synology,ds411"
+"synology,ds411j"
+"synology,ds411slim"
+"synology,ds413jv10"
+"synology,rs212"
+"synology,rs409"
+"synology,rs411"
+"synology,rs812"
+"usi,topkick"
+"usi,topkick-1281P2"
+"zyxel,nsa310"
+"zyxel,nsa310a"
-- 
1.9.0

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [RESEND Patch 2/2] DT: bindings: add missing Marvell Kirkwood SoC documentation

[Jason Cooper]

On Wed, Mar 05, 2014 at 10:21:14PM +0100, Andrew Lunn wrote:
> Marvell Dove SoC binding was not documented, yet. Add the

s/Dove/Kirkwood/

> documentation including the supported SoCs and boards.
> 
> Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
> ---
>  .../devicetree/bindings/arm/marvell,kirkwood.txt   | 97 ++++++++++++++++++++++
>  1 file changed, 97 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
> 
> diff --git a/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt b/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
> new file mode 100644
> index 000000000000..925ecbf6e7b7
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
> @@ -0,0 +1,97 @@
> +Marvell Kirkwood SoC Family Device Tree Bindings
> +------------------------------------------------
> +
> +Boards with a SoC of the Marvell Kirkwook family, eg 88f6281

s/Kirkwook/Kirkwood/

> +
> +* Required root node properties:
> +compatible: must contain "marvell,kirkwood"
> +
> +In addition, the above compatible shall be extended with the specific
> +SoC. Currently known SoC compatibles are:
> +
> +"marvell,kirkwood-88f6192"
> +"marvell,kirkwood-88f6281"
> +"marvell,kirkwood-88f6282"
> +"marvell,kirkwood-88f6283"
> +"marvell,kirkwood-88f6702"
> +"marvell,kirkwood-98DX4122"
> +
> +And in addition, the compatible shall be extended with the specific
> +board. Currently known boards are:

This is more a question for Grant and the other DT maintainers: Do we
really need this?  I understand documenting the manufacturer names, but
this looks like an unnecessary maintenance headache.

If the end goal of a separate devicetree repository includes the dts{i}
files and the binding docs in one repository, why do we need to maintain
this list?

thx,

Jason.

> +
> +"buffalo,lschlv2"
> +"buffalo,lsxhl"
> +"buffalo,lsxl"
> +"dlink,dns-320"
> +"dlink,dns-320-a1"
> +"dlink,dns-325"
> +"dlink,dns-325-a1"
> +"dlink,dns-kirkwood"
> +"excito,b3"
> +"globalscale,dreamplug-003-ds2001"
> +"globalscale,guruplug"
> +"globalscale,guruplug-server-plus"
> +"globalscale,sheevaplug"
> +"globalscale,sheevaplug"
> +"globalscale,sheevaplug-esata"
> +"globalscale,sheevaplug-esata-rev13"
> +"iom,iconnect"
> +"iom,iconnect-1.1"
> +"iom,ix2-200"
> +"keymile,km_kirkwood"
> +"lacie,cloudbox"
> +"lacie,inetspace_v2"
> +"lacie,laplug"
> +"lacie,netspace_lite_v2"
> +"lacie,netspace_max_v2"
> +"lacie,netspace_mini_v2"
> +"lacie,netspace_v2"
> +"marvell,db-88f6281-bp"
> +"marvell,db-88f6282-bp"
> +"marvell,mv88f6281gtw-ge"
> +"marvell,rd88f6281"
> +"marvell,rd88f6281"
> +"marvell,rd88f6281-a0"
> +"marvell,rd88f6281-a1"
> +"mpl,cec4"
> +"mpl,cec4-10"
> +"netgear,readynas"
> +"netgear,readynas"
> +"netgear,readynas-duo-v2"
> +"netgear,readynas-nv+-v2"
> +"plathome,openblocks-a6"
> +"plathome,openblocks-a7"
> +"raidsonic,ib-nas6210"
> +"raidsonic,ib-nas6210-b"
> +"raidsonic,ib-nas6220"
> +"raidsonic,ib-nas6220-b"
> +"raidsonic,ib-nas62x0"
> +"seagate,dockstar"
> +"seagate,goflexnet"
> +"synology,ds109"
> +"synology,ds110jv10"
> +"synology,ds110jv20"
> +"synology,ds110jv30"
> +"synology,ds111"
> +"synology,ds209"
> +"synology,ds210jv10"
> +"synology,ds210jv20"
> +"synology,ds212"
> +"synology,ds212jv10"
> +"synology,ds212jv20"
> +"synology,ds212pv10"
> +"synology,ds409"
> +"synology,ds409slim"
> +"synology,ds410j"
> +"synology,ds411"
> +"synology,ds411j"
> +"synology,ds411slim"
> +"synology,ds413jv10"
> +"synology,rs212"
> +"synology,rs409"
> +"synology,rs411"
> +"synology,rs812"
> +"usi,topkick"
> +"usi,topkick-1281P2"
> +"zyxel,nsa310"
> +"zyxel,nsa310a"
> -- 
> 1.9.0
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [RESEND Patch 2/2] DT: bindings: add missing Marvell Kirkwood SoC documentation

[Andrew Lunn]

On Thu, Mar 06, 2014 at 07:23:00AM -0500, Jason Cooper wrote:
> On Wed, Mar 05, 2014 at 10:21:14PM +0100, Andrew Lunn wrote:
> > Marvell Dove SoC binding was not documented, yet. Add the
> 
> s/Dove/Kirkwood/

Doh. I cut/paste from the Dove file.
 
> > documentation including the supported SoCs and boards.
> > 
> > Signed-off-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
> > ---
> >  .../devicetree/bindings/arm/marvell,kirkwood.txt   | 97 ++++++++++++++++++++++
> >  1 file changed, 97 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt b/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
> > new file mode 100644
> > index 000000000000..925ecbf6e7b7
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/arm/marvell,kirkwood.txt
> > @@ -0,0 +1,97 @@
> > +Marvell Kirkwood SoC Family Device Tree Bindings
> > +------------------------------------------------
> > +
> > +Boards with a SoC of the Marvell Kirkwook family, eg 88f6281
> 
> s/Kirkwook/Kirkwood/

I really need to add Kirkwood to my dictionary.
 
> > +
> > +* Required root node properties:
> > +compatible: must contain "marvell,kirkwood"
> > +
> > +In addition, the above compatible shall be extended with the specific
> > +SoC. Currently known SoC compatibles are:
> > +
> > +"marvell,kirkwood-88f6192"
> > +"marvell,kirkwood-88f6281"
> > +"marvell,kirkwood-88f6282"
> > +"marvell,kirkwood-88f6283"
> > +"marvell,kirkwood-88f6702"
> > +"marvell,kirkwood-98DX4122"
> > +
> > +And in addition, the compatible shall be extended with the specific
> > +board. Currently known boards are:
> 
> This is more a question for Grant and the other DT maintainers: Do we
> really need this?

checkpatch.pl says yes, we need this. It gives warnings for every
unknown compatible string. I would say it is up to Grant and the other
DT maintainers to decide if checkpatch.pl is wrong.

       Andrew
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html