On Friday, January 22, 2016 at 02:30:43 AM, Stefan Brüns wrote: > USB protocol allows for 16 IN and 16 OUT endpoints (USB 2.0 Spec, > 8.3.2.2 Endpoint Field). A function may have an EP 1 for both IN and OUT, > so these two should be kept separate. As EPs are either BULK or INTERRUPT > (or ISO), it is fine to have one array per direction for all transfer > types (also see e236519b7365ef75c5da6a5623f0b03d9c00cfae). > > USB device address is 7 bits, so a bus may have more than 16 devices. > Check the device number, as the DWC2 driver only supports BULK/ISO for > the first 16 devices. > > Signed-off-by: Stefan Brüns <stefan.bru...@rwth-aachen.de> > ---
Since this is patch 7/7, where are the remaining 6 ? > drivers/usb/host/dwc2.c | 29 +++++++++++++++++++---------- > 1 file changed, 19 insertions(+), 10 deletions(-) > > diff --git a/drivers/usb/host/dwc2.c b/drivers/usb/host/dwc2.c > index 291e4a5..7c107bc 100644 > --- a/drivers/usb/host/dwc2.c > +++ b/drivers/usb/host/dwc2.c > @@ -34,7 +34,8 @@ struct dwc2_priv { > uint8_t *aligned_buffer; > uint8_t *status_buffer; > #endif > - int bulk_data_toggle[MAX_DEVICE][MAX_ENDPOINT]; > + uint8_t in_data_toggle[MAX_DEVICE][MAX_ENDPOINT]; > + uint8_t out_data_toggle[MAX_DEVICE][MAX_ENDPOINT]; Use u8/u16/u32 please. All around the place . > struct dwc2_core_regs *regs; > int root_hub_devnum; > }; The rest is fine, Acked-by: Marek Vasut <ma...@denx.de> Thanks! Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot