Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752205Ab2FRGIb (ORCPT ); Mon, 18 Jun 2012 02:08:31 -0400 Received: from linux-sh.org ([111.68.239.195]:58798 "EHLO linux-sh.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751579Ab2FRGI3 (ORCPT ); Mon, 18 Jun 2012 02:08:29 -0400 Date: Mon, 18 Jun 2012 15:08:19 +0900 From: Paul Mundt To: Joe Perches Cc: Greg Kroah-Hartman , Andrew Morton , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/5] usb: Convert dbg to pr_eliminated and pr_debug Message-ID: <20120618060818.GA10225@linux-sh.org> References: <25071de4def201f947780d6937b9732ccac3b3f0.1339985627.git.joe@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <25071de4def201f947780d6937b9732ccac3b3f0.1339985627.git.joe@perches.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1597 Lines: 42 On Sun, Jun 17, 2012 at 07:25:10PM -0700, Joe Perches wrote: > Change the dbg macro to use pr_eliminated when not > DEBUG and pr_debug with DEBUG so dynamic_debug can > be used. > > Signed-off-by: Joe Perches > --- > include/linux/usb.h | 12 +++++------- > 1 files changed, 5 insertions(+), 7 deletions(-) > > diff --git a/include/linux/usb.h b/include/linux/usb.h > index f717fbd..56b88b8 100644 > --- a/include/linux/usb.h > +++ b/include/linux/usb.h > @@ -1714,14 +1714,12 @@ extern void usb_register_notify(struct notifier_block *nb); > extern void usb_unregister_notify(struct notifier_block *nb); > > #ifdef DEBUG > -#define dbg(format, arg...) \ > - printk(KERN_DEBUG "%s: " format "\n", __FILE__, ##arg) > +#define dbg(format, ...) \ > + pr_debug("%s: " format "\n", __FILE__, ##__VA_ARGS__) > #else > -#define dbg(format, arg...) \ > -do { \ > - if (0) \ > - printk(KERN_DEBUG "%s: " format "\n", __FILE__, ##arg); \ > -} while (0) > +#define dbg(format, ...) \ > + printk_eliminated(KERN_DEBUG "%s: " format "\n", \ > + __FILE__, ##__VA_ARGS) > #endif > Either you didn't test this or your API needs to be punched in the face. The interface you just added is eliminated_printk(), I sure hope the above is a typo and you don't actually also have a printk_eliminated(). -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/