2024-04-29 09:27:50

by FLAVIO SULIGOI

[permalink] [raw]
Subject: [PATCH net-next v3 0/1] dt-bindings: net: snps, dwmac: remove tx-sched-sp property

Strict priority for the tx scheduler is by default in Linux driver, so the
tx-sched-sp property was removed in commit aed6864035b1 ("net: stmmac:
platform: Delete a redundant condition branch").

This property is still in use in the following DT (and it will be removed
in a separate patch series):

- arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi
- arch/arm64/boot/dts/freescale/imx8mp-evk.dts
- arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
- arch/arm64/boot/dts/qcom/sa8540p-ride.dts
- arch/arm64/boot/dts/qcom/sa8775p-ride.dts

There is no problem if that property is still used in the DTs above,
since, as seen above, it is a default property of the driver.

v3 - Added history in the patch, as well as in the cover-letter.
- Add "Acked-by: Adam Ford <[email protected]>".
v2 - This patch is the 2nd version of a previous patch series, where both
the DTS and the yaml files were included toghether. Then I split this
1st patch series in two, as suggested by Krzysztof.
- Add "Acked-by: Krzysztof Kozlowski <[email protected]>".
v1 - Original version of the patch series, including, in addition to this
patch, also other five DTS patches, in which the property
"snps,tx-sched-sp" appeared.

Flavio Suligoi (1):
dt-bindings: net: snps,dwmac: remove tx-sched-sp property

.../devicetree/bindings/net/snps,dwmac.yaml | 14 --------------
1 file changed, 14 deletions(-)

--
2.34.1



2024-04-29 09:27:58

by FLAVIO SULIGOI

[permalink] [raw]
Subject: [PATCH net-next v3 1/1] dt-bindings: net: snps, dwmac: remove tx-sched-sp property

Strict priority for the tx scheduler is by default in Linux driver, so the
tx-sched-sp property was removed in commit aed6864035b1 ("net: stmmac:
platform: Delete a redundant condition branch").

This property is still in use in the following DT (and it will be removed
in a separate patch series):

- arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi
- arch/arm64/boot/dts/freescale/imx8mp-evk.dts
- arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
- arch/arm64/boot/dts/qcom/sa8540p-ride.dts
- arch/arm64/boot/dts/qcom/sa8775p-ride.dts

There is no problem if that property is still used in the DTs above,
since, as seen above, it is a default property of the driver.

Signed-off-by: Flavio Suligoi <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
Acked-by: Adam Ford <[email protected]>
---

v3 - Added history in the patch, as well as in the cover-letter.
- Add "Acked-by: Adam Ford <[email protected]>".
v2 - This patch is the 2nd version of a previous patch series, where both
the DTS and the yaml files were included toghether. Then I split this
1st patch series in two, as suggested by Krzysztof.
- Add "Acked-by: Krzysztof Kozlowski <[email protected]>"
v1 - Original version of the patch series, including, in addition to this
patch, also other five DTS patches, in which the property
"snps,tx-sched-sp" appeared.

.../devicetree/bindings/net/snps,dwmac.yaml | 14 --------------
1 file changed, 14 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
index 15073627c53a..21cc27e75f50 100644
--- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml
+++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
@@ -328,9 +328,6 @@ properties:
snps,tx-sched-dwrr:
type: boolean
description: Deficit Weighted Round Robin
- snps,tx-sched-sp:
- type: boolean
- description: Strict priority
allOf:
- if:
required:
@@ -339,7 +336,6 @@ properties:
properties:
snps,tx-sched-wfq: false
snps,tx-sched-dwrr: false
- snps,tx-sched-sp: false
- if:
required:
- snps,tx-sched-wfq
@@ -347,7 +343,6 @@ properties:
properties:
snps,tx-sched-wrr: false
snps,tx-sched-dwrr: false
- snps,tx-sched-sp: false
- if:
required:
- snps,tx-sched-dwrr
@@ -355,15 +350,6 @@ properties:
properties:
snps,tx-sched-wrr: false
snps,tx-sched-wfq: false
- snps,tx-sched-sp: false
- - if:
- required:
- - snps,tx-sched-sp
- then:
- properties:
- snps,tx-sched-wrr: false
- snps,tx-sched-wfq: false
- snps,tx-sched-dwrr: false
patternProperties:
"^queue[0-9]$":
description: Each subnode represents a queue.
--
2.34.1


2024-05-01 02:11:06

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH net-next v3 0/1] dt-bindings: net: snps, dwmac: remove tx-sched-sp property

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <[email protected]>:

On Mon, 29 Apr 2024 11:26:53 +0200 you wrote:
> Strict priority for the tx scheduler is by default in Linux driver, so the
> tx-sched-sp property was removed in commit aed6864035b1 ("net: stmmac:
> platform: Delete a redundant condition branch").
>
> This property is still in use in the following DT (and it will be removed
> in a separate patch series):
>
> [...]

Here is the summary with links:
- [net-next,v3,1/1] dt-bindings: net: snps, dwmac: remove tx-sched-sp property
https://git.kernel.org/netdev/net-next/c/a2af49293db6

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html