* [PATCH 0/2] ARM: dts: mvebu: marvell,prestera
@ 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 2/2] ARM: dts: mvebu: add "marvell,prestera" to PP nodes Chris Packham
0 siblings, 2 replies; 6+ messages in thread
From: Chris Packham @ 2018-09-07 0:59 UTC (permalink / raw)
To: robh+dt, gregory.clement
Cc: jason, davem, andrew, sebastian.hesselbarth, devicetree,
linux-arm-kernel, linux-kernel, netdev, Chris Packham
Define a generic compatible string so that drivers don't need to deal with the
specific variants.
Chris Packham (2):
dt-bindings: marvell,prestera: Add common compatible string
ARM: dts: mvebu: add "marvell,prestera" to PP nodes
Documentation/devicetree/bindings/net/marvell,prestera.txt | 4 ++--
arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 2 +-
arch/arm/boot/dts/armada-xp-98dx3336.dtsi | 2 +-
arch/arm/boot/dts/armada-xp-98dx4251.dtsi | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
--
2.18.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] dt-bindings: marvell,prestera: Add common compatible string
2018-09-07 0:59 [PATCH 0/2] ARM: dts: mvebu: marvell,prestera Chris Packham
@ 2018-09-07 0:59 ` Chris Packham
2018-09-17 5:45 ` [PATCH 1/2] dt-bindings: marvell, prestera: " Rob Herring
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
1 sibling, 2 replies; 6+ messages in thread
From: Chris Packham @ 2018-09-07 0:59 UTC (permalink / raw)
To: robh+dt, gregory.clement
Cc: jason, davem, andrew, sebastian.hesselbarth, devicetree,
linux-arm-kernel, linux-kernel, netdev, Chris Packham,
Mark Rutland
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>
---
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
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] ARM: dts: mvebu: add "marvell,prestera" to PP nodes
2018-09-07 0:59 [PATCH 0/2] ARM: dts: mvebu: marvell,prestera 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 ` Chris Packham
2018-09-21 11:08 ` Gregory CLEMENT
1 sibling, 1 reply; 6+ messages in thread
From: Chris Packham @ 2018-09-07 0:59 UTC (permalink / raw)
To: robh+dt, gregory.clement
Cc: jason, davem, andrew, sebastian.hesselbarth, devicetree,
linux-arm-kernel, linux-kernel, netdev, Chris Packham,
Mark Rutland
The compatible string "marvell,prestera" allows drivers to have code
common to any prestera variant.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 2 +-
arch/arm/boot/dts/armada-xp-98dx3336.dtsi | 2 +-
arch/arm/boot/dts/armada-xp-98dx4251.dtsi | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/armada-xp-98dx3236.dtsi b/arch/arm/boot/dts/armada-xp-98dx3236.dtsi
index 8d708cc22495..2185ea58abfe 100644
--- a/arch/arm/boot/dts/armada-xp-98dx3236.dtsi
+++ b/arch/arm/boot/dts/armada-xp-98dx3236.dtsi
@@ -243,7 +243,7 @@
ranges = <0 MBUS_ID(0x03, 0x00) 0 0x100000>;
pp0: packet-processor@0 {
- compatible = "marvell,prestera-98dx3236";
+ compatible = "marvell,prestera-98dx3236", "marvell,prestera";
reg = <0 0x4000000>;
interrupts = <33>, <34>, <35>;
dfx = <&dfx>;
diff --git a/arch/arm/boot/dts/armada-xp-98dx3336.dtsi b/arch/arm/boot/dts/armada-xp-98dx3336.dtsi
index 2f5fc67dd6dc..1d9d8a8ea60c 100644
--- a/arch/arm/boot/dts/armada-xp-98dx3336.dtsi
+++ b/arch/arm/boot/dts/armada-xp-98dx3336.dtsi
@@ -35,5 +35,5 @@
};
&pp0 {
- compatible = "marvell,prestera-98dx3336";
+ compatible = "marvell,prestera-98dx3336", "marvell,prestera";
};
diff --git a/arch/arm/boot/dts/armada-xp-98dx4251.dtsi b/arch/arm/boot/dts/armada-xp-98dx4251.dtsi
index 7a9e8839880b..48ffdc72bfc7 100644
--- a/arch/arm/boot/dts/armada-xp-98dx4251.dtsi
+++ b/arch/arm/boot/dts/armada-xp-98dx4251.dtsi
@@ -49,6 +49,6 @@
};
&pp0 {
- compatible = "marvell,prestera-98dx4251";
+ compatible = "marvell,prestera-98dx4251", "marvell,prestera";
interrupts = <33>, <34>, <35>, <36>;
};
--
2.18.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] dt-bindings: marvell, prestera: Add common compatible string
2018-09-07 0:59 ` [PATCH 1/2] dt-bindings: marvell,prestera: Add common compatible string Chris Packham
@ 2018-09-17 5:45 ` Rob Herring
2018-09-21 11:07 ` [PATCH 1/2] dt-bindings: marvell,prestera: " Gregory CLEMENT
1 sibling, 0 replies; 6+ messages in thread
From: Rob Herring @ 2018-09-17 5:45 UTC (permalink / raw)
To: Chris Packham
Cc: Mark Rutland, andrew, jason, devicetree, netdev, gregory.clement,
linux-kernel, robh+dt, Chris Packham, davem, linux-arm-kernel,
sebastian.hesselbarth
On Fri, 7 Sep 2018 12:59:25 +1200, Chris Packham 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>
> ---
> Documentation/devicetree/bindings/net/marvell,prestera.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] dt-bindings: marvell,prestera: Add common compatible string
2018-09-07 0:59 ` [PATCH 1/2] dt-bindings: marvell,prestera: Add common compatible string Chris Packham
2018-09-17 5:45 ` [PATCH 1/2] dt-bindings: marvell, prestera: " Rob Herring
@ 2018-09-21 11:07 ` Gregory CLEMENT
1 sibling, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2018-09-21 11:07 UTC (permalink / raw)
To: Chris Packham
Cc: robh+dt, jason, davem, andrew, sebastian.hesselbarth, devicetree,
linux-arm-kernel, linux-kernel, netdev, Mark Rutland
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] ARM: dts: mvebu: add "marvell,prestera" to PP nodes
2018-09-07 0:59 ` [PATCH 2/2] ARM: dts: mvebu: add "marvell,prestera" to PP nodes Chris Packham
@ 2018-09-21 11:08 ` Gregory CLEMENT
0 siblings, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2018-09-21 11:08 UTC (permalink / raw)
To: Chris Packham
Cc: robh+dt, jason, davem, andrew, sebastian.hesselbarth, devicetree,
linux-arm-kernel, linux-kernel, netdev, Mark Rutland
Hi Chris,
On ven., sept. 07 2018, Chris Packham <chris.packham@alliedtelesis.co.nz> wrote:
> The compatible string "marvell,prestera" allows drivers to have code
> common to any prestera variant.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Applied on mvebu/dt
Thanks,
Gregory
> ---
> arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 2 +-
> arch/arm/boot/dts/armada-xp-98dx3336.dtsi | 2 +-
> arch/arm/boot/dts/armada-xp-98dx4251.dtsi | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-xp-98dx3236.dtsi b/arch/arm/boot/dts/armada-xp-98dx3236.dtsi
> index 8d708cc22495..2185ea58abfe 100644
> --- a/arch/arm/boot/dts/armada-xp-98dx3236.dtsi
> +++ b/arch/arm/boot/dts/armada-xp-98dx3236.dtsi
> @@ -243,7 +243,7 @@
> ranges = <0 MBUS_ID(0x03, 0x00) 0 0x100000>;
>
> pp0: packet-processor@0 {
> - compatible = "marvell,prestera-98dx3236";
> + compatible = "marvell,prestera-98dx3236", "marvell,prestera";
> reg = <0 0x4000000>;
> interrupts = <33>, <34>, <35>;
> dfx = <&dfx>;
> diff --git a/arch/arm/boot/dts/armada-xp-98dx3336.dtsi b/arch/arm/boot/dts/armada-xp-98dx3336.dtsi
> index 2f5fc67dd6dc..1d9d8a8ea60c 100644
> --- a/arch/arm/boot/dts/armada-xp-98dx3336.dtsi
> +++ b/arch/arm/boot/dts/armada-xp-98dx3336.dtsi
> @@ -35,5 +35,5 @@
> };
>
> &pp0 {
> - compatible = "marvell,prestera-98dx3336";
> + compatible = "marvell,prestera-98dx3336", "marvell,prestera";
> };
> diff --git a/arch/arm/boot/dts/armada-xp-98dx4251.dtsi b/arch/arm/boot/dts/armada-xp-98dx4251.dtsi
> index 7a9e8839880b..48ffdc72bfc7 100644
> --- a/arch/arm/boot/dts/armada-xp-98dx4251.dtsi
> +++ b/arch/arm/boot/dts/armada-xp-98dx4251.dtsi
> @@ -49,6 +49,6 @@
> };
>
> &pp0 {
> - compatible = "marvell,prestera-98dx4251";
> + compatible = "marvell,prestera-98dx4251", "marvell,prestera";
> interrupts = <33>, <34>, <35>, <36>;
> };
> --
> 2.18.0
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-09-21 11:08 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-07 0:59 [PATCH 0/2] ARM: dts: mvebu: marvell,prestera Chris Packham
2018-09-07 0:59 ` [PATCH 1/2] dt-bindings: marvell,prestera: Add common compatible string Chris Packham
2018-09-17 5:45 ` [PATCH 1/2] dt-bindings: marvell, prestera: " Rob Herring
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-21 11:08 ` Gregory CLEMENT
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).