From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] ARM: dts: mvebu: linksys: enable buffer manager support
Date: Thu, 23 Mar 2017 17:29:52 +0100 [thread overview]
Message-ID: <8760j0dkhr.fsf@free-electrons.com> (raw)
In-Reply-To: <20170316055320.7929-1-ralph.sennhauser@gmail.com> (Ralph Sennhauser's message of "Thu, 16 Mar 2017 06:53:19 +0100")
Hi Ralph,
On jeu., mars 16 2017, Ralph Sennhauser <ralph.sennhauser@gmail.com> wrote:
> Add appropriate properties to devices in the Linksys WRT AC Series for the
> mvneta driver to use hardware buffer management.
>
> Also update "soc" ranges property and set the status of bm and bm-bppi
> to "okay" (SRAM).
>
> Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
>
> v1 -> v2:
> Use independent pools for short (suggested by Gregory Clement)
>
>
> arch/arm/boot/dts/armada-385-linksys.dtsi | 17 ++++++++++++++++-
> arch/arm/boot/dts/armada-xp-linksys-mamba.dts | 17 ++++++++++++++++-
> 2 files changed, 32 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-385-linksys.dtsi b/arch/arm/boot/dts/armada-385-linksys.dtsi
> index df47bf1..aa0d225 100644
> --- a/arch/arm/boot/dts/armada-385-linksys.dtsi
> +++ b/arch/arm/boot/dts/armada-385-linksys.dtsi
> @@ -59,7 +59,8 @@
> ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000
> MBUS_ID(0x01, 0x1d) 0 0xfff00000 0x100000
> MBUS_ID(0x09, 0x19) 0 0xf1100000 0x10000
> - MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000>;
> + MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000
> + MBUS_ID(0x0c, 0x04) 0 0xf1200000 0x100000>;
>
> internal-regs {
> i2c at 11000 {
> @@ -88,6 +89,9 @@
> ethernet at 70000 {
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <2>;
> + bm,pool-short = <3>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -97,6 +101,9 @@
> ethernet at 34000 {
> status = "okay";
> phy-mode = "sgmii";
> + buffer-manager = <&bm>;
> + bm,pool-long = <0>;
> + bm,pool-short = <1>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -159,6 +166,10 @@
> status = "okay";
> };
>
> + bm at c8000 {
> + status = "okay";
> + };
> +
> /* USB part of the eSATA/USB 2.0 port */
> usb at 58000 {
> status = "okay";
> @@ -241,6 +252,10 @@
> };
> };
>
> + bm-bppi {
> + status = "okay";
> + };
> +
> pcie-controller {
> status = "okay";
>
> diff --git a/arch/arm/boot/dts/armada-xp-linksys-mamba.dts b/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> index 3744ba3..9efcf59 100644
> --- a/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> +++ b/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> @@ -71,7 +71,8 @@
> ranges = <MBUS_ID(0xf0, 0x01) 0 0 0xf1000000 0x100000
> MBUS_ID(0x01, 0x1d) 0 0 0xfff00000 0x100000
> MBUS_ID(0x09, 0x09) 0 0 0xf1100000 0x10000
> - MBUS_ID(0x09, 0x05) 0 0 0xf1110000 0x10000>;
> + MBUS_ID(0x09, 0x05) 0 0 0xf1110000 0x10000
> + MBUS_ID(0x0c, 0x04) 0 0 0xf1200000 0x100000>;
>
> internal-regs {
>
> @@ -95,6 +96,9 @@
> pinctrl-names = "default";
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <0>;
> + bm,pool-short = <1>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -106,6 +110,9 @@
> pinctrl-names = "default";
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <2>;
> + bm,pool-short = <3>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -186,6 +193,10 @@
> };
> };
>
> + bm at c8000 {
> + status = "okay";
> + };
> +
> nand at d0000 {
> status = "okay";
> num-cs = <1>;
> @@ -259,6 +270,10 @@
> };
> };
> };
> +
> + bm-bppi {
> + status = "okay";
> + };
> };
>
> gpio_keys {
> --
> 2.10.2
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Ralph Sennhauser <ralph.sennhauser@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>, Andrew Lunn <andrew@lunn.ch>,
Jason Cooper <jason@lakedaemon.net>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
Russell King <linux@armlinux.org.uk>,
open list <linux-kernel@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>,
"moderated list:ARM/Marvell Kirkwood and Armada 370, 375, 38x,
..." <linux-arm-kernel@lists.infradead.org>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [PATCH v2] ARM: dts: mvebu: linksys: enable buffer manager support
Date: Thu, 23 Mar 2017 17:29:52 +0100 [thread overview]
Message-ID: <8760j0dkhr.fsf@free-electrons.com> (raw)
In-Reply-To: <20170316055320.7929-1-ralph.sennhauser@gmail.com> (Ralph Sennhauser's message of "Thu, 16 Mar 2017 06:53:19 +0100")
Hi Ralph,
On jeu., mars 16 2017, Ralph Sennhauser <ralph.sennhauser@gmail.com> wrote:
> Add appropriate properties to devices in the Linksys WRT AC Series for the
> mvneta driver to use hardware buffer management.
>
> Also update "soc" ranges property and set the status of bm and bm-bppi
> to "okay" (SRAM).
>
> Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
>
> v1 -> v2:
> Use independent pools for short (suggested by Gregory Clement)
>
>
> arch/arm/boot/dts/armada-385-linksys.dtsi | 17 ++++++++++++++++-
> arch/arm/boot/dts/armada-xp-linksys-mamba.dts | 17 ++++++++++++++++-
> 2 files changed, 32 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-385-linksys.dtsi b/arch/arm/boot/dts/armada-385-linksys.dtsi
> index df47bf1..aa0d225 100644
> --- a/arch/arm/boot/dts/armada-385-linksys.dtsi
> +++ b/arch/arm/boot/dts/armada-385-linksys.dtsi
> @@ -59,7 +59,8 @@
> ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000
> MBUS_ID(0x01, 0x1d) 0 0xfff00000 0x100000
> MBUS_ID(0x09, 0x19) 0 0xf1100000 0x10000
> - MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000>;
> + MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000
> + MBUS_ID(0x0c, 0x04) 0 0xf1200000 0x100000>;
>
> internal-regs {
> i2c@11000 {
> @@ -88,6 +89,9 @@
> ethernet@70000 {
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <2>;
> + bm,pool-short = <3>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -97,6 +101,9 @@
> ethernet@34000 {
> status = "okay";
> phy-mode = "sgmii";
> + buffer-manager = <&bm>;
> + bm,pool-long = <0>;
> + bm,pool-short = <1>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -159,6 +166,10 @@
> status = "okay";
> };
>
> + bm@c8000 {
> + status = "okay";
> + };
> +
> /* USB part of the eSATA/USB 2.0 port */
> usb@58000 {
> status = "okay";
> @@ -241,6 +252,10 @@
> };
> };
>
> + bm-bppi {
> + status = "okay";
> + };
> +
> pcie-controller {
> status = "okay";
>
> diff --git a/arch/arm/boot/dts/armada-xp-linksys-mamba.dts b/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> index 3744ba3..9efcf59 100644
> --- a/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> +++ b/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> @@ -71,7 +71,8 @@
> ranges = <MBUS_ID(0xf0, 0x01) 0 0 0xf1000000 0x100000
> MBUS_ID(0x01, 0x1d) 0 0 0xfff00000 0x100000
> MBUS_ID(0x09, 0x09) 0 0 0xf1100000 0x10000
> - MBUS_ID(0x09, 0x05) 0 0 0xf1110000 0x10000>;
> + MBUS_ID(0x09, 0x05) 0 0 0xf1110000 0x10000
> + MBUS_ID(0x0c, 0x04) 0 0 0xf1200000 0x100000>;
>
> internal-regs {
>
> @@ -95,6 +96,9 @@
> pinctrl-names = "default";
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <0>;
> + bm,pool-short = <1>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -106,6 +110,9 @@
> pinctrl-names = "default";
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <2>;
> + bm,pool-short = <3>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -186,6 +193,10 @@
> };
> };
>
> + bm@c8000 {
> + status = "okay";
> + };
> +
> nand@d0000 {
> status = "okay";
> num-cs = <1>;
> @@ -259,6 +270,10 @@
> };
> };
> };
> +
> + bm-bppi {
> + status = "okay";
> + };
> };
>
> gpio_keys {
> --
> 2.10.2
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Ralph Sennhauser <ralph.sennhauser@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>, Andrew Lunn <andrew@lunn.ch>,
Jason Cooper <jason@lakedaemon.net>,
"open list\:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
Russell King <linux@armlinux.org.uk>,
open list <linux-kernel@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>,
"moderated list\:ARM\/Marvell Kirkwood and Armada 370\, 375\,
38x\, ..." <linux-arm-kernel@lists.infradead.org>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [PATCH v2] ARM: dts: mvebu: linksys: enable buffer manager support
Date: Thu, 23 Mar 2017 17:29:52 +0100 [thread overview]
Message-ID: <8760j0dkhr.fsf@free-electrons.com> (raw)
In-Reply-To: <20170316055320.7929-1-ralph.sennhauser@gmail.com> (Ralph Sennhauser's message of "Thu, 16 Mar 2017 06:53:19 +0100")
Hi Ralph,
On jeu., mars 16 2017, Ralph Sennhauser <ralph.sennhauser@gmail.com> wrote:
> Add appropriate properties to devices in the Linksys WRT AC Series for the
> mvneta driver to use hardware buffer management.
>
> Also update "soc" ranges property and set the status of bm and bm-bppi
> to "okay" (SRAM).
>
> Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
>
> v1 -> v2:
> Use independent pools for short (suggested by Gregory Clement)
>
>
> arch/arm/boot/dts/armada-385-linksys.dtsi | 17 ++++++++++++++++-
> arch/arm/boot/dts/armada-xp-linksys-mamba.dts | 17 ++++++++++++++++-
> 2 files changed, 32 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-385-linksys.dtsi b/arch/arm/boot/dts/armada-385-linksys.dtsi
> index df47bf1..aa0d225 100644
> --- a/arch/arm/boot/dts/armada-385-linksys.dtsi
> +++ b/arch/arm/boot/dts/armada-385-linksys.dtsi
> @@ -59,7 +59,8 @@
> ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000
> MBUS_ID(0x01, 0x1d) 0 0xfff00000 0x100000
> MBUS_ID(0x09, 0x19) 0 0xf1100000 0x10000
> - MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000>;
> + MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000
> + MBUS_ID(0x0c, 0x04) 0 0xf1200000 0x100000>;
>
> internal-regs {
> i2c@11000 {
> @@ -88,6 +89,9 @@
> ethernet@70000 {
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <2>;
> + bm,pool-short = <3>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -97,6 +101,9 @@
> ethernet@34000 {
> status = "okay";
> phy-mode = "sgmii";
> + buffer-manager = <&bm>;
> + bm,pool-long = <0>;
> + bm,pool-short = <1>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -159,6 +166,10 @@
> status = "okay";
> };
>
> + bm@c8000 {
> + status = "okay";
> + };
> +
> /* USB part of the eSATA/USB 2.0 port */
> usb@58000 {
> status = "okay";
> @@ -241,6 +252,10 @@
> };
> };
>
> + bm-bppi {
> + status = "okay";
> + };
> +
> pcie-controller {
> status = "okay";
>
> diff --git a/arch/arm/boot/dts/armada-xp-linksys-mamba.dts b/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> index 3744ba3..9efcf59 100644
> --- a/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> +++ b/arch/arm/boot/dts/armada-xp-linksys-mamba.dts
> @@ -71,7 +71,8 @@
> ranges = <MBUS_ID(0xf0, 0x01) 0 0 0xf1000000 0x100000
> MBUS_ID(0x01, 0x1d) 0 0 0xfff00000 0x100000
> MBUS_ID(0x09, 0x09) 0 0 0xf1100000 0x10000
> - MBUS_ID(0x09, 0x05) 0 0 0xf1110000 0x10000>;
> + MBUS_ID(0x09, 0x05) 0 0 0xf1110000 0x10000
> + MBUS_ID(0x0c, 0x04) 0 0 0xf1200000 0x100000>;
>
> internal-regs {
>
> @@ -95,6 +96,9 @@
> pinctrl-names = "default";
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <0>;
> + bm,pool-short = <1>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -106,6 +110,9 @@
> pinctrl-names = "default";
> status = "okay";
> phy-mode = "rgmii-id";
> + buffer-manager = <&bm>;
> + bm,pool-long = <2>;
> + bm,pool-short = <3>;
> fixed-link {
> speed = <1000>;
> full-duplex;
> @@ -186,6 +193,10 @@
> };
> };
>
> + bm@c8000 {
> + status = "okay";
> + };
> +
> nand@d0000 {
> status = "okay";
> num-cs = <1>;
> @@ -259,6 +270,10 @@
> };
> };
> };
> +
> + bm-bppi {
> + status = "okay";
> + };
> };
>
> gpio_keys {
> --
> 2.10.2
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2017-03-23 16:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-16 5:53 [PATCH v2] ARM: dts: mvebu: linksys: enable buffer manager support Ralph Sennhauser
2017-03-16 5:53 ` Ralph Sennhauser
2017-03-16 5:53 ` Ralph Sennhauser
2017-03-23 16:29 ` Gregory CLEMENT [this message]
2017-03-23 16:29 ` Gregory CLEMENT
2017-03-23 16:29 ` 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=8760j0dkhr.fsf@free-electrons.com \
--to=gregory.clement@free-electrons.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.