Re: [v5,net-next,1/9] net: stmmac: multiple queues dt configuration

2017-03-21 Thread Thierry Reding
On Fri, Mar 10, 2017 at 06:24:51PM +, Joao Pinto wrote:
> This patch adds the multiple queues configuration in the Device Tree.
> It was also created a set of structures to keep the RX and TX queues
> configurations to be used in the driver.
> 
> Signed-off-by: Joao Pinto 
> ---
> Changes v4->v5:
> - patch title update (stmicro replaced by stmmac)
> Changes v2->v4:
> - Just to keep up with patch-set version
> Changes v1->v2:
> - RX and TX queues child nodes had bad handle
> 
>  Documentation/devicetree/bindings/net/stmmac.txt   | 40 ++
>  .../net/ethernet/stmicro/stmmac/stmmac_platform.c  | 91 
> ++
>  include/linux/stmmac.h | 30 +++
>  3 files changed, 161 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/stmmac.txt 
> b/Documentation/devicetree/bindings/net/stmmac.txt
> index d3bfc2b..4107e67 100644
> --- a/Documentation/devicetree/bindings/net/stmmac.txt
> +++ b/Documentation/devicetree/bindings/net/stmmac.txt
> @@ -72,6 +72,27 @@ Optional properties:
>   - snps,mb: mixed-burst
>   - snps,rb: rebuild INCRx Burst
>  - mdio: with compatible = "snps,dwmac-mdio", create and register mdio bus.
> +- Multiple RX Queues parameters: below the list of all the parameters to
> +  configure the multiple RX queues:
> + - snps,rx-queues-to-use: number of RX queues to be used in the driver
> + - Choose one of these RX scheduling algorithms:
> + - snps,rx-sched-sp: Strict priority
> + - snps,rx-sched-wsp: Weighted Strict priority
> + - For each RX queue
> + - Choose one of these modes:
> + - snps,dcb-algorithm: Queue to be enabled as DCB
> + - snps,avb-algorithm: Queue to be enabled as AVB
> + - snps,map-to-dma-channel: Channel to map
> +- Multiple TX Queues parameters: below the list of all the parameters to
> +  configure the multiple TX queues:
> + - snps,tx-queues-to-use: number of TX queues to be used in the driver
> + - Choose one of these TX scheduling algorithms:
> + - snps,tx-sched-wrr: Weighted Round Robin
> + - snps,tx-sched-wfq: Weighted Fair Queuing
> + - snps,tx-sched-dwrr: Deficit Weighted Round Robin
> + - snps,tx-sched-sp: Strict priority
> + - For each TX queue
> + - snps,weight: TX queue weight (if using a weighted algorithm)
>  
>  Examples:
>  
> @@ -81,6 +102,23 @@ Examples:
>   snps,blen = <256 128 64 32 0 0 0>;
>   };
>  
> + mtl_rx_setup: rx-queues-config {
> + snps,rx-queues-to-use = <1>;
> + snps,rx-sched-sp;
> + queue0 {
> + snps,dcb-algorithm;
> + snps,map-to-dma-channel = <0x0>;
> + };
> + };
> +
> + mtl_tx_setup: tx-queues-config {
> + snps,tx-queues-to-use = <1>;
> + snps,tx-sched-wrr;
> + queue0 {
> + snps,weight = <0x10>;
> + };
> + };
> +
>   gmac0: ethernet@e080 {
>   compatible = "st,spear600-gmac";
>   reg = <0xe080 0x8000>;
> @@ -104,4 +142,6 @@ Examples:
>   phy1: ethernet-phy@0 {
>   };
>   };
> + snps,mtl-rx-config = <&mtl_rx_setup>;
> + snps,mtl-tx-config = <&mtl_tx_setup>;
>   };

That's kind of an odd placement for these nodes. Wouldn't they be better
located within the controller's device tree node, rather than referenced
by phandle? Making them children of the root node leaves them completely
without context.

Thierry


signature.asc
Description: PGP signature


Re: [v5,net-next,1/9] net: stmmac: multiple queues dt configuration

2017-03-21 Thread Joao Pinto

Hi Thierry,

