usb: dwc3: gadget: fix checkpatch warnings
authorFelipe Balbi <balbi@kernel.org>
Thu, 13 Aug 2020 05:35:38 +0000 (08:35 +0300)
committerFelipe Balbi <balbi@kernel.org>
Fri, 2 Oct 2020 06:43:36 +0000 (09:43 +0300)
no functional changes

Signed-off-by: Felipe Balbi <balbi@kernel.org>
drivers/usb/dwc3/gadget.c

index f9843ad..08111b9 100644 (file)
@@ -227,7 +227,8 @@ void dwc3_gadget_giveback(struct dwc3_ep *dep, struct dwc3_request *req,
  * Caller should take care of locking. Issue @cmd with a given @param to @dwc
  * and wait for its completion.
  */
-int dwc3_send_gadget_generic_command(struct dwc3 *dwc, unsigned cmd, u32 param)
+int dwc3_send_gadget_generic_command(struct dwc3 *dwc, unsigned int cmd,
+               u32 param)
 {
        u32             timeout = 500;
        int             status = 0;
@@ -268,7 +269,7 @@ static int __dwc3_gadget_wakeup(struct dwc3 *dwc);
  * Caller should handle locking. This function will issue @cmd with given
  * @params to @dep and wait for its completion.
  */
-int dwc3_send_gadget_ep_cmd(struct dwc3_ep *dep, unsigned cmd,
+int dwc3_send_gadget_ep_cmd(struct dwc3_ep *dep, unsigned int cmd,
                struct dwc3_gadget_ep_cmd_params *params)
 {
        const struct usb_endpoint_descriptor *desc = dep->endpoint.desc;
@@ -564,6 +565,7 @@ static int dwc3_gadget_set_ep_config(struct dwc3_ep *dep, unsigned int action)
        /* Burst size is only needed in SuperSpeed mode */
        if (dwc->gadget.speed >= USB_SPEED_SUPER) {
                u32 burst = dep->endpoint.maxburst;
+
                params.param0 |= DWC3_DEPCFG_BURST_SIZE(burst - 1);
        }
 
@@ -942,9 +944,10 @@ static u32 dwc3_calc_trbs_left(struct dwc3_ep *dep)
 }
 
 static void __dwc3_prepare_one_trb(struct dwc3_ep *dep, struct dwc3_trb *trb,
-               dma_addr_t dma, unsigned length, unsigned chain, unsigned node,
-               unsigned stream_id, unsigned short_not_ok,
-               unsigned no_interrupt, unsigned is_last)
+               dma_addr_t dma, unsigned int length, unsigned int chain,
+               unsigned int node, unsigned int stream_id,
+               unsigned int short_not_ok, unsigned int no_interrupt,
+               unsigned int is_last)
 {
        struct dwc3             *dwc = dep->dwc;
        struct usb_gadget       *gadget = &dwc->gadget;
@@ -1060,14 +1063,14 @@ static void __dwc3_prepare_one_trb(struct dwc3_ep *dep, struct dwc3_trb *trb,
  */
 static void dwc3_prepare_one_trb(struct dwc3_ep *dep,
                struct dwc3_request *req, unsigned int trb_length,
-               unsigned chain, unsigned node)
+               unsigned int chain, unsigned int node)
 {
        struct dwc3_trb         *trb;
        dma_addr_t              dma;
-       unsigned                stream_id = req->request.stream_id;
-       unsigned                short_not_ok = req->request.short_not_ok;
-       unsigned                no_interrupt = req->request.no_interrupt;
-       unsigned                is_last = req->request.is_last;
+       unsigned int            stream_id = req->request.stream_id;
+       unsigned int            short_not_ok = req->request.short_not_ok;
+       unsigned int            no_interrupt = req->request.no_interrupt;
+       unsigned int            is_last = req->request.is_last;
 
        if (req->request.num_sgs > 0)
                dma = sg_dma_address(req->start_sg);
@@ -1109,7 +1112,7 @@ static void dwc3_prepare_one_trb_sg(struct dwc3_ep *dep,
                unsigned int maxp = usb_endpoint_maxp(dep->endpoint.desc);
                unsigned int rem = length % maxp;
                unsigned int trb_length;
-               unsigned chain = true;
+               unsigned int chain = true;
 
                trb_length = min_t(unsigned int, length, sg_dma_len(s));
 
@@ -1653,9 +1656,8 @@ static int __dwc3_gadget_ep_queue(struct dwc3_ep *dep, struct dwc3_request *req)
                        return 0;
 
                if ((dep->flags & DWC3_EP_PENDING_REQUEST)) {
-                       if (!(dep->flags & DWC3_EP_TRANSFER_STARTED)) {
+                       if (!(dep->flags & DWC3_EP_TRANSFER_STARTED))
                                return __dwc3_gadget_start_isoc(dep);
-                       }
                }
        }
 
@@ -1793,8 +1795,8 @@ int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol)
        if (value) {
                struct dwc3_trb *trb;
 
-               unsigned transfer_in_flight;
-               unsigned started;
+               unsigned int transfer_in_flight;
+               unsigned int started;
 
                if (dep->number > 1)
                        trb = dwc3_ep_prev_trb(dep, dep->trb_enqueue);