2020-11-10 13:13:41

by Amelie Delaunay

[permalink] [raw]
Subject: [PATCH 1/1] ARM: dts: stm32: adjust USB OTG gadget fifo sizes in stm32mp151

Defaut use case on stm32mp151 USB OTG is ethernet gadget, using EP1 bulk
endpoint (MPS=512 bytes) and EP2 interrupt endpoint (MPS=16 bytes).
This patch optimizes USB OTG FIFO sizes accordingly.

Signed-off-by: Amelie Delaunay <[email protected]>
---
arch/arm/boot/dts/stm32mp151.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
index 6ffcf06dc0e8..928e31db1526 100644
--- a/arch/arm/boot/dts/stm32mp151.dtsi
+++ b/arch/arm/boot/dts/stm32mp151.dtsi
@@ -1070,9 +1070,9 @@
resets = <&rcc USBO_R>;
reset-names = "dwc2";
interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_HIGH>;
- g-rx-fifo-size = <256>;
+ g-rx-fifo-size = <512>;
g-np-tx-fifo-size = <32>;
- g-tx-fifo-size = <128 128 64 64 64 64 32 32>;
+ g-tx-fifo-size = <256 16 16 16 16 16 16 16>;
dr_mode = "otg";
usb33d-supply = <&usb33>;
status = "disabled";
--
2.17.1


2020-11-17 10:02:44

by Alexandre Torgue

[permalink] [raw]
Subject: Re: [PATCH 1/1] ARM: dts: stm32: adjust USB OTG gadget fifo sizes in stm32mp151

Hi Amélie

On 11/10/20 2:10 PM, Amelie Delaunay wrote:
> Defaut use case on stm32mp151 USB OTG is ethernet gadget, using EP1 bulk
> endpoint (MPS=512 bytes) and EP2 interrupt endpoint (MPS=16 bytes).
> This patch optimizes USB OTG FIFO sizes accordingly.
>
> Signed-off-by: Amelie Delaunay <[email protected]>
> ---
> arch/arm/boot/dts/stm32mp151.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
> index 6ffcf06dc0e8..928e31db1526 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -1070,9 +1070,9 @@
> resets = <&rcc USBO_R>;
> reset-names = "dwc2";
> interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_HIGH>;
> - g-rx-fifo-size = <256>;
> + g-rx-fifo-size = <512>;
> g-np-tx-fifo-size = <32>;
> - g-tx-fifo-size = <128 128 64 64 64 64 32 32>;
> + g-tx-fifo-size = <256 16 16 16 16 16 16 16>;
> dr_mode = "otg";
> usb33d-supply = <&usb33>;
> status = "disabled";
>

Applied on stm32-next.

Thanks.
Alex