[PATCH v2 net 3/4] powerpc/fsl/dts: Enable WA for erratum A-009885 on fman3l MDIO buses

2022-01-18 Thread Tobias Waldekranz
This block is used in (at least) T1024 and T1040, including their variants like T1023 etc. Fixes: d55ad2967d89 ("powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan") Signed-off-by: Tobias Waldekranz --- arch/powerpc/boot/dts/fsl/qoriq-fman3l-0.dtsi | 2 ++ 1 file

Re: [PATCH net 3/4] powerpc/fsl/dts: Enable WA for erratum A-009885 on fman3l MDIO buses

2022-01-18 Thread Tobias Waldekranz
On Sun, Jan 16, 2022 at 22:15, Tobias Waldekranz wrote: > This block is used in (at least) T1024 and T1040, including their > variants like T1023 etc. > > Signed-off-by: Tobias Waldekranz Fixes: d55ad2967d89 ("powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan")

Re: [PATCH net 1/4] net/fsl: xgmac_mdio: Add workaround for erratum A-009885

2022-01-18 Thread Tobias Waldekranz
On Sun, Jan 16, 2022 at 22:15, Tobias Waldekranz wrote: > Once an MDIO read transaction is initiated, we must read back the data > register within 16 MDC cycles after the transaction completes. Outside > of this window, reads may return corrupt data. > > Therefore, disable l

Re: [PATCH net 4/4] net/fsl: xgmac_mdio: Fix incorrect iounmap when removing module

2022-01-16 Thread Tobias Waldekranz
On Sun, Jan 16, 2022 at 22:54, Andrew Lunn wrote: > On Sun, Jan 16, 2022 at 10:15:29PM +0100, Tobias Waldekranz wrote: >> As reported by sparse: In the remove path, the driver would attempt to >> unmap its own priv pointer - instead of the io memory that it mapped >> in

Re: [PATCH net 1/4] net/fsl: xgmac_mdio: Add workaround for erratum A-009885

2022-01-16 Thread Tobias Waldekranz
On Sun, Jan 16, 2022 at 23:02, Andrew Lunn wrote: > On Sun, Jan 16, 2022 at 10:15:26PM +0100, Tobias Waldekranz wrote: >> Once an MDIO read transaction is initiated, we must read back the data >> register within 16 MDC cycles after the transaction completes. Outside >> of t

[PATCH net 0/4] net/fsl: xgmac_mdio: Add workaround for erratum A-009885

2022-01-16 Thread Tobias Waldekranz
someone from NXP can supply a follow-up if need be. The final commit is an unrelated fix that was brought to my attention by sparse. Tobias Waldekranz (4): net/fsl: xgmac_mdio: Add workaround for erratum A-009885 dt-bindings: net: Document fsl,erratum-a009885 powerpc/fsl/dts: Enable WA

[PATCH net 1/4] net/fsl: xgmac_mdio: Add workaround for erratum A-009885

2022-01-16 Thread Tobias Waldekranz
this requirement. Signed-off-by: Tobias Waldekranz --- drivers/net/ethernet/freescale/xgmac_mdio.c | 25 - 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/drivers/net/ethernet/freescale/xgmac_mdio.c b/drivers/net/ethernet/freescale/xgmac_mdio.c index

[PATCH net 4/4] net/fsl: xgmac_mdio: Fix incorrect iounmap when removing module

2022-01-16 Thread Tobias Waldekranz
As reported by sparse: In the remove path, the driver would attempt to unmap its own priv pointer - instead of the io memory that it mapped in probe. Fixes: 9f35a7342cff ("net/fsl: introduce Freescale 10G MDIO driver") Signed-off-by: Tobias Waldekranz --- drivers/net/ethernet

[PATCH net 3/4] powerpc/fsl/dts: Enable WA for erratum A-009885 on fman3l MDIO buses

2022-01-16 Thread Tobias Waldekranz
This block is used in (at least) T1024 and T1040, including their variants like T1023 etc. Signed-off-by: Tobias Waldekranz --- arch/powerpc/boot/dts/fsl/qoriq-fman3l-0.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/powerpc/boot/dts/fsl/qoriq-fman3l-0.dtsi b/arch/powerpc/boot

[PATCH net 2/4] dt-bindings: net: Document fsl,erratum-a009885

2022-01-16 Thread Tobias Waldekranz
Update FMan binding documentation with the newly added workaround for erratum A-009885. Signed-off-by: Tobias Waldekranz --- Documentation/devicetree/bindings/net/fsl-fman.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/devicetree/bindings/net/fsl-fman.txt b