From: Roel Kluin <12o3l@tiscali.nl> Subject: Re: [PATCH] [Coding Style]: fs/ext{3,4}/ext{3,4}_jbd{,2}.c Date: Fri, 11 Jan 2008 10:45:30 +0100 Message-ID: <47873ABA.5050604@tiscali.nl> References: <1199452896-20145-1-git-send-email-mathieu.segaud@regala.cx> <1199452896-20145-2-git-send-email-mathieu.segaud@regala.cx> <1199452896-20145-3-git-send-email-mathieu.segaud@regala.cx> <1199452896-20145-4-git-send-email-mathieu.segaud@regala.cx> <1199452896-20145-5-git-send-email-mathieu.segaud@regala.cx> <477E379F.4000103@student.ltu.se> <20080105041228.GP3351@webber.adilger.int> <20080105051817.GD27894@ZenIV.linux.org.uk> <4786883E.30604@tiscali.nl> <20080111030945.27954.qmail@cdy.org> <20080111034240.GA29861@linux-sh.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit To: Paul Mundt , Roel Kluin <12o3l@tiscali.nl>, Al Viro , Mathieu Segaud , Richard Knutsson Received: from smtp-out0.tiscali.nl ([195.241.79.175]:37060 "EHLO smtp-out0.tiscali.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755239AbYAKJpp (ORCPT ); Fri, 11 Jan 2008 04:45:45 -0500 In-Reply-To: <20080111034240.GA29861@linux-sh.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: Paul Mundt wrote: > On Fri, Jan 11, 2008 at 04:09:45AM +0100, Peter Stuge wrote: >> On Thu, Jan 10, 2008 at 10:03:58PM +0100, Roel Kluin wrote: >>> -#define DEBUG(x,args...) printk(__FUNCTION__ ": " x,##args) >>> +#define DEBUG(x, args...) printk("%s: ", __func__, x, ##args) >> Can this really be expected to work when x contains conversions? >> >> How about: >> >> #define DEBUG(x, args...) printk("%s: " x, __func__, ##args) >> > How about throwing out hand-rolled debug printk wrappers for the > brain-damage they are and using the ones the kernel provides instead? > Should it be done like this? -- Replace printk wrapper - with a syntax error - by pr_debug Signed-off-by: Roel Kluin <12o3l@tiscali.nl> --- diff --git a/drivers/pcmcia/au1000_xxs1500.c b/drivers/pcmcia/au1000_xxs1500.c index ce9d5c4..4c32389 100644 --- a/drivers/pcmcia/au1000_xxs1500.c +++ b/drivers/pcmcia/au1000_xxs1500.c @@ -25,6 +25,10 @@ * * */ +#ifdef CONFIG_MIPS_XXS1500_DEBUG +#define DEBUG 1 +#endif + #include #include #include @@ -55,11 +59,6 @@ #define PCMCIA_NUM_SOCKS (PCMCIA_MAX_SOCK + 1) #define PCMCIA_IRQ AU1000_GPIO_4 -#if 0 -#define DEBUG(x,args...) printk(__FUNCTION__ ": " x,##args) -#else -#define DEBUG(x,args...) -#endif static int xxs1500_pcmcia_init(struct pcmcia_init *init) { @@ -143,13 +142,13 @@ xxs1500_pcmcia_configure_socket(const struct pcmcia_configure *configure) if(configure->sock > PCMCIA_MAX_SOCK) return -1; - DEBUG("Vcc %dV Vpp %dV, reset %d\n", + pr_debug("Vcc %dV Vpp %dV, reset %d\n", configure->vcc, configure->vpp, configure->reset); switch(configure->vcc){ case 33: /* Vcc 3.3V */ /* turn on power */ - DEBUG("turn on power\n"); + pr_debug("turn on power\n"); au_writel((au_readl(GPIO2_PINSTATE) & ~(1<<14))|(1<<30), GPIO2_OUTPUT); au_sync_delay(100); @@ -166,7 +165,7 @@ xxs1500_pcmcia_configure_socket(const struct pcmcia_configure *configure) } if (!configure->reset) { - DEBUG("deassert reset\n"); + pr_debug("deassert reset\n"); au_writel((au_readl(GPIO2_PINSTATE) & ~(1<<4))|(1<<20), GPIO2_OUTPUT); au_sync_delay(100); @@ -174,7 +173,7 @@ xxs1500_pcmcia_configure_socket(const struct pcmcia_configure *configure) GPIO2_OUTPUT); } else { - DEBUG("assert reset\n"); + pr_debug("assert reset\n"); au_writel(au_readl(GPIO2_PINSTATE) | (1<<4)|(1<<20), GPIO2_OUTPUT); }