Às 11:32 AM de 3/21/2017, Thierry Reding escreveu:
> On Fri, Mar 10, 2017 at 06:24:51PM +, Joao Pinto wrote:
>> This patch adds the multiple queues configuration in the Device Tree.
>> It was also created a set of structures to keep the RX and TX queues
>> configurations to be used in the driver.
>>
>> Signed-off-by: Joao Pinto 
>> ---
>> Changes v4->v5:
>> - patch title update (stmicro replaced by stmmac)
>> Changes v2->v4:
>> - Just to keep up with patch-set version
>> Changes v1->v2:
>> - RX and TX queues child nodes had bad handle
>>
>>  Documentation/devicetree/bindings/net/stmmac.txt   | 40 ++
>>  .../net/ethernet/stmicro/stmmac/stmmac_platform.c  | 91 
>> ++
>>  include/linux/stmmac.h | 30 +++
>>  3 files changed, 161 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/net/stmmac.txt 
>> b/Documentation/devicetree/bindings/net/stmmac.txt
>> index d3bfc2b..4107e67 100644
>> --- a/Documentation/devicetree/bindings/net/stmmac.txt
>> +++ b/Documentation/devicetree/bindings/net/stmmac.txt
>> @@ -72,6 +72,27 @@ Optional properties:
>>  - snps,mb: mixed-burst
>>  - snps,rb: rebuild INCRx Burst
>>  - mdio: with compatible = "snps,dwmac-mdio", create and register mdio bus.
>> +- Multiple RX Queues parameters: below the list of all the parameters to
>> + configure the multiple RX queues:
>> +- snps,rx-queues-to-use: number of RX queues to be used in the driver
>> +- Choose one of these RX scheduling algorithms:
>> +- snps,rx-sched-sp: Strict priority
>> +- snps,rx-sched-wsp: Weighted Strict priority
>> +- For each RX queue
>> +- Choose one of these modes:
>> +- snps,dcb-algorithm: Queue to be enabled as DCB
>> +- snps,avb-algorithm: Queue to be enabled as AVB
>> +- snps,map-to-dma-channel: Channel to map
>> +- Multiple TX Queues parameters: below the list of all the parameters to
>> + configure the multiple TX queues:
>> +- snps,tx-queues-to-use: number of TX queues to be used in the driver
>> +- Choose one of these TX scheduling algorithms:
>> +- snps,tx-sched-wrr: Weighted Round Robin
>> +- snps,tx-sched-wfq: Weighted Fair Queuing
>> +- snps,tx-sched-dwrr: Deficit Weighted Round Robin
>> +- snps,tx-sched-sp: Strict priority
>> +- For each TX queue
>> +- snps,weight: TX queue weight (if using a weighted algorithm)
>>  
>>  Examples:
>>  
>> @@ -81,6 +102,23 @@ Examples:
>>  snps,blen = <256 128 64 32 0 0 0>;
>>  };
>>  
>> +mtl_rx_setup: rx-queues-config {
>> +snps,rx-queues-to-use = <1>;
>> +snps,rx-sched-sp;
>> +queue0 {
>> +snps,dcb-algorithm;
>> +snps,map-to-dma-channel = <0x0>;
>> +};
>> +};
>> +
>> +mtl_tx_setup: tx-queues-config {
>> +snps,tx-queues-to-use = <1>;
>> +snps,tx-sched-wrr;
>> +queue0 {
>> +snps,weight = <0x10>;
>> +};
>> +};
>> +
>>  gmac0: ethernet@e080 {
>>  compatible = "st,spear600-gmac";
>>  reg = <0xe080 0x8000>;
>> @@ -104,4 +142,6 @@ Examples:
>>  phy1: ethernet-phy@0 {
>>  };
>>  };
>> +snps,mtl-rx-config = <&mtl_rx_setup>;
>> +snps,mtl-tx-config = <&mtl_tx_setup>;
>>  };
> 
> That's kind of an odd placement for these nodes. Wouldn't they be better
> located within the controller's device tree node, rather than referenced
> by phandle? Making them children of the root node leaves them completely
> without context.
> 
> Thierry
> 

I followed the same approach as "stmmac_axi_setup":
https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/tree/Documentation/devicetree/bindings/net/stmmac.txt

I personally find it cleaner and easier to read.

Thanks,
Joao




Re: [v5,net-next,1/9] net: stmmac: multiple queues dt configuration

2017-03-21 Thread Thierry Reding
On Tue, Mar 21, 2017 at 11:39:24AM +, Joao Pinto wrote:
> 
> Hi Thierry,
> 
> Às 11:32 AM de 3/21/2017, Thierry Reding escreveu:
> > On Fri, Mar 10, 2017 at 06:24:51PM +, Joao Pinto wrote:
> >> This patch adds the multiple queues configuration in the Device Tree.
> >> It was also created a set of structures to keep the RX and TX queues
> >> configurations to be used in the driver.
> >>
> >> Signed-off-by: Joao Pinto 
> >> ---
> >> Changes v4->v5:
> >> - patch title update (stmicro replaced by stmmac)
> >> Changes v2->v4:
> >> - Just to keep up with patch-set version
> >> Changes v1->v2:
> >> - RX and TX queues child nodes had bad handle
> >>
> >>  Documentation/devicetree/bindings/net/stmmac.txt   | 40 ++
> >>  .../net/ethernet/stmicro/stmmac/stmmac_platform.c  | 91 
> >> ++
> >>  include/linux/stmmac.h | 30 +++
> >>  3 files changed, 161 insertions(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/net/stmmac.txt 
> >> b/Documentation/devicetree/bindings/net/stmmac.txt
> >> index d3bfc2b..4107e67 100644
> >> --- a/Documentation/devicetree/bindings/net/stmmac.txt
> >> +++ b/Documentation/devicetree/bindings/net/stmmac.txt
> >> @@ -72,6 +72,27 @@ Optional properties:
> >>- snps,mb: mixed-burst
> >>- snps,rb: rebuild INCRx Burst
> >>  - mdio: with compatible = "snps,dwmac-mdio", create and register mdio bus.
> >> +- Multiple RX Queues parameters: below the list of all the parameters to
> >> +   configure the multiple RX queues:
> >> +  - snps,rx-queues-to-use: number of RX queues to be used in the driver
> >> +  - Choose one of these RX scheduling algorithms:
> >> +  - snps,rx-sched-sp: Strict priority
> >> +  - snps,rx-sched-wsp: Weighted Strict priority
> >> +  - For each RX queue
> >> +  - Choose one of these modes:
> >> +  - snps,dcb-algorithm: Queue to be enabled as DCB
> >> +  - snps,avb-algorithm: Queue to be enabled as AVB
> >> +  - snps,map-to-dma-channel: Channel to map
> >> +- Multiple TX Queues parameters: below the list of all the parameters to
> >> +   configure the multiple TX queues:
> >> +  - snps,tx-queues-to-use: number of TX queues to be used in the driver
> >> +  - Choose one of these TX scheduling algorithms:
> >> +  - snps,tx-sched-wrr: Weighted Round Robin
> >> +  - snps,tx-sched-wfq: Weighted Fair Queuing
> >> +  - snps,tx-sched-dwrr: Deficit Weighted Round Robin
> >> +  - snps,tx-sched-sp: Strict priority
> >> +  - For each TX queue
> >> +  - snps,weight: TX queue weight (if using a weighted algorithm)
> >>  
> >>  Examples:
> >>  
> >> @@ -81,6 +102,23 @@ Examples:
> >>snps,blen = <256 128 64 32 0 0 0>;
> >>};
> >>  
> >> +  mtl_rx_setup: rx-queues-config {
> >> +  snps,rx-queues-to-use = <1>;
> >> +  snps,rx-sched-sp;
> >> +  queue0 {
> >> +  snps,dcb-algorithm;
> >> +  snps,map-to-dma-channel = <0x0>;
> >> +  };
> >> +  };
> >> +
> >> +  mtl_tx_setup: tx-queues-config {
> >> +  snps,tx-queues-to-use = <1>;
> >> +  snps,tx-sched-wrr;
> >> +  queue0 {
> >> +  snps,weight = <0x10>;
> >> +  };
> >> +  };
> >> +
> >>gmac0: ethernet@e080 {
> >>compatible = "st,spear600-gmac";
> >>reg = <0xe080 0x8000>;
> >> @@ -104,4 +142,6 @@ Examples:
> >>phy1: ethernet-phy@0 {
> >>};
> >>};
> >> +  snps,mtl-rx-config = <&mtl_rx_setup>;
> >> +  snps,mtl-tx-config = <&mtl_tx_setup>;
> >>};
> > 
> > That's kind of an odd placement for these nodes. Wouldn't they be better
> > located within the controller's device tree node, rather than referenced
> > by phandle? Making them children of the root node leaves them completely
> > without context.
> > 
> > Thierry
> > 
> 
> I followed the same approach as "stmmac_axi_setup":
> https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/tree/Documentation/devicetree/bindings/net/stmmac.txt
> 
> I personally find it cleaner and easier to read.

Okay, I see. I've never really seen this done, and I suspect the device
tree maintainers would've objected to the AXI bindings, but looks like
it fell through the cracks. Given that this has been upstream for a very
long time now, it is what it is, and I suppose this is at least
consistent.

Thierry


signature.asc
Description: PGP signature


[PATCH v5 net-next 1/9] net: stmmac: multiple queues dt configuration

2017-03-10 Thread Joao Pinto
This patch adds the multiple queues configuration in the Device Tree.
It was also created a set of structures to keep the RX and TX queues
configurations to be used in the driver.

