usb: dwc2: rockchip: Make the max_transfer_size automatic
authorDouglas Anderson <dianders@chromium.org>
Fri, 29 Jan 2016 02:19:52 +0000 (18:19 -0800)
committerFelipe Balbi <balbi@kernel.org>
Fri, 4 Mar 2016 13:14:39 +0000 (15:14 +0200)
Previously we needed to set the max_transfer_size to explicitly be 65535
because the old driver would detect that our hardware could support much
bigger transfers and then would try to do them.  This wouldn't work
since the DMA alignment code couldn't support it.

Later in commit e8f8c14d9da7 ("usb: dwc2: clip max_transfer_size to
65535") upstream added support for clipping this automatically.  Since
that commit it has been OK to just use "-1" (default), but nobody
bothered to change it.

Let's change it to default now for two reasons:
- It's nice to use autodetected params.
- If we can remove the 65535 limit, we can transfer more!

Signed-off-by: Douglas Anderson <dianders@chromium.org>
Acked-by: John Youn <johnyoun@synopsys.com>
Tested-by: Heiko Stuebner <heiko@sntech.de>
Signed-off-by: Felipe Balbi <balbi@kernel.org>
drivers/usb/dwc2/platform.c

index 690b9fd98b55165a8b1d6a7b8bce30f730ac6805..ecaa3d20e1350e43c8368c9f342b16e22c8817d4 100644 (file)
@@ -129,7 +129,7 @@ static const struct dwc2_core_params params_rk3066 = {
        .host_rx_fifo_size              = 520,  /* 520 DWORDs */
        .host_nperio_tx_fifo_size       = 128,  /* 128 DWORDs */
        .host_perio_tx_fifo_size        = 256,  /* 256 DWORDs */
-       .max_transfer_size              = 65535,
+       .max_transfer_size              = -1,
        .max_packet_count               = -1,
        .host_channels                  = -1,
        .phy_type                       = -1,