[2/2] drivers: usb: musb: musb_gadget: replaced bare unsigned to unsigned int

Message ID 20230606102910.1013674-1-neelchakrabortykernelwork@gmail.com
State New
Headers
Series None |

Commit Message

Neel Chakraborty June 6, 2023, 10:29 a.m. UTC
  Replaced code that was using bare unsigned to use unsigned int

Signed-off-by: Neel Chakraborty <neelchakrabortykernelwork@gmail.com>
---
 drivers/usb/musb/musb_gadget.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Greg KH June 6, 2023, 10:33 a.m. UTC | #1
On Tue, Jun 06, 2023 at 03:59:10PM +0530, Neel Chakraborty wrote:
> Replaced code that was using bare unsigned to use unsigned int
> 
> Signed-off-by: Neel Chakraborty <neelchakrabortykernelwork@gmail.com>
> ---
>  drivers/usb/musb/musb_gadget.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Where is patch 1/2 of this series?

As it looks like you are new to kernel development, I strongly
recomomend you start out doing coding cleanups in the drivers/staging/
portion of the kernel first, as that is explicitly what it is there for.
That keeps kernel developers and maintainers of other subsystems from
having to work through basic issues like this.

Also, I recommend re-reading the kernel documentation for how to submit
a patch, this isn't the correct way :)

thanks,

greg k-h
  

Patch

diff --git a/drivers/usb/musb/musb_gadget.c b/drivers/usb/musb/musb_gadget.c
index 3cb7fc4c84ed..8912118883dc 100644
--- a/drivers/usb/musb/musb_gadget.c
+++ b/drivers/usb/musb/musb_gadget.c
@@ -521,7 +521,7 @@  static void rxstate(struct musb *musb, struct musb_request *req)
 	struct usb_request	*request = &req->request;
 	struct musb_ep		*musb_ep;
 	void __iomem		*epio = musb->endpoints[epnum].regs;
-	unsigned		len = 0;
+	unsigned int		len = 0;
 	u16			fifo_count;
 	u16			csr = musb_readw(epio, MUSB_RXCSR);
 	struct musb_hw_ep	*hw_ep = &musb->endpoints[epnum];