| Message ID | 20240223195526.1161232-1-niklas.soderlund+renesas@ragnatech.se |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | dt-bindings: net: renesas,ethertsn: Document default for delays | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Fri, Feb 23, 2024 at 8:56 PM Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> wrote: > The internal delay properties are not mandatory and should have a > documented default value. The device only supports either no delay or a > fixed delay and the device reset default is no delay, document the > default as no delay. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert
Hello: This patch was applied to netdev/net.git (main) by David S. Miller <davem@davemloft.net>: On Fri, 23 Feb 2024 20:55:26 +0100 you wrote: > The internal delay properties are not mandatory and should have a > documented default value. The device only supports either no delay or a > fixed delay and the device reset default is no delay, document the > default as no delay. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > > [...] Here is the summary with links: - dt-bindings: net: renesas,ethertsn: Document default for delays https://git.kernel.org/netdev/net/c/7be40883b1cb You are awesome, thank you!
diff --git a/Documentation/devicetree/bindings/net/renesas,ethertsn.yaml b/Documentation/devicetree/bindings/net/renesas,ethertsn.yaml index 475aff7714d6..ea35d19be829 100644 --- a/Documentation/devicetree/bindings/net/renesas,ethertsn.yaml +++ b/Documentation/devicetree/bindings/net/renesas,ethertsn.yaml @@ -65,9 +65,11 @@ properties: rx-internal-delay-ps: enum: [0, 1800] + default: 0 tx-internal-delay-ps: enum: [0, 2000] + default: 0 '#address-cells': const: 1
The internal delay properties are not mandatory and should have a documented default value. The device only supports either no delay or a fixed delay and the device reset default is no delay, document the default as no delay. Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> --- Documentation/devicetree/bindings/net/renesas,ethertsn.yaml | 2 ++ 1 file changed, 2 insertions(+)