From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.free-electrons.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.89 #1 (Red Hat Linux)) id 1enmn4-0008QJ-1R for linux-mtd@lists.infradead.org; Mon, 19 Feb 2018 14:51:40 +0000 Date: Mon, 19 Feb 2018 15:51:15 +0100 From: Boris Brezillon To: Shreeya Patel Cc: boris.brezillon@free-electrons.com, richard@nod.at, dwmw2@infradead.org, computersforpeace@gmail.com, marek.vasut@gmail.com, cyrille.pitchen@wedev4u.fr, maximlevitsky@gmail.com, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, ezequiel@vanguardiasur.com.ar, outreachy-kernel@googlegroups.com Subject: Re: [PATCH NAND v2] mtd: nand: Replace printk() with appropriate pr_*macro() Message-ID: <20180219155115.3d9d1bd5@bbrezillon> In-Reply-To: <1519046025-9412-1-git-send-email-shreeya.patel23498@gmail.com> References: <1519046025-9412-1-git-send-email-shreeya.patel23498@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Shreeya, On Mon, 19 Feb 2018 18:43:45 +0530 Shreeya Patel 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 > --- > > 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