From c3d10ad0aa8b8cc87384ad1367cea5ce12cb552e Mon Sep 17 00:00:00 2001 From: David Sidrane Date: Thu, 4 Nov 2021 05:10:27 -0700 Subject: [PATCH] imxrt:usbdev formatting cleanup --- arch/arm/src/imxrt/imxrt_usbdev.c | 173 +++++++++++++++--------------- 1 file changed, 87 insertions(+), 86 deletions(-) diff --git a/arch/arm/src/imxrt/imxrt_usbdev.c b/arch/arm/src/imxrt/imxrt_usbdev.c index 69f6f000b3..6a2e542e49 100644 --- a/arch/arm/src/imxrt/imxrt_usbdev.c +++ b/arch/arm/src/imxrt/imxrt_usbdev.c @@ -237,7 +237,7 @@ struct imxrt_dtd_s #define DTD_CONFIG_BUFFER_ERROR (1 << 5) /* Bit 6 : Status Buffer Error */ #define DTD_CONFIG_TRANSACTION_ERROR (1 << 3) /* Bit 3 : Status Transaction Error */ -/* This represents a queue head - not these must be aligned to a 2048 byte +/* This represents a queue head - note these must be aligned to a 2048 byte * boundary */ @@ -751,6 +751,7 @@ static inline void imxrt_writedtd(struct imxrt_dtd_s *dtd, (uintptr_t)dtd + sizeof(struct imxrt_dtd_s)); up_flush_dcache((uintptr_t)data, (uintptr_t)data + nbytes); + } /**************************************************************************** @@ -780,9 +781,9 @@ static void imxrt_queuedtd(uint8_t epphy, struct imxrt_dtd_s *dtd) uint32_t bit = IMXRT_ENDPTMASK(epphy); - imxrt_setbits (bit, IMXRT_USBDEV_ENDPTPRIME); + imxrt_setbits(bit, IMXRT_USBDEV_ENDPTPRIME); - while (imxrt_getreg (IMXRT_USBDEV_ENDPTPRIME) & bit) + while (imxrt_getreg(IMXRT_USBDEV_ENDPTPRIME) & bit) ; } @@ -841,7 +842,7 @@ static void imxrt_readsetup(uint8_t epphy, struct usb_ctrlreq_s *ctrl) /* Clear the Setup Interrupt */ - imxrt_putreg (IMXRT_ENDPTMASK(IMXRT_EP0_OUT), IMXRT_USBDEV_ENDPTSETUPSTAT); + imxrt_putreg(IMXRT_ENDPTMASK(IMXRT_EP0_OUT), IMXRT_USBDEV_ENDPTSETUPSTAT); } /**************************************************************************** @@ -876,7 +877,7 @@ static void imxrt_flushep(struct imxrt_ep_s *privep) uint32_t mask = IMXRT_ENDPTMASK(privep->epphy); do { - imxrt_putreg (mask, IMXRT_USBDEV_ENDPTFLUSH); + imxrt_putreg(mask, IMXRT_USBDEV_ENDPTFLUSH); while ((imxrt_getreg(IMXRT_USBDEV_ENDPTFLUSH) & mask) != 0) ; } @@ -910,13 +911,13 @@ static int imxrt_progressep(struct imxrt_ep_s *privep) if (privreq->req.len == 0) { /* If the class driver is responding to a setup packet, then wait for - * the host to illicit thr response + * the host to illicit the response */ if (privep->epphy == IMXRT_EP0_IN && privep->dev->ep0state == EP0STATE_SETUP_OUT) { - imxrt_ep0state (privep->dev, EP0STATE_WAIT_NAK_IN); + imxrt_ep0state(privep->dev, EP0STATE_WAIT_NAK_IN); } else { @@ -936,11 +937,11 @@ static int imxrt_progressep(struct imxrt_ep_s *privep) if (privep->epphy == IMXRT_EP0_IN) { - imxrt_ep0state (privep->dev, EP0STATE_DATA_IN); + imxrt_ep0state(privep->dev, EP0STATE_DATA_IN); } else if (privep->epphy == IMXRT_EP0_OUT) { - imxrt_ep0state (privep->dev, EP0STATE_DATA_OUT); + imxrt_ep0state(privep->dev, EP0STATE_DATA_OUT); } int bytesleft = privreq->req.len - privreq->req.xfrd; @@ -956,7 +957,7 @@ static int imxrt_progressep(struct imxrt_ep_s *privep) /* Initialise the DTD to transfer the next chunk */ - imxrt_writedtd (dtd, privreq->req.buf + privreq->req.xfrd, bytesleft); + imxrt_writedtd(dtd, privreq->req.buf + privreq->req.xfrd, bytesleft); /* Then queue onto the DQH */ @@ -1126,8 +1127,8 @@ static void imxrt_ep0configure(struct imxrt_usbdev_s *priv) /* Enable EP0 */ - imxrt_setbits (USBDEV_ENDPTCTRL0_RXE | USBDEV_ENDPTCTRL0_TXE, - IMXRT_USBDEV_ENDPTCTRL0); + imxrt_setbits(USBDEV_ENDPTCTRL0_RXE | USBDEV_ENDPTCTRL0_TXE, + IMXRT_USBDEV_ENDPTCTRL0); } /**************************************************************************** @@ -1144,34 +1145,34 @@ static void imxrt_usbreset(struct imxrt_usbdev_s *priv) /* Disable all endpoints. Control endpoint 0 is always enabled */ - imxrt_clrbits (USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL1); - imxrt_clrbits (USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL2); - imxrt_clrbits (USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL3); - imxrt_clrbits (USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL4); - imxrt_clrbits (USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL5); + imxrt_clrbits(USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL1); + imxrt_clrbits(USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL2); + imxrt_clrbits(USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL3); + imxrt_clrbits(USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL4); + imxrt_clrbits(USBDEV_ENDPTCTRL_RXE | USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL5); /* Clear all pending interrupts */ - imxrt_putreg (imxrt_getreg(IMXRT_USBDEV_ENDPTNAK), - IMXRT_USBDEV_ENDPTNAK); - imxrt_putreg (imxrt_getreg(IMXRT_USBDEV_ENDPTSETUPSTAT), - IMXRT_USBDEV_ENDPTSETUPSTAT); - imxrt_putreg (imxrt_getreg(IMXRT_USBDEV_ENDPTCOMPLETE), - IMXRT_USBDEV_ENDPTCOMPLETE); + imxrt_putreg(imxrt_getreg(IMXRT_USBDEV_ENDPTNAK), + IMXRT_USBDEV_ENDPTNAK); + imxrt_putreg(imxrt_getreg(IMXRT_USBDEV_ENDPTSETUPSTAT), + IMXRT_USBDEV_ENDPTSETUPSTAT); + imxrt_putreg(imxrt_getreg(IMXRT_USBDEV_ENDPTCOMPLETE), + IMXRT_USBDEV_ENDPTCOMPLETE); /* Wait for all prime operations to have completed and then flush all * DTDs */ - while (imxrt_getreg (IMXRT_USBDEV_ENDPTPRIME) != 0) + while (imxrt_getreg(IMXRT_USBDEV_ENDPTPRIME) != 0) ; - imxrt_putreg (IMXRT_ENDPTMASK_ALL, IMXRT_USBDEV_ENDPTFLUSH); - while (imxrt_getreg (IMXRT_USBDEV_ENDPTFLUSH)) + imxrt_putreg(IMXRT_ENDPTMASK_ALL, IMXRT_USBDEV_ENDPTFLUSH); + while (imxrt_getreg(IMXRT_USBDEV_ENDPTFLUSH)) ; /* Reset endpoints */ @@ -1180,7 +1181,7 @@ static void imxrt_usbreset(struct imxrt_usbdev_s *priv) { struct imxrt_ep_s *privep = &priv->eplist[epphy]; - imxrt_cancelrequests (privep, -ESHUTDOWN); + imxrt_cancelrequests(privep, -ESHUTDOWN); /* Reset endpoint status */ @@ -1211,11 +1212,11 @@ static void imxrt_usbreset(struct imxrt_usbdev_s *priv) /* Set USB address to 0 */ - imxrt_set_address (priv, 0); + imxrt_set_address(priv, 0); /* Initialise the Enpoint List Address */ - imxrt_putreg ((uint32_t)g_qh, IMXRT_USBDEV_ENDPOINTLIST); + imxrt_putreg((uint32_t)g_qh, IMXRT_USBDEV_ENDPOINTLIST); /* EndPoint 0 initialization */ @@ -1244,11 +1245,11 @@ static inline void imxrt_ep0state(struct imxrt_usbdev_s *priv, switch (state) { case EP0STATE_WAIT_NAK_IN: - imxrt_putreg (IMXRT_ENDPTMASK(IMXRT_EP0_IN), IMXRT_USBDEV_ENDPTNAKEN); + imxrt_putreg(IMXRT_ENDPTMASK(IMXRT_EP0_IN), IMXRT_USBDEV_ENDPTNAKEN); break; case EP0STATE_WAIT_NAK_OUT: - imxrt_putreg (IMXRT_ENDPTMASK(IMXRT_EP0_OUT), IMXRT_USBDEV_ENDPTNAKEN); + imxrt_putreg(IMXRT_ENDPTMASK(IMXRT_EP0_OUT), IMXRT_USBDEV_ENDPTNAKEN); break; default: @@ -1308,11 +1309,11 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) if (ctrl->type & USB_REQ_DIR_IN) { - imxrt_ep0state (priv, EP0STATE_SETUP_IN); + imxrt_ep0state(priv, EP0STATE_SETUP_IN); } else { - imxrt_ep0state (priv, EP0STATE_SETUP_OUT); + imxrt_ep0state(priv, EP0STATE_SETUP_OUT); if (len > 0) { @@ -1379,8 +1380,8 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) priv->ep0buf[1] = 0; - imxrt_ep0xfer (IMXRT_EP0_IN, priv->ep0buf, 2); - imxrt_ep0state (priv, EP0STATE_SHORTWRITE); + imxrt_ep0xfer(IMXRT_EP0_IN, priv->ep0buf, 2); + imxrt_ep0state(priv, EP0STATE_SHORTWRITE); } } break; @@ -1402,7 +1403,7 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) priv->ep0buf[1] = 0; imxrt_ep0xfer(IMXRT_EP0_IN, priv->ep0buf, 2); - imxrt_ep0state (priv, EP0STATE_SHORTWRITE); + imxrt_ep0state(priv, EP0STATE_SHORTWRITE); } else { @@ -1422,7 +1423,7 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) priv->ep0buf[1] = 0; imxrt_ep0xfer(IMXRT_EP0_IN, priv->ep0buf, 2); - imxrt_ep0state (priv, EP0STATE_SHORTWRITE); + imxrt_ep0state(priv, EP0STATE_SHORTWRITE); } break; @@ -1457,7 +1458,7 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) len == 0 && (privep = imxrt_epfindbyaddr(priv, index)) != NULL) { imxrt_epstall(&privep->ep, true); - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_IN); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_IN); } else { @@ -1491,7 +1492,7 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) len == 0 && (privep = imxrt_epfindbyaddr(priv, index)) != NULL) { imxrt_epstall(&privep->ep, false); - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_IN); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_IN); } else { @@ -1521,7 +1522,7 @@ static inline void imxrt_ep0setup(struct imxrt_usbdev_s *priv) priv->paddr = ctrl->value[0]; priv->paddrset = false; - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_IN); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_IN); } else { @@ -1679,9 +1680,9 @@ static void imxrt_ep0complete(struct imxrt_usbdev_s *priv, uint8_t epphy) return; } - if (imxrt_epcomplete (priv, epphy)) + if (imxrt_epcomplete(priv, epphy)) { - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_OUT); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_OUT); } break; @@ -1691,9 +1692,9 @@ static void imxrt_ep0complete(struct imxrt_usbdev_s *priv, uint8_t epphy) return; } - if (imxrt_epcomplete (priv, epphy)) + if (imxrt_epcomplete(priv, epphy)) { - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_IN); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_IN); } break; @@ -1707,15 +1708,15 @@ static void imxrt_ep0complete(struct imxrt_usbdev_s *priv, uint8_t epphy) (uintptr_t)priv->ep0buf + sizeof(priv->ep0buf)); imxrt_dispatchrequest(priv, &priv->ep0ctrl); - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_IN); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_IN); break; case EP0STATE_SHORTWRITE: - imxrt_ep0state (priv, EP0STATE_WAIT_NAK_OUT); + imxrt_ep0state(priv, EP0STATE_WAIT_NAK_OUT); break; case EP0STATE_WAIT_STATUS_IN: - imxrt_ep0state (priv, EP0STATE_IDLE); + imxrt_ep0state(priv, EP0STATE_IDLE); /* If we've received a SETADDRESS packet, then we set the address * now that the status phase has completed @@ -1725,13 +1726,13 @@ static void imxrt_ep0complete(struct imxrt_usbdev_s *priv, uint8_t epphy) { usbtrace(TRACE_INTDECODE(IMXRT_TRACEINTID_EP0INSETADDRESS), (uint16_t)priv->paddr); - imxrt_set_address (priv, priv->paddr); + imxrt_set_address(priv, priv->paddr); } break; case EP0STATE_WAIT_STATUS_OUT: - imxrt_ep0state (priv, EP0STATE_IDLE); + imxrt_ep0state(priv, EP0STATE_IDLE); break; default: @@ -1771,13 +1772,13 @@ static void imxrt_ep0nak(struct imxrt_usbdev_s *priv, uint8_t epphy) switch (priv->ep0state) { case EP0STATE_WAIT_NAK_IN: - imxrt_ep0xfer (IMXRT_EP0_IN, NULL, 0); - imxrt_ep0state (priv, EP0STATE_WAIT_STATUS_IN); + imxrt_ep0xfer(IMXRT_EP0_IN, NULL, 0); + imxrt_ep0state(priv, EP0STATE_WAIT_STATUS_IN); break; case EP0STATE_WAIT_NAK_OUT: - imxrt_ep0xfer (IMXRT_EP0_OUT, NULL, 0); - imxrt_ep0state (priv, EP0STATE_WAIT_STATUS_OUT); + imxrt_ep0xfer(IMXRT_EP0_OUT, NULL, 0); + imxrt_ep0state(priv, EP0STATE_WAIT_STATUS_OUT); break; default: @@ -1869,7 +1870,7 @@ bool imxrt_epcomplete(struct imxrt_usbdev_s *priv, uint8_t epphy) if (complete) { - privreq = imxrt_rqdequeue (privep); + privreq = imxrt_rqdequeue(privep); } if (!imxrt_rqempty(privep)) @@ -2005,14 +2006,14 @@ static int imxrt_usbinterrupt(int irq, FAR void *context, FAR void *arg) { /* Handle completion interrupts */ - uint32_t mask = imxrt_getreg (IMXRT_USBDEV_ENDPTCOMPLETE); + uint32_t mask = imxrt_getreg(IMXRT_USBDEV_ENDPTCOMPLETE); if (mask) { /* Clear any NAK interrupt and completion interrupts */ - imxrt_putreg (mask, IMXRT_USBDEV_ENDPTNAK); - imxrt_putreg (mask, IMXRT_USBDEV_ENDPTCOMPLETE); + imxrt_putreg(mask, IMXRT_USBDEV_ENDPTNAK); + imxrt_putreg(mask, IMXRT_USBDEV_ENDPTCOMPLETE); if (mask & IMXRT_ENDPTMASK(0)) { @@ -2028,7 +2029,7 @@ static int imxrt_usbinterrupt(int irq, FAR void *context, FAR void *arg) { if (mask & IMXRT_ENDPTMASK((n << 1))) { - imxrt_epcomplete (priv, (n << 1)); + imxrt_epcomplete(priv, (n << 1)); } if (mask & IMXRT_ENDPTMASK((n << 1)+1)) @@ -2157,8 +2158,8 @@ static int imxrt_epconfigure(FAR struct usbdev_ep_s *ep, cfg |= USBDEV_ENDPTCTRL_TXT_INTR; break; } - imxrt_chgbits (0xffff0000, cfg, - IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); + imxrt_chgbits(0xffff0000, cfg, + IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); } else { @@ -2180,8 +2181,8 @@ static int imxrt_epconfigure(FAR struct usbdev_ep_s *ep, cfg |= USBDEV_ENDPTCTRL_RXT_INTR; break; } - imxrt_chgbits (0x0000ffff, cfg, - IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); + imxrt_chgbits(0x0000ffff, cfg, + IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); } /* Reset endpoint status */ @@ -2192,13 +2193,13 @@ static int imxrt_epconfigure(FAR struct usbdev_ep_s *ep, if (IMXRT_EPPHYIN(privep->epphy)) { - imxrt_setbits (USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); + imxrt_setbits(USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); } else { - imxrt_setbits (USBDEV_ENDPTCTRL_RXE, - IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); + imxrt_setbits(USBDEV_ENDPTCTRL_RXE, + IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); } return OK; @@ -2233,13 +2234,13 @@ static int imxrt_epdisable(FAR struct usbdev_ep_s *ep) if (IMXRT_EPPHYIN(privep->epphy)) { - imxrt_clrbits (USBDEV_ENDPTCTRL_TXE, - IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); + imxrt_clrbits(USBDEV_ENDPTCTRL_TXE, + IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); } else { - imxrt_clrbits (USBDEV_ENDPTCTRL_RXE, - IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); + imxrt_clrbits(USBDEV_ENDPTCTRL_RXE, + IMXRT_USBDEV_ENDPTCTRL(privep->epphy >> 1)); } privep->stalled = true; @@ -2502,13 +2503,13 @@ static int imxrt_epstall(FAR struct usbdev_ep_s *ep, bool resume) /* Clear stall and reset the data toggle */ - imxrt_chgbits (ctrl_xs | ctrl_xr, ctrl_xr, addr); + imxrt_chgbits(ctrl_xs | ctrl_xr, ctrl_xr, addr); } else { privep->stalled = true; - imxrt_setbits (ctrl_xs, addr); + imxrt_setbits(ctrl_xs, addr); } leave_critical_section(flags); @@ -2769,18 +2770,18 @@ static int imxrt_pullup(struct usbdev_s *dev, bool enable) irqstate_t flags = enter_critical_section(); if (enable) { - imxrt_setbits (USBDEV_USBCMD_RS, IMXRT_USBDEV_USBCMD); + imxrt_setbits(USBDEV_USBCMD_RS, IMXRT_USBDEV_USBCMD); #ifdef CONFIG_IMXRT_USB0DEV_NOVBUS /* Create a 'false' power event on the USB port so the MAC connects */ - imxrt_clrbits (USBOTG_OTGSC_VD, IMXRT_USBOTG_OTGSC); - imxrt_setbits (USBOTG_OTGSC_VC, IMXRT_USBOTG_OTGSC); + imxrt_clrbits(USBOTG_OTGSC_VD, IMXRT_USBOTG_OTGSC); + imxrt_setbits(USBOTG_OTGSC_VC, IMXRT_USBOTG_OTGSC); #endif } else { - imxrt_clrbits (USBDEV_USBCMD_RS, IMXRT_USBDEV_USBCMD); + imxrt_clrbits(USBDEV_USBCMD_RS, IMXRT_USBDEV_USBCMD); } leave_critical_section(flags); @@ -2887,8 +2888,8 @@ void arm_usbinitialize(void) /* Reset the controller */ - imxrt_setbits (USBDEV_USBCMD_RST, IMXRT_USBDEV_USBCMD); - while (imxrt_getreg (IMXRT_USBDEV_USBCMD) & USBDEV_USBCMD_RST) + imxrt_setbits(USBDEV_USBCMD_RST, IMXRT_USBDEV_USBCMD); + while (imxrt_getreg(IMXRT_USBDEV_USBCMD) & USBDEV_USBCMD_RST) ; /* Power up the PHY (turn off power disable) - USBPHYx_PWDn @@ -2902,7 +2903,7 @@ void arm_usbinitialize(void) /* Program the controller to be the USB device controller */ - imxrt_putreg (USBDEV_USBMODE_SDIS | USBDEV_USBMODE_SLOM | + imxrt_putreg(USBDEV_USBMODE_SDIS | USBDEV_USBMODE_SLOM | USBDEV_USBMODE_CM_DEVICE, IMXRT_USBDEV_USBMODE); /* Attach USB controller interrupt handler */ @@ -2950,8 +2951,8 @@ void arm_usbuninitialize(void) /* Reset the controller */ - imxrt_setbits (USBDEV_USBCMD_RST, IMXRT_USBDEV_USBCMD); - while (imxrt_getreg (IMXRT_USBDEV_USBCMD) & USBDEV_USBCMD_RST) + imxrt_setbits(USBDEV_USBCMD_RST, IMXRT_USBDEV_USBCMD); + while (imxrt_getreg(IMXRT_USBDEV_USBCMD) & USBDEV_USBCMD_RST) ; /* Turn off USB power and clocking */