Signed-off-by: Joao Pinto 
---
Changes v4->v5:
- patch title update (stmicro replaced by stmmac)
Changes v2->v4:
- Just to keep up with patch-set version
Changes v1->v2:
- RX and TX queues child nodes had bad handle

 Documentation/devicetree/bindings/net/stmmac.txt   | 40 ++
 .../net/ethernet/stmicro/stmmac/stmmac_platform.c  | 91 ++
 include/linux/stmmac.h | 30 +++
 3 files changed, 161 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/stmmac.txt 
b/Documentation/devicetree/bindings/net/stmmac.txt
index d3bfc2b..4107e67 100644
--- a/Documentation/devicetree/bindings/net/stmmac.txt
+++ b/Documentation/devicetree/bindings/net/stmmac.txt
@@ -72,6 +72,27 @@ Optional properties:
- snps,mb: mixed-burst
- snps,rb: rebuild INCRx Burst
 - mdio: with compatible = "snps,dwmac-mdio", create and register mdio bus.
+- Multiple RX Queues parameters: below the list of all the parameters to
+configure the multiple RX queues:
+   - snps,rx-queues-to-use: number of RX queues to be used in the driver
+   - Choose one of these RX scheduling algorithms:
+   - snps,rx-sched-sp: Strict priority
+   - snps,rx-sched-wsp: Weighted Strict priority
+   - For each RX queue
+   - Choose one of these modes:
+   - snps,dcb-algorithm: Queue to be enabled as DCB
+   - snps,avb-algorithm: Queue to be enabled as AVB
+   - snps,map-to-dma-channel: Channel to map
+- Multiple TX Queues parameters: below the list of all the parameters to
+configure the multiple TX queues:
+   - snps,tx-queues-to-use: number of TX queues to be used in the driver
+   - Choose one of these TX scheduling algorithms:
+   - snps,tx-sched-wrr: Weighted Round Robin
+   - snps,tx-sched-wfq: Weighted Fair Queuing
+   - snps,tx-sched-dwrr: Deficit Weighted Round Robin
+   - snps,tx-sched-sp: Strict priority
+   - For each TX queue
+   - snps,weight: TX queue weight (if using a weighted algorithm)
 
 Examples:
 
@@ -81,6 +102,23 @@ Examples:
snps,blen = <256 128 64 32 0 0 0>;
};
 
+   mtl_rx_setup: rx-queues-config {
+   snps,rx-queues-to-use = <1>;
+   snps,rx-sched-sp;
+   queue0 {
+   snps,dcb-algorithm;
+   snps,map-to-dma-channel = <0x0>;
+   };
+   };
+
+   mtl_tx_setup: tx-queues-config {
+   snps,tx-queues-to-use = <1>;
+   snps,tx-sched-wrr;
+   queue0 {
+   snps,weight = <0x10>;
+   };
+   };
+
gmac0: ethernet@e080 {
compatible = "st,spear600-gmac";
reg = <0xe080 0x8000>;
@@ -104,4 +142,6 @@ Examples:
phy1: ethernet-phy@0 {
};
};
+   snps,mtl-rx-config = <&mtl_rx_setup>;
+   snps,mtl-tx-config = <&mtl_tx_setup>;
};
diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c 
b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
index 433a842..ff6af8d 100644
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
@@ -132,6 +132,95 @@ static struct stmmac_axi *stmmac_axi_setup(struct 
platform_device *pdev)
 }
 
 /**
+ * stmmac_mtl_setup - parse DT parameters for multiple queues configuration
+ * @pdev: platform device
+ */
+static void stmmac_mtl_setup(struct platform_device *pdev,
+struct plat_stmmacenet_data *plat)
+{
+   struct device_node *q_node;
+   struct device_node *rx_node;
+   struct device_node *tx_node;
+   u8 queue = 0;
+
+   rx_node = of_parse_phandle(pdev->dev.of_node, "snps,mtl-rx-config", 0);
+   if (!rx_node)
+   return;
+
+   tx_node = of_parse_phandle(pdev->dev.of_node, "snps,mtl-tx-config", 0);
+   if (!tx_node) {
+   of_node_put(rx_node);
+   return;
+   }
+
+   /* Processing RX queues common config */
+   if (of_property_read_u8(rx_node, "snps,rx-queues-to-use",
+   &plat->rx_queues_to_use))
+   plat->rx_queues_to_use = 1;
+
+   if (of_property_read_bool(rx_node, "snps,rx-sched-sp"))
+   plat->rx_sched_algorithm = MTL_RX_ALGORITHM_SP;
+   else if (of_property_read_bool(rx_node, "snps,rx-sched-wsp"))
+   plat->rx_sched_algorithm = MTL_RX_ALGORITHM_WSP;
+   else
+   plat->rx_sched_algorithm = MTL_RX_ALGORITHM_SP;
+
+   /* Processing individua