[RESEND PATCH 10/16] usb: musb: Ensure that we set musb dynamic FIFO buffer for every endpoint

Lukasz Majewski
2021-02-06T15:18:59Z

On Fri, 5 Feb 2021 20:12:06 +0100
Pali Roh?r <pali@kernel.org> wrote:

> If we do not set FIFO buffer address and size for some endpoint which
> is in use then default address 0x0 would be used which is in conflict
> with FIFO buffer for endpoint 0 which is at fixed address 0x0.
> Sharing address space between more endpoint cause data loss and
> unexpected errors.
>
> This patch is fixing transmission of characters over usbtty serial
> console and allow using of usbtty for debugging purposes on Nokia
> N900.

Reviewed-by: Lukasz Majewski <lukma@denx.de>

>
> Signed-off-by: Pali Roh?r <pali@kernel.org>
> ---
> drivers/usb/musb/musb_udc.c | 14 ++------------
> 1 file changed, 2 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/usb/musb/musb_udc.c b/drivers/usb/musb/musb_udc.c
> index 50d8bc319c..ea1284850e 100644
> --- a/drivers/usb/musb/musb_udc.c
> +++ b/drivers/usb/musb/musb_udc.c
> @@ -875,18 +875,8 @@ void udc_setup_ep(struct usb_device_instance
> *device, unsigned int id, ep0_endpoint->endpoint_address = 0xff;
> ep0_urb = usbd_alloc_urb(device, endpoint);
> } else if (MAX_ENDPOINT >= id) {
> - int ep_addr;
> -
> - /* Check the direction */
> - ep_addr = endpoint->endpoint_address;
> - if (USB_DIR_IN == (ep_addr & USB_ENDPOINT_DIR_MASK))
> {
> - /* IN */
> - epinfo[(id * 2) + 1].epsize =
> endpoint->tx_packetSize;
> - } else {
> - /* OUT */
> - epinfo[id * 2].epsize =
> endpoint->rcv_packetSize;
> - }
> -
> + epinfo[(id * 2) + 0].epsize =
> endpoint->rcv_packetSize;
> + epinfo[(id * 2) + 1].epsize =
> endpoint->tx_packetSize; musb_configure_ep(&epinfo[0],
> ARRAY_SIZE(epinfo)); } else {
> if (debug_level > 0)

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210206/27ac89da/attachment.sig>