The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
This patch serie remove this property both from the "snps,dwmac.yaml"
documentation and from some device-tree boards files where was still used.
Flavio Suligoi (6):
dt-bindings: net: snps,dwmac: remove tx-sched-sp property
arm64: dts: imx8mp-beacon: remove tx-sched-sp property
arm64: dts: imx8mp-evk: remove tx-sched-sp property
arm64: dts: imx8mp-verdin: remove tx-sched-sp property
arm64: dts: qcom: sa8540p-ride: remove tx-sched-sp property
dts: qcom: sa8775p-ride: remove tx-sched-sp property
.../devicetree/bindings/net/snps,dwmac.yaml | 14 --------------
.../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 --
6 files changed, 21 deletions(-)
--
2.34.1
The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
Signed-off-by: Flavio Suligoi <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi
index 8be251b69378..34339dc4a635 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-beacon-som.dtsi
@@ -106,7 +106,6 @@ queue4 {
mtl_tx_setup: tx-queues-config {
snps,tx-queues-to-use = <5>;
- snps,tx-sched-sp;
queue0 {
snps,dcb-algorithm;
--
2.34.1
The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
Signed-off-by: Flavio Suligoi <[email protected]>
---
.../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
The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
Signed-off-by: Flavio Suligoi <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
index 8be5b2a57f27..bb1003363e3e 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
@@ -251,7 +251,6 @@ ethphy0: ethernet-phy@1 {
mtl_tx_setup: tx-queues-config {
snps,tx-queues-to-use = <5>;
- snps,tx-sched-sp;
queue0 {
snps,dcb-algorithm;
--
2.34.1
The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
Signed-off-by: Flavio Suligoi <[email protected]>
---
arch/arm64/boot/dts/qcom/sa8540p-ride.dts | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sa8540p-ride.dts b/arch/arm64/boot/dts/qcom/sa8540p-ride.dts
index 177b9dad6ff7..11663cf81e45 100644
--- a/arch/arm64/boot/dts/qcom/sa8540p-ride.dts
+++ b/arch/arm64/boot/dts/qcom/sa8540p-ride.dts
@@ -225,7 +225,6 @@ queue3 {
ethernet0_mtl_tx_setup: tx-queues-config {
snps,tx-queues-to-use = <1>;
- snps,tx-sched-sp;
queue0 {
snps,dcb-algorithm;
@@ -302,7 +301,6 @@ queue3 {
ethernet1_mtl_tx_setup: tx-queues-config {
snps,tx-queues-to-use = <1>;
- snps,tx-sched-sp;
queue0 {
snps,dcb-algorithm;
--
2.34.1
The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
Signed-off-by: Flavio Suligoi <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
index f033d4310305..5b1b36a7ad38 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
@@ -260,7 +260,6 @@ queue4 {
mtl_tx_setup: tx-queues-config {
snps,tx-queues-to-use = <5>;
- snps,tx-sched-sp;
queue0 {
snps,dcb-algorithm;
--
2.34.1
The property "tx-sched-sp" no longer exists, as it was removed from the
file:
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
by the commit:
commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
branch")
Signed-off-by: Flavio Suligoi <[email protected]>
---
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
On 05/04/2024 17:27, Flavio Suligoi wrote:
> The property "tx-sched-sp" no longer exists, as it was removed from the
> file:
>
> drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
>
> by the commit:
Keep syntax as asked by submitting patches, so "by the commit sha ("foo
bar").
>
> commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
> branch")
>
> Signed-off-by: Flavio Suligoi <[email protected]>
> ---
> .../devicetree/bindings/net/snps,dwmac.yaml | 14 --------------
> 1 file changed, 14 deletions(-)
This means by default we have tx-sched-sp... I guess it is fine,
assuming there are no other users (projects) of this binding property.
Acked-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On 05/04/2024 17:27, Flavio Suligoi wrote:
> The property "tx-sched-sp" no longer exists, as it was removed from the
> file:
>
> drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
>
> by the commit:
>
> commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
> branch")
>
> Signed-off-by: Flavio Suligoi <[email protected]>
> ---
> .../devicetree/bindings/net/snps,dwmac.yaml | 14 --------------
> 1 file changed, 14 deletions(-)
One more thought though:
1. Missing net-next patch annotation,
2. Please split DTS from net. DTS goes via separate trees.
Best regards,
Krzysztof
On 05/04/2024 17:27, Flavio Suligoi wrote:
> The property "tx-sched-sp" no longer exists, as it was removed from the
> file:
>
> drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
>
> by the commit:
>
> commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
> branch")
Same problem with commit. BTW, your commit msg does not say what happens
if this property is removed. Instead it could be "Strict priority is by
default in Linux driver and tx-sched-sp was removed in commit sha ("foo
bar")".
With fixed in commit msg.
Reviewed-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On 05/04/2024 17:28, Flavio Suligoi wrote:
> The property "tx-sched-sp" no longer exists, as it was removed from the
> file:
>
> drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
>
> by the commit:
>
> commit aed6864035b1 ("net: stmmac: platform: Delete a redundant condition
> branch")
>
> Signed-off-by: Flavio Suligoi <[email protected]>
Fixes in commit msg needed and patch prefix got mangled: missing arm64.
Best regards,
Krzysztof
Hi Krzysztof,
...
> On 05/04/2024 17:27, Flavio Suligoi wrote:
> > The property "tx-sched-sp" no longer exists, as it was removed from
> the
> > file:
> >
> > drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
> >
> > by the commit:
> >
> > commit aed6864035b1 ("net: stmmac: platform: Delete a redundant
> condition
> > branch")
> >
> > Signed-off-by: Flavio Suligoi <[email protected]>
> > ---
> > .../devicetree/bindings/net/snps,dwmac.yaml | 14 -------------
> -
> > 1 file changed, 14 deletions(-)
>
> One more thought though:
> 1. Missing net-next patch annotation,
> 2. Please split DTS from net. DTS goes via separate trees.
Thanks for all your suggestions.
I'll resend the patches, with your suggested changes and
splitting the DTS-related patches in a separate series.
>
> Best regards,
> Krzysztof
Best regards,
Flavio