From: gregory.clement@bootlin.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] dt-bindings: marvell, prestera: Add common compatible string
Date: Fri, 21 Sep 2018 13:07:32 +0200 [thread overview]
Message-ID: <87a7obhzwr.fsf@bootlin.com> (raw)
In-Reply-To: <20180907005926.27134-2-chris.packham@alliedtelesis.co.nz> (Chris Packham's message of "Fri, 7 Sep 2018 12:59:25 +1200")
Hi Chris,
On ven., sept. 07 2018, Chris Packham <chris.packham@alliedtelesis.co.nz> wrote:
> Add "marvell,prestera" as a compatible string so that drivers can be
> written to account for any prestera variant without needing to
> specialise to the more specific values.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Applied on mvebu/soc with Rob reviewed-by tag
Thanks,
Gregory
> ---
> Documentation/devicetree/bindings/net/marvell,prestera.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/marvell,prestera.txt b/Documentation/devicetree/bindings/net/marvell,prestera.txt
> index c329608fa887..83370ebf5b89 100644
> --- a/Documentation/devicetree/bindings/net/marvell,prestera.txt
> +++ b/Documentation/devicetree/bindings/net/marvell,prestera.txt
> @@ -2,7 +2,7 @@ Marvell Prestera Switch Chip bindings
> -------------------------------------
>
> Required properties:
> -- compatible: one of the following
> +- compatible: must be "marvell,prestera" and one of the following
> "marvell,prestera-98dx3236",
> "marvell,prestera-98dx3336",
> "marvell,prestera-98dx4251",
> @@ -21,7 +21,7 @@ switch {
> ranges = <0 MBUS_ID(0x03, 0x00) 0 0x100000>;
>
> packet-processor at 0 {
> - compatible = "marvell,prestera-98dx3236";
> + compatible = "marvell,prestera-98dx3236", "marvell,prestera";
> reg = <0 0x4000000>;
> interrupts = <33>, <34>, <35>;
> dfx = <&dfx>;
> --
> 2.18.0
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: Chris Packham <chris.packham@alliedtelesis.co.nz>
Cc: robh+dt@kernel.org, jason@lakedaemon.net, davem@davemloft.net,
andrew@lunn.ch, sebastian.hesselbarth@gmail.com,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
Mark Rutland <mark.rutland@arm.com>
Subject: Re: [PATCH 1/2] dt-bindings: marvell,prestera: Add common compatible string
Date: Fri, 21 Sep 2018 13:07:32 +0200 [thread overview]
Message-ID: <87a7obhzwr.fsf@bootlin.com> (raw)
In-Reply-To: <20180907005926.27134-2-chris.packham@alliedtelesis.co.nz> (Chris Packham's message of "Fri, 7 Sep 2018 12:59:25 +1200")
Hi Chris,
On ven., sept. 07 2018, Chris Packham <chris.packham@alliedtelesis.co.nz> wrote:
> Add "marvell,prestera" as a compatible string so that drivers can be
> written to account for any prestera variant without needing to
> specialise to the more specific values.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Applied on mvebu/soc with Rob reviewed-by tag
Thanks,
Gregory
> ---
> Documentation/devicetree/bindings/net/marvell,prestera.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/marvell,prestera.txt b/Documentation/devicetree/bindings/net/marvell,prestera.txt
> index c329608fa887..83370ebf5b89 100644
> --- a/Documentation/devicetree/bindings/net/marvell,prestera.txt
> +++ b/Documentation/devicetree/bindings/net/marvell,prestera.txt
> @@ -2,7 +2,7 @@ Marvell Prestera Switch Chip bindings
> -------------------------------------
>
> Required properties:
> -- compatible: one of the following
> +- compatible: must be "marvell,prestera" and one of the following
> "marvell,prestera-98dx3236",
> "marvell,prestera-98dx3336",
> "marvell,prestera-98dx4251",
> @@ -21,7 +21,7 @@ switch {
> ranges = <0 MBUS_ID(0x03, 0x00) 0 0x100000>;
>
> packet-processor@0 {
> - compatible = "marvell,prestera-98dx3236";
> + compatible = "marvell,prestera-98dx3236", "marvell,prestera";
> reg = <0 0x4000000>;
> interrupts = <33>, <34>, <35>;
> dfx = <&dfx>;
> --
> 2.18.0
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
next prev parent reply other threads:[~2018-09-21 11:07 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-07 0:59 [PATCH 0/2] ARM: dts: mvebu: marvell,prestera Chris Packham
2018-09-07 0:59 ` Chris Packham
2018-09-07 0:59 ` [PATCH 1/2] dt-bindings: marvell, prestera: Add common compatible string Chris Packham
2018-09-07 0:59 ` [PATCH 1/2] dt-bindings: marvell,prestera: " Chris Packham
2018-09-17 5:45 ` [PATCH 1/2] dt-bindings: marvell, prestera: " Rob Herring
2018-09-17 5:45 ` Rob Herring
2018-09-17 5:45 ` Rob Herring
2018-09-21 11:07 ` Gregory CLEMENT [this message]
2018-09-21 11:07 ` [PATCH 1/2] dt-bindings: marvell,prestera: " Gregory CLEMENT
2018-09-07 0:59 ` [PATCH 2/2] ARM: dts: mvebu: add "marvell,prestera" to PP nodes Chris Packham
2018-09-07 0:59 ` Chris Packham
2018-09-21 11:08 ` Gregory CLEMENT
2018-09-21 11:08 ` Gregory CLEMENT
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87a7obhzwr.fsf@bootlin.com \
--to=gregory.clement@bootlin.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.