| Message ID | 20190718090301.19283-1-jbrunet@baylibre.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | arm64: dts: meson: add ethernet fifo sizes | expand |
On Thu, Jul 18, 2019 at 11:03 AM Jerome Brunet <jbrunet@baylibre.com> wrote: > > If unspecified in DT, the fifo sizes are not automatically detected by > the dwmac1000 dma driver and the reported fifo sizes default to 0. > Because of this, flow control will be turned off on the device. > > Add the fifo sizes provided by the datasheets in the SoC in DT so > flow control may be enabled if necessary. (only worth updating if you have to re-send it for whatever reason) as far as I understand the stmmac code (dwmac1000_dma_operation_mode_{rx,tx}) "RX flow control" depends on the RX FIFO size but TX doesn't > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Jerome Brunet <jbrunet@baylibre.com> writes: > If unspecified in DT, the fifo sizes are not automatically detected by > the dwmac1000 dma driver and the reported fifo sizes default to 0. > Because of this, flow control will be turned off on the device. > > Add the fifo sizes provided by the datasheets in the SoC in DT so > flow control may be enabled if necessary. > > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> Queued for v5.4, Thanks, Kevin
diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi index 6219337033a0..12bf959c17a7 100644 --- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi @@ -182,6 +182,8 @@ <&clkc CLKID_FCLK_DIV2>, <&clkc CLKID_MPLL2>; clock-names = "stmmaceth", "clkin0", "clkin1"; + rx-fifo-depth = <4096>; + tx-fifo-depth = <2048>; status = "disabled"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi index 0bb9705a33f1..c8b9b9754598 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi @@ -150,6 +150,8 @@ <&clkc CLKID_FCLK_DIV2>, <&clkc CLKID_MPLL2>; clock-names = "stmmaceth", "clkin0", "clkin1"; + rx-fifo-depth = <4096>; + tx-fifo-depth = <2048>; status = "disabled"; mdio0: mdio { diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi index 74d03fc706be..e62aad5bf867 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi @@ -493,6 +493,8 @@ 0x0 0xc8834540 0x0 0x4>; interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>; interrupt-names = "macirq"; + rx-fifo-depth = <4096>; + tx-fifo-depth = <2048>; status = "disabled"; };
If unspecified in DT, the fifo sizes are not automatically detected by the dwmac1000 dma driver and the reported fifo sizes default to 0. Because of this, flow control will be turned off on the device. Add the fifo sizes provided by the datasheets in the SoC in DT so flow control may be enabled if necessary. Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> --- arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 2 ++ arch/arm64/boot/dts/amlogic/meson-g12a.dtsi | 2 ++ arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 2 ++ 3 files changed, 6 insertions(+)