On 22 August 2012 11:43, ABRAHAM, KISHON VIJAY <kishon@xxxxxx> wrote: > Hi, > > On Wed, Aug 22, 2012 at 11:13 AM, Sachin Kamat <sachin.kamat@xxxxxxxxxx> wrote: >> Replace printk with corresponding pr_* functions. >> >> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> >> --- >> 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html