2024-05-28 10:39:08

by FLAVIO SULIGOI

[permalink] [raw]
Subject: [PATCH v4 0/5] arm64: dts: remove tx-sched-sp property in snps,dwmac

In the ethernet stmmac device driver:

- drivers/net/ethernet/stmicro/stmmac/

The "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 patch series remove this property from the following device-tree
files:

- 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 these DTS,
since, as seen above, it is a default property of the driver.

The property is also removed, in a separate patch, from the corresponding
dt_bindings file:
- Documentation/devicetree/bindings/net/snps,dwmac.yaml

**************************************************************************
NOTE about this v4 patch series: resending this v4 version of the patches,
I omitted the word "RESEND" in the subject line, since I added a new
tag in the patch num. 5/5.
**************************************************************************

v4 - Resend after some weeks and added the tag "Reviewed-by: Krzysztof
Kozlowski <[email protected]>" in patch num. 5/5.
v3 - Removed the tag "Reviewed-by: Krzysztof Kozlowski <[email protected]>"
in patches num. 1/5, 2/5, 3/5 and 5/5 (it was added by mistake).
Added history in each of the patches, as well as in the cover-letter.
v2 - This patch is the 2nd version of a previous patch, where both the DTS
and the yaml files were included toghether. Then I split this 1st
patch series in two, as suggested by Krzysztof.
v1 - Original version of the patch where, in addition to these DTS patches,
there was also the one related to the correspondent snps,dwmac.yaml
dt_binding file.

Flavio Suligoi (5):
arm64: dts: freescale: imx8mp-beacon: remove tx-sched-sp property
arm64: dts: freescale: imx8mp-evk: remove tx-sched-sp property
arm64: dts: freescale: imx8mp-verdin: remove tx-sched-sp property
arm64: dts: qcom: sa8540p-ride: remove tx-sched-sp property
arm64: dts: qcom: sa8775p-ride: remove tx-sched-sp property

arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi | 1 -
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 1 -
arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi | 1 -
arch/arm64/boot/dts/qcom/sa8540p-ride.dts | 2 --
arch/arm64/boot/dts/qcom/sa8775p-ride.dts | 2 --
5 files changed, 7 deletions(-)

--
2.34.1



2024-05-28 10:41:07

by FLAVIO SULIGOI

[permalink] [raw]
Subject: [PATCH v4 5/5] arm64: dts: qcom: sa8775p-ride: 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").

So we can safely remove this property from this device-tree.

Signed-off-by: Flavio Suligoi <[email protected]>
Reviewed-by: Krzysztof Kozlowski <[email protected]>
---

v4 - Resend after some weeks.
Added the tag "Reviewed-by: Krzysztof Kozlowski
<[email protected]>"
v3 - Removed the tag "Reviewed-by: Krzysztof Kozlowski <[email protected]>"
(it was added by mistake).
Added history, as well as in the cover-letter.
v2 - This patch is the 2nd version of a previous patch, where both the DTS
and the yaml files were included toghether. Then I split this 1st patch
series in two, as suggested by Krzysztof.
v1 - Original version of the patch where, in addition to this DTS patch,
there was also the one related to the correspondent snps,dwmac.yaml
dt_binding file.

arch/arm64/boot/dts/qcom/sa8775p-ride.dts | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
index 26ad05bd3b3f..2e1770e07f45 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
+++ b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
@@ -334,7 +334,6 @@ queue3 {

mtl_tx_setup: tx-queues-config {
snps,tx-queues-to-use = <4>;
- snps,tx-sched-sp;

queue0 {
snps,dcb-algorithm;
@@ -404,7 +403,6 @@ queue3 {

mtl_tx_setup1: tx-queues-config {
snps,tx-queues-to-use = <4>;
- snps,tx-sched-sp;

queue0 {
snps,dcb-algorithm;
--
2.34.1