2023-05-11 07:15:34

by Somisetty, Pranavi

[permalink] [raw]
Subject: [PATCH net-next v2 1/2] dt-bindings: net: cdns,macb: Add rx-watermark property

watermark value is the minimum amount of packet data
required to activate the forwarding process. The watermark
implementation and maximum size is dependent on the device
where Cadence MACB/GEM is used.

Signed-off-by: Pranavi Somisetty <[email protected]>
---
Documentation/devicetree/bindings/net/cdns,macb.yaml | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/cdns,macb.yaml b/Documentation/devicetree/bindings/net/cdns,macb.yaml
index bef5e0f895be..779bc25cf005 100644
--- a/Documentation/devicetree/bindings/net/cdns,macb.yaml
+++ b/Documentation/devicetree/bindings/net/cdns,macb.yaml
@@ -109,6 +109,13 @@ properties:
power-domains:
maxItems: 1

+ rx-watermark:
+ maxItems: 1
+ $ref: /schemas/types.yaml#/definitions/uint16
+ description:
+ Set watermark value for pbuf_rxcutthru reg and enable
+ rx partial store and forward.
+
'#address-cells':
const: 1

@@ -166,6 +173,7 @@ examples:
compatible = "cdns,macb";
reg = <0xfffc4000 0x4000>;
interrupts = <21>;
+ rx-watermark = /bits/ 16 <0x44>;
phy-mode = "rmii";
local-mac-address = [3a 0e 03 04 05 06];
clock-names = "pclk", "hclk", "tx_clk";
--
2.36.1



2023-05-11 07:54:43

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH net-next v2 1/2] dt-bindings: net: cdns,macb: Add rx-watermark property

On Thu, May 11, 2023 at 01:12:13AM -0600, Pranavi Somisetty wrote:
> watermark value is the minimum amount of packet data
> required to activate the forwarding process. The watermark
> implementation and maximum size is dependent on the device
> where Cadence MACB/GEM is used.
>
> Signed-off-by: Pranavi Somisetty <[email protected]>

Please send dt-binding patches to the dt-binding maintainers and list.
get_maintainer.pl should have told you to do so & without having done
so, the bindings will not get tested :/

Thanks,
Conor.

> ---
> Documentation/devicetree/bindings/net/cdns,macb.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/cdns,macb.yaml b/Documentation/devicetree/bindings/net/cdns,macb.yaml
> index bef5e0f895be..779bc25cf005 100644
> --- a/Documentation/devicetree/bindings/net/cdns,macb.yaml
> +++ b/Documentation/devicetree/bindings/net/cdns,macb.yaml
> @@ -109,6 +109,13 @@ properties:
> power-domains:
> maxItems: 1
>
> + rx-watermark:
> + maxItems: 1
> + $ref: /schemas/types.yaml#/definitions/uint16
> + description:
> + Set watermark value for pbuf_rxcutthru reg and enable
> + rx partial store and forward.
> +
> '#address-cells':
> const: 1
>
> @@ -166,6 +173,7 @@ examples:
> compatible = "cdns,macb";
> reg = <0xfffc4000 0x4000>;
> interrupts = <21>;
> + rx-watermark = /bits/ 16 <0x44>;
> phy-mode = "rmii";
> local-mac-address = [3a 0e 03 04 05 06];
> clock-names = "pclk", "hclk", "tx_clk";
> --
> 2.36.1
>


Attachments:
(No filename) (1.65 kB)
signature.asc (235.00 B)
Download all attachments

2023-05-11 14:15:46

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH net-next v2 1/2] dt-bindings: net: cdns,macb: Add rx-watermark property

On 11/05/2023 09:12, Pranavi Somisetty wrote:
> watermark value is the minimum amount of packet data
> required to activate the forwarding process. The watermark
> implementation and maximum size is dependent on the device
> where Cadence MACB/GEM is used.
>
> Signed-off-by: Pranavi Somisetty <[email protected]>

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC. It might happen, that command when run on an older
kernel, gives you outdated entries. Therefore please be sure you base
your patches on recent Linux kernel.

You missed at least DT list (maybe more), so this won't be tested.
Please resend and include all necessary entries.

Best regards,
Krzysztof


2023-05-11 15:46:34

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH net-next v2 1/2] dt-bindings: net: cdns,macb: Add rx-watermark property

On Thu, May 11, 2023 at 01:12:13AM -0600, Pranavi Somisetty wrote:
> watermark value is the minimum amount of packet data
> required to activate the forwarding process. The watermark
> implementation and maximum size is dependent on the device
> where Cadence MACB/GEM is used.
>
> Signed-off-by: Pranavi Somisetty <[email protected]>
> ---
> Documentation/devicetree/bindings/net/cdns,macb.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/cdns,macb.yaml b/Documentation/devicetree/bindings/net/cdns,macb.yaml
> index bef5e0f895be..779bc25cf005 100644
> --- a/Documentation/devicetree/bindings/net/cdns,macb.yaml
> +++ b/Documentation/devicetree/bindings/net/cdns,macb.yaml
> @@ -109,6 +109,13 @@ properties:
> power-domains:
> maxItems: 1
>
> + rx-watermark:
> + maxItems: 1
> + $ref: /schemas/types.yaml#/definitions/uint16
> + description:
> + Set watermark value for pbuf_rxcutthru reg and enable
> + rx partial store and forward.

What are the units? Frames, octets, DMA scatter-gather segments, % of
maximum?

Andrew

---
pw-bot: cr


2023-05-11 22:37:48

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH net-next v2 1/2] dt-bindings: net: cdns,macb: Add rx-watermark property

On Thu, May 11, 2023 at 08:25:15AM +0100, Conor Dooley wrote:
> On Thu, May 11, 2023 at 01:12:13AM -0600, Pranavi Somisetty wrote:
> > watermark value is the minimum amount of packet data
> > required to activate the forwarding process. The watermark
> > implementation and maximum size is dependent on the device
> > where Cadence MACB/GEM is used.
> >
> > Signed-off-by: Pranavi Somisetty <[email protected]>
>
> Please send dt-binding patches to the dt-binding maintainers and list.
> get_maintainer.pl should have told you to do so & without having done
> so, the bindings will not get tested :/

The automation we run for the linux-riscv did actually run against
this though, and even dtbs_check complains about this binding:

Documentation/devicetree/bindings/net/cdns,macb.yaml: properties:rx-watermark:maxItems: False schema does not allow 1
hint: Scalar properties should not have array keywords
from schema $id: http://devicetree.org/meta-schemas/keywords.yaml#

Please test your bindings :/

Cheers,
Conor.


Attachments:
(No filename) (1.03 kB)
signature.asc (235.00 B)
Download all attachments