Re: [PATCH 2/5] usb: gadget: s3c2410_udc: Use pr_* functions
Hi, On Wed, Aug 22, 2012 at 11:13 AM, Sachin Kamat wrote: > Replace printk with corresponding pr_* functions. > > Signed-off-by: Sachin Kamat > --- > drivers/usb/gadget/s3c2410_udc.c | 12 +++- > 1 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/drivers/usb/gadget/s3c2410_udc.c > b/drivers/usb/gadget/s3c2410_udc.c > index 7acecc0..7e2ce44 100644 > --- a/drivers/usb/gadget/s3c2410_udc.c > +++ b/drivers/usb/gadget/s3c2410_udc.c > @@ -12,6 +12,8 @@ > * (at your option) any later version. > */ > > +#define pr_fmt(fmt) "s3c2410_udc: " fmt > + Is this a stray change? I dont see pr_fmt being used anywhere in this patch.. Thanks Kishon -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/5] usb: gadget: s3c2410_udc: Use pr_* functions
On 22 August 2012 11:43, ABRAHAM, KISHON VIJAY wrote: > Hi, > > On Wed, Aug 22, 2012 at 11:13 AM, Sachin Kamat > wrote: >> Replace printk with corresponding pr_* functions. >> >> Signed-off-by: Sachin Kamat >> --- >> drivers/usb/gadget/s3c2410_udc.c | 12 +++- >> 1 files changed, 7 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/usb/gadget/s3c2410_udc.c >> b/drivers/usb/gadget/s3c2410_udc.c >> index 7acecc0..7e2ce44 100644 >> --- a/drivers/usb/gadget/s3c2410_udc.c >> +++ b/drivers/usb/gadget/s3c2410_udc.c >> @@ -12,6 +12,8 @@ >> * (at your option) any later version. >> */ >> >> +#define pr_fmt(fmt) "s3c2410_udc: " fmt >> + > > Is this a stray change? I dont see pr_fmt being used anywhere in this patch.. No. pr_fmt prepends the string (in the above case "s3c2410_udc: ") to the fmt string provided by other pr_* functions. For example, if i have pr_err("error") in this file, then the output will be: s3c2410_udc: error Hope I was able to clarify your doubt. > > Thanks > Kishon -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/5] usb: gadget: s3c2410_udc: Use pr_* functions
Hi, On Wed, Aug 22, 2012 at 11:13:16AM +0530, Sachin Kamat wrote: > Replace printk with corresponding pr_* functions. > > Signed-off-by: Sachin Kamat > --- > drivers/usb/gadget/s3c2410_udc.c | 12 +++- > 1 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/drivers/usb/gadget/s3c2410_udc.c > b/drivers/usb/gadget/s3c2410_udc.c > index 7acecc0..7e2ce44 100644 > --- a/drivers/usb/gadget/s3c2410_udc.c > +++ b/drivers/usb/gadget/s3c2410_udc.c > @@ -12,6 +12,8 @@ > * (at your option) any later version. > */ > > +#define pr_fmt(fmt) "s3c2410_udc: " fmt > + > #include > #include > #include > @@ -115,7 +117,7 @@ static int dprintk(int level, const char *fmt, ...) > sizeof(printk_buf)-len, fmt, args); > va_end(args); > > - return printk(KERN_DEBUG "%s", printk_buf); > + return pr_debug("%s", printk_buf); > } > #else > static int dprintk(int level, const char *fmt, ...) > @@ -1683,13 +1685,13 @@ static int s3c2410_udc_start(struct usb_gadget_driver > *driver, > return -EBUSY; > > if (!bind || !driver->setup || driver->max_speed < USB_SPEED_FULL) { > - printk(KERN_ERR "Invalid driver: bind %p setup %p speed %d\n", > + pr_err("Invalid driver: bind %p setup %p speed %d\n", > bind, driver->setup, driver->max_speed); you have access to a struct device *. Please use dev_* instead. -- balbi signature.asc Description: Digital signature
Re: [PATCH 2/5] usb: gadget: s3c2410_udc: Use pr_* functions
On Wed, Aug 22, 2012 at 11:50 AM, Sachin Kamat wrote: > On 22 August 2012 11:43, ABRAHAM, KISHON VIJAY wrote: >> Hi, >> >> On Wed, Aug 22, 2012 at 11:13 AM, Sachin Kamat >> wrote: >>> Replace printk with corresponding pr_* functions. >>> >>> Signed-off-by: Sachin Kamat >>> --- >>> drivers/usb/gadget/s3c2410_udc.c | 12 +++- >>> 1 files changed, 7 insertions(+), 5 deletions(-) >>> >>> diff --git a/drivers/usb/gadget/s3c2410_udc.c >>> b/drivers/usb/gadget/s3c2410_udc.c >>> index 7acecc0..7e2ce44 100644 >>> --- a/drivers/usb/gadget/s3c2410_udc.c >>> +++ b/drivers/usb/gadget/s3c2410_udc.c >>> @@ -12,6 +12,8 @@ >>> * (at your option) any later version. >>> */ >>> >>> +#define pr_fmt(fmt) "s3c2410_udc: " fmt >>> + >> >> Is this a stray change? I dont see pr_fmt being used anywhere in this patch.. > > No. > pr_fmt prepends the string (in the above case "s3c2410_udc: ") to the > fmt string provided by other pr_* functions. > For example, > if i have pr_err("error") in this file, > then the output will be: s3c2410_udc: error Ah.. got it.. Thanks Kishon -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/5] usb: gadget: s3c2410_udc: Use pr_* functions
On 22 August 2012 11:48, Felipe Balbi wrote: > Hi, > > On Wed, Aug 22, 2012 at 11:13:16AM +0530, Sachin Kamat wrote: >> Replace printk with corresponding pr_* functions. >> >> Signed-off-by: Sachin Kamat >> --- >> drivers/usb/gadget/s3c2410_udc.c | 12 +++- >> 1 files changed, 7 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/usb/gadget/s3c2410_udc.c >> b/drivers/usb/gadget/s3c2410_udc.c >> index 7acecc0..7e2ce44 100644 >> --- a/drivers/usb/gadget/s3c2410_udc.c >> +++ b/drivers/usb/gadget/s3c2410_udc.c >> @@ -12,6 +12,8 @@ >> * (at your option) any later version. >> */ >> >> +#define pr_fmt(fmt) "s3c2410_udc: " fmt >> + >> #include >> #include >> #include >> @@ -115,7 +117,7 @@ static int dprintk(int level, const char *fmt, ...) >> sizeof(printk_buf)-len, fmt, args); >> va_end(args); >> >> - return printk(KERN_DEBUG "%s", printk_buf); >> + return pr_debug("%s", printk_buf); >> } >> #else >> static int dprintk(int level, const char *fmt, ...) >> @@ -1683,13 +1685,13 @@ static int s3c2410_udc_start(struct >> usb_gadget_driver *driver, >> return -EBUSY; >> >> if (!bind || !driver->setup || driver->max_speed < USB_SPEED_FULL) { >> - printk(KERN_ERR "Invalid driver: bind %p setup %p speed %d\n", >> + pr_err("Invalid driver: bind %p setup %p speed %d\n", >> bind, driver->setup, driver->max_speed); > > you have access to a struct device *. Please use dev_* instead. Ok. I will re-send this one after changing to dev_err. > > -- > balbi -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html