Removed the uses of a table to convert the usb endpoint type for the controller. Signed-off-by: Jules Maselbas <jmaselbas@xxxxxxxxx> --- drivers/usb/dwc2/host.c | 47 +++++++++++++++++++++++------------------ 1 file changed, 27 insertions(+), 20 deletions(-) diff --git a/drivers/usb/dwc2/host.c b/drivers/usb/dwc2/host.c index a97c7dac5..083c964e8 100644 --- a/drivers/usb/dwc2/host.c +++ b/drivers/usb/dwc2/host.c @@ -7,13 +7,6 @@ /* Use only HC channel 0. */ #define DWC2_HC_CHANNEL 0 -static int dwc2_eptype[] = { - DXEPCTL_EPTYPE_ISO, - DXEPCTL_EPTYPE_INTERRUPT, - DXEPCTL_EPTYPE_CONTROL, - DXEPCTL_EPTYPE_BULK, -}; - static int dwc2_do_split(struct dwc2 *dwc2, struct usb_device *dev) { uint32_t hprt0 = dwc2_readl(dwc2, HPRT0); @@ -80,15 +73,32 @@ static void dwc2_hc_enable_ints(struct dwc2 *dwc2, uint8_t hc) * @param regs Programming view of DWC2 controller * @param hc Information needed to initialize the host channel */ -static void dwc2_hc_init(struct dwc2 *dwc2, uint8_t hc, - struct usb_device *dev, uint8_t dev_addr, uint8_t ep_num, - uint8_t ep_is_in, uint32_t ep_type, uint16_t max_packet) +static void dwc2_hc_init(struct dwc2 *dwc2, struct usb_device *dev, u8 hc, + unsigned long pipe, int is_in) { - uint32_t hcchar = (dev_addr << HCCHAR_DEVADDR_SHIFT) | - (ep_num << HCCHAR_EPNUM_SHIFT) | - (ep_is_in ? HCCHAR_EPDIR : 0) | - ep_type | - (max_packet << HCCHAR_MPS_SHIFT); + int addr = usb_pipedevice(pipe); + int endp = usb_pipeendpoint(pipe); + int type = usb_pipetype(pipe); + int mps = usb_maxpacket(dev, pipe); + uint32_t hcchar = (addr << HCCHAR_DEVADDR_SHIFT) | + (endp << HCCHAR_EPNUM_SHIFT) | + (is_in ? HCCHAR_EPDIR : 0) | + (mps << HCCHAR_MPS_SHIFT); + + switch (type) { + case PIPE_ISOCHRONOUS: + hcchar |= DXEPCTL_EPTYPE_ISO; + break; + case PIPE_INTERRUPT: + hcchar |= DXEPCTL_EPTYPE_INTERRUPT; + break; + case PIPE_CONTROL: + hcchar |= DXEPCTL_EPTYPE_CONTROL; + break; + case PIPE_BULK: + hcchar |= DXEPCTL_EPTYPE_BULK; + break; + } if (dev->speed == USB_SPEED_LOW) hcchar |= HCCHAR_LSPDDEV; @@ -207,10 +217,7 @@ static int dwc2_submit_packet(struct dwc2 *dwc2, struct usb_device *dev, unsigned long pipe, u8 *pid, int in, void *buf, int len) { - int devnum = usb_pipedevice(pipe); - int ep = usb_pipeendpoint(pipe); int mps = usb_maxpacket(dev, pipe); - int eptype = dwc2_eptype[usb_pipetype(pipe)]; int do_split = dwc2_do_split(dwc2, dev); int complete_split = 0; int done = 0; @@ -231,7 +238,7 @@ static int dwc2_submit_packet(struct dwc2 *dwc2, struct usb_device *dev, max_xfer_len = num_packets * mps; /* Initialize channel */ - dwc2_hc_init(dwc2, DWC2_HC_CHANNEL, dev, devnum, ep, in, eptype, mps); + dwc2_hc_init(dwc2, dev, DWC2_HC_CHANNEL, pipe, in); /* Check if the target is a FS/LS device behind a HS hub */ if (do_split) { @@ -262,7 +269,7 @@ static int dwc2_submit_packet(struct dwc2 *dwc2, struct usb_device *dev, dwc2_writel(dwc2, hcsplt, HCSPLT(DWC2_HC_CHANNEL)); } - if (eptype == DXEPCTL_EPTYPE_INTERRUPT) { + if (usb_pipeint(pipe)) { int uframe_num = dwc2_readl(dwc2, HFNUM); if (!(uframe_num & 0x1)) -- 2.21.0.196.g041f5ea _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox