Hi Zixun, Thank you for the patch.
On mar., juil. 23, 2024 at 15:18, Zixun LI <ad...@hifiphile.com> wrote: > To have a uniform printing function, also drop linux/printk.h as no > longer used. > > Signed-off-by: Zixun LI <z...@ogga.fr> Checkpatch complains here: checkpatch.pl: 164: WARNING: From:/Signed-off-by: email address mismatch: 'From: Zixun LI <ad...@hifiphile.com>' != 'Signed-off-by: Zixun LI <z...@ogga.fr>' Please make sure that the commiter matches the Signed-off-by. This can be done with git commit --reset-author or --author="name <email>" > --- > drivers/usb/gadget/atmel_usba_udc.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/usb/gadget/atmel_usba_udc.c > b/drivers/usb/gadget/atmel_usba_udc.c > index 5f78251fdb..83fdc36870 100644 > --- a/drivers/usb/gadget/atmel_usba_udc.c > +++ b/drivers/usb/gadget/atmel_usba_udc.c > @@ -7,13 +7,13 @@ > * Bo Shen <voice.s...@atmel.com> > */ > > +#include <log.h> > #include <malloc.h> > #include <asm/gpio.h> > #include <asm/hardware.h> > #include <linux/bitops.h> > #include <linux/errno.h> > #include <linux/list.h> > -#include <linux/printk.h> > #include <linux/usb/ch9.h> > #include <linux/usb/gadget.h> > #include <linux/usb/atmel_usba_udc.h> > @@ -1204,12 +1204,12 @@ int usb_gadget_register_driver(struct > usb_gadget_driver *driver) > int ret; > > if (!driver || !driver->bind || !driver->setup) { > - printf("bad paramter\n"); > + log_err("bad paramter\n"); > return -EINVAL; > } > > if (udc->driver) { > - printf("UDC already has a gadget driver\n"); > + log_err("UDC already has a gadget driver\n"); > return -EBUSY; > } > > @@ -1219,7 +1219,7 @@ int usb_gadget_register_driver(struct usb_gadget_driver > *driver) > > ret = driver->bind(&udc->gadget); > if (ret) { > - pr_err("driver->bind() returned %d\n", ret); > + log_err("driver->bind() returned %d\n", ret); > udc->driver = NULL; > } > > @@ -1231,7 +1231,7 @@ int usb_gadget_unregister_driver(struct > usb_gadget_driver *driver) > struct usba_udc *udc = &controller; > > if (!driver || !driver->unbind || !driver->disconnect) { > - pr_err("bad paramter\n"); > + log_err("bad parameter\n"); Here we change both: 1. pr_err() to log_err() 2. fix paramter -> parameter 2. should be done in patch 3/7 instead of here. With the above fixed: Reviewed-by: Mattijs Korpershoek <mkorpersh...@baylibre.com> > return -EINVAL; > } > > @@ -1252,7 +1252,7 @@ static struct usba_ep *usba_udc_pdata(struct > usba_platform_data *pdata, > > eps = malloc(sizeof(struct usba_ep) * pdata->num_ep); > if (!eps) { > - pr_err("failed to alloc eps\n"); > + log_err("failed to alloc eps\n"); > return NULL; > } > > -- > 2.45.2