linux-kernel - Re: [PATCH net-next 2/2] bindings: net: stmmac: Add documentation for TSN parameters

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 25 Oct 2017 12:34:43 -0700
From:   Florian Fainelli <f.fainelli@...il.com>
To:     Jose Abreu <Jose.Abreu@...opsys.com>, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Cc:     "David S. Miller" <davem@...emloft.net>,
        Joao Pinto <Joao.Pinto@...opsys.com>,
        Giuseppe Cavallaro <peppe.cavallaro@...com>,
        Alexandre Torgue <alexandre.torgue@...com>,
        Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH net-next 2/2] bindings: net: stmmac: Add documentation for
 TSN parameters

On 10/25/2017 07:04 AM, Jose Abreu wrote:
> This adds the documentation for TSN feature EST and FP.
> 
> Signed-off-by: Jose Abreu <joabreu@...opsys.com>
> Cc: David S. Miller <davem@...emloft.net>
> Cc: Joao Pinto <jpinto@...opsys.com>
> Cc: Giuseppe Cavallaro <peppe.cavallaro@...com>
> Cc: Alexandre Torgue <alexandre.torgue@...com>
> Cc: Rob Herring <robh+dt@...nel.org>
> ---
>  Documentation/devicetree/bindings/net/stmmac.txt | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/stmmac.txt b/Documentation/devicetree/bindings/net/stmmac.txt
> index c3a7be6..6359df6 100644
> --- a/Documentation/devicetree/bindings/net/stmmac.txt
> +++ b/Documentation/devicetree/bindings/net/stmmac.txt
> @@ -114,6 +114,16 @@ Optional properties:
>  			- snps,high_credit: max write outstanding req. limit
>  			- snps,low_credit: max read outstanding req. limit
>  		- snps,priority: TX queue priority (Range: 0x0 to 0xF)
> +- TSN parameters: below the list of all the parameters to configure TSN
> +		  features (only applicable for IP version >= 5.00)
> +	- snps,est: Enable EST algorithm
> +	- If EST enabled provide these parameters:
> +		- snps,btr: Array size of 2 with BTR value for EST algorithm
> +		- snps,ctr: Array size of 2 with CTR value for EST algorithm
> +		- snps,ter: TER value for EST algorithm
> +		- snps,gcl: Variable size array with GCL table entries.
> +	- snps,fp: Enable FP feature. This needs EST enabled.

You are using Device Tree as a configuration interface, that does not
sound right at all.

> +
>  Examples:
>  
>  	stmmac_axi_setup: stmmac-axi-config {
> @@ -151,6 +161,15 @@ Examples:
>  		};
>  	};
>  
> +	tsn_setup: tsn-config {
> +		snps,est;
> +		snps,btr = <0x00000000 0x00000001>;
> +		snps,ctr = <0x00000000 0x10000000>;
> +		snps,ter = <0x00000000>;
> +		snps,gcl = <0x00000001 0x00000002 0x00000003>;
> +		snps,fp;
> +	};
> +
>  	gmac0: ethernet@...00000 {
>  		compatible = "st,spear600-gmac";
>  		reg = <0xe0800000 0x8000>;
> @@ -176,4 +195,5 @@ Examples:
>  		};
>  		snps,mtl-rx-config = <&mtl_rx_setup>;
>  		snps,mtl-tx-config = <&mtl_tx_setup>;
> +		snps,tsn-config = <&tsn_setup>;

Yes, this is definitively using Device Tree as a configuration
interface, don't do that please.
-- 
Florian