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
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 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
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]>
---
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
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]>
---
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 faa17cbbe2fd..21d4b6a9a1af 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
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]>
---
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 9beba8d6a0df..8747c86689cc 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
@@ -226,7 +226,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
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]>
---
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
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]>
---
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 18/04/2024 14:28, Flavio Suligoi 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").
>
> So we can safely remove this property from this device-tree.
>
> Signed-off-by: Flavio Suligoi <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
NAK, not true.
Best regards,
Krzysztof
On 18/04/2024 14:28, Flavio Suligoi 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").
>
> So we can safely remove this property from this device-tree.
>
> Signed-off-by: Flavio Suligoi <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
NAK, not true.
Best regards,
Krzysztof
On 18/04/2024 14:28, Flavio Suligoi 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").
>
> So we can safely remove this property from this device-tree.
>
> Signed-off-by: Flavio Suligoi <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
NAK, not true. Please point me to this.
Best regards,
Krzysztof
On 18/04/2024 14:28, Flavio Suligoi 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").
>
> So we can safely remove this property from this device-tree.
>
> Signed-off-by: Flavio Suligoi <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
NAK. Not true. Please point me to my review for this patch.
Best regards,
Krzysztof
Hi Krzysztof,
...
> On 18/04/2024 14:28, Flavio Suligoi 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").
> >
> > So we can safely remove this property from this device-tree.
> >
> > Signed-off-by: Flavio Suligoi <[email protected]>
> > Reviewed-by: Krzysztof Kozlowski <[email protected]>
>
> NAK. Not true. Please point me to my review for this patch.
I'm very sorry for the misunderstanding, I saw your reply in:
https://lore.kernel.org/netdev/[email protected]/
and thought it might be okay for all the other patches.
Do you want me to resend a v3 version of the patches without the
"Reviewed-by: Krzysztof Kozlowski <[email protected]>" ?
>
> Best regards,
> Krzysztof
Thanks and best regards,
Flavio
On 19/04/2024 10:45, FLAVIO SULIGOI wrote:
> Hi Krzysztof,
>
> ...
>
>> On 18/04/2024 14:28, Flavio Suligoi 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").
>>>
>>> So we can safely remove this property from this device-tree.
>>>
>>> Signed-off-by: Flavio Suligoi <[email protected]>
>>> Reviewed-by: Krzysztof Kozlowski <[email protected]>
>>
>> NAK. Not true. Please point me to my review for this patch.
>
> I'm very sorry for the misunderstanding, I saw your reply in:
>
> https://lore.kernel.org/netdev/[email protected]/
>
> and thought it might be okay for all the other patches.
Where was it replied? In patch 5.
Why can't you use b4 for this? It solves the problems.
>
> Do you want me to resend a v3 version of the patches without the
> "Reviewed-by: Krzysztof Kozlowski <[email protected]>" ?
Yes, you must send new version because you added fake tags. Use b4
trailers to collect tags.
https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitting-patches.rst#L577
Best regards,
Krzysztof