| Message ID | 20191216064132.78015-1-joel@jms.id.au (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | d4d171bd3e04f4c2400a19f0afd73b8c0efa9894 |
| Headers | show |
| Series | ARM: dts: aspeed: AST2400 disables hw checksum | expand |
On Mon, Dec 16, 2019 at 05:41:32PM +1100, Joel Stanley wrote: > There is no need to specify this property in the device tree as the > AST2400 does not have working hardware checksum and disables it in the > driver. > > Signed-off-by: Joel Stanley <joel@jms.id.au> > --- > arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts | 1 - > arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts | 1 - > arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts | 3 --- > 3 files changed, 5 deletions(-) > > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts > index b1e10f0c85c9..322587b7b67d 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts > @@ -76,7 +76,6 @@ &uart5 { > > &mac1 { > status = "okay"; > - no-hw-checksum; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; > }; > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts > index aaa77a597d1a..54e508530dce 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts > @@ -75,7 +75,6 @@ &uart5 { > > &mac1 { > status = "okay"; > - no-hw-checksum; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; > }; > diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts > index affd2c8743b1..041f28e3ac10 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts > @@ -107,10 +107,7 @@ flash@0 { > > &mac0 { > status = "okay"; > - > use-ncsi; > - no-hw-checksum; > - > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_rmii1_default>; > }; > -- > 2.24.0 Thanks for the fix, Joel. I totally forget I added the property in ast2400 dts. Reviewed-by: Tao Ren <rentao.bupt@gmail.com>
Reviewed-by: Alexander Filippov <a.filippov@yadro.com> On Mon, Dec 16, 2019 at 05:41:32PM +1100, Joel Stanley wrote: > There is no need to specify this property in the device tree as the > AST2400 does not have working hardware checksum and disables it in the > driver. > > Signed-off-by: Joel Stanley <joel@jms.id.au> > --- > arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts | 1 - > arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts | 1 - > arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts | 3 --- > 3 files changed, 5 deletions(-) > > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts > index b1e10f0c85c9..322587b7b67d 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts > @@ -76,7 +76,6 @@ &uart5 { > > &mac1 { > status = "okay"; > - no-hw-checksum; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; > }; > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts > index aaa77a597d1a..54e508530dce 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts > @@ -75,7 +75,6 @@ &uart5 { > > &mac1 { > status = "okay"; > - no-hw-checksum; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; > }; > diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts > index affd2c8743b1..041f28e3ac10 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts > @@ -107,10 +107,7 @@ flash@0 { > > &mac0 { > status = "okay"; > - > use-ncsi; > - no-hw-checksum; > - > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_rmii1_default>; > }; > -- > 2.24.0 >
diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts index b1e10f0c85c9..322587b7b67d 100644 --- a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts @@ -76,7 +76,6 @@ &uart5 { &mac1 { status = "okay"; - no-hw-checksum; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; }; diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts index aaa77a597d1a..54e508530dce 100644 --- a/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts @@ -75,7 +75,6 @@ &uart5 { &mac1 { status = "okay"; - no-hw-checksum; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; }; diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts index affd2c8743b1..041f28e3ac10 100644 --- a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts +++ b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts @@ -107,10 +107,7 @@ flash@0 { &mac0 { status = "okay"; - use-ncsi; - no-hw-checksum; - pinctrl-names = "default"; pinctrl-0 = <&pinctrl_rmii1_default>; };
There is no need to specify this property in the device tree as the AST2400 does not have working hardware checksum and disables it in the driver. Signed-off-by: Joel Stanley <joel@jms.id.au> --- arch/arm/boot/dts/aspeed-bmc-facebook-wedge100.dts | 1 - arch/arm/boot/dts/aspeed-bmc-facebook-wedge40.dts | 1 - arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts | 3 --- 3 files changed, 5 deletions(-)