lkml.org 
[lkml]   [2018]   [Feb]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH NAND v2] mtd: nand: Replace printk() with appropriate pr_*macro()
Oh, and in the subject 'pr_*macro()' -> 'pr_*() macro'

On Mon, 19 Feb 2018 15:51:15 +0100
Boris Brezillon <boris.brezillon@bootlin.com> wrote:

> Hi Shreeya,
>
> On Mon, 19 Feb 2018 18:43:45 +0530
> Shreeya Patel <shreeya.patel23498@gmail.com> wrote:
>
> > The log levels embedded with the name are more concise than
> > printk.
> > Replace printks having a log level with the appropriate
> > pr_*macro.
> >
> > Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
> > ---
> >
> > Changes in v2:
> > -Merge previous patches of the patchset regarding replacement
> > of printk with pr_*macro, into single patch.
> >
> >
> > drivers/mtd/nand/cs553x_nand.c | 9 ++---
> > drivers/mtd/nand/diskonchip.c | 76 +++++++++++++++++++++-------------------
> > drivers/mtd/nand/fsl_elbc_nand.c | 2 +-
> > drivers/mtd/nand/fsl_ifc_nand.c | 2 +-
> > drivers/mtd/nand/mxc_nand.c | 2 +-
> > drivers/mtd/nand/nand_bch.c | 12 +++----
> > drivers/mtd/nand/nandsim.c | 10 +++---
> > drivers/mtd/nand/r852.c | 2 +-
> > drivers/mtd/nand/r852.h | 6 ++--
> > drivers/mtd/nand/sm_common.c | 5 ++-
> > 10 files changed, 65 insertions(+), 61 deletions(-)
> >
>
> [...]
>
> >
> > diff --git a/drivers/mtd/nand/diskonchip.c b/drivers/mtd/nand/diskonchip.c
> > index c3aa53c..b97d88c 100644
> > --- a/drivers/mtd/nand/diskonchip.c
> > +++ b/drivers/mtd/nand/diskonchip.c
>
> [...]
>
> > @@ -438,7 +438,7 @@ static void __init doc2000_count_chips(struct mtd_info *mtd)
> > break;
> > }
> > doc->chips_per_floor = i;
> > - printk(KERN_DEBUG "Detected %d chips per floor.\n", i);
> > + pr_info("Detected %d chips per floor.\n", i);
>
> Should be pr_debug() here.
>
> > }
> >
>
> [...]
>
> > diff --git a/drivers/mtd/nand/nandsim.c b/drivers/mtd/nand/nandsim.c
> > index 246b439..4e5f817 100644
> > --- a/drivers/mtd/nand/nandsim.c
> > +++ b/drivers/mtd/nand/nandsim.c
> > @@ -184,15 +184,15 @@ MODULE_PARM_DESC(bch, "Enable BCH ecc and set how many bits should "
> >
> > /* Simulator's output macros (logging, debugging, warning, error) */
> > #define NS_LOG(args...) \
> > - do { if (log) printk(KERN_DEBUG NS_OUTPUT_PREFIX " log: " args); } while(0)
> > + do { if (log) pr_debug(NS_OUTPUT_PREFIX " log: " args); } while(0)
>
> You could define pr_fmt() to avoid passing NS_OUTPUT_PREFIX. Something
> like:
>
> #define pr_fmt(fmt) "[nandsim]" fmt
>
> (remember to put this definition before include directives).
>
> Then, all you have to do is
>
> do { if (log) pr_debug(" log: " args); } while(0)
>
> > #define NS_DBG(args...) \
> > - do { if (dbg) printk(KERN_DEBUG NS_OUTPUT_PREFIX " debug: " args); } while(0)
> > + do { if (dbg) pr_debug(NS_OUTPUT_PREFIX " debug: " args); } while(0)
> > #define NS_WARN(args...) \
> > - do { printk(KERN_WARNING NS_OUTPUT_PREFIX " warning: " args); } while(0)
> > + do { pr_warn(NS_OUTPUT_PREFIX " warning: " args); } while(0)
> > #define NS_ERR(args...) \
> > - do { printk(KERN_ERR NS_OUTPUT_PREFIX " error: " args); } while(0)
> > + do { pr_err(NS_OUTPUT_PREFIX " error: " args); } while(0)
> > #define NS_INFO(args...) \
> > - do { printk(KERN_INFO NS_OUTPUT_PREFIX " " args); } while(0)
> > + do { pr_info(NS_OUTPUT_PREFIX " " args); } while(0)
> >
> > /* Busy-wait delay macros (microseconds, milliseconds) */
> > #define NS_UDELAY(us) \
> > diff --git a/drivers/mtd/nand/r852.c b/drivers/mtd/nand/r852.c
> > index fc9287a..3d54c6a 100644
> > --- a/drivers/mtd/nand/r852.c
> > +++ b/drivers/mtd/nand/r852.c
> > @@ -935,7 +935,7 @@ static int r852_probe(struct pci_dev *pci_dev, const struct pci_device_id *id)
> > &dev->card_detect_work, 0);
> >
> >
> > - printk(KERN_NOTICE DRV_NAME ": driver loaded successfully\n");
> > + pr_notice(DRV_NAME ": driver loaded successfully\n");
>
> Same here:
>
> #define pr_fmt(fmt) DRV_NAME fmt
>
> > return 0;
> >
> > error10:
> > diff --git a/drivers/mtd/nand/r852.h b/drivers/mtd/nand/r852.h
> > index 8713c57..63de9b6 100644
> > --- a/drivers/mtd/nand/r852.h
> > +++ b/drivers/mtd/nand/r852.h
> > @@ -149,12 +149,12 @@ struct r852_device {
> >
> > #define dbg(format, ...) \
> > if (debug) \
> > - printk(KERN_DEBUG DRV_NAME ": " format "\n", ## __VA_ARGS__)
> > + pr_debug(DRV_NAME ": " format "\n", ## __VA_ARGS__)
> >
> > #define dbg_verbose(format, ...) \
> > if (debug > 1) \
> > - printk(KERN_DEBUG DRV_NAME ": " format "\n", ## __VA_ARGS__)
> > + pr_debug(DRV_NAME ": " format "\n", ## __VA_ARGS__)
> >
>
> And here as well.
>
> Regards,
>
> Boris
>



--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

\
 
 \ /
  Last update: 2018-02-19 15:55    [W:0.058 / U:0.312 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site