Messages in this thread | | | Date | Tue, 30 Mar 2010 13:38:42 -0700 | From | Greg KH <> | Subject | Re: [PATCH] drivers:staging: sources for ST core |
| |
On Tue, Mar 30, 2010 at 09:23:23PM +0530, Pavan Savoy wrote: > Alan, > > --- On Tue, 30/3/10, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote: > > > From: Alan Cox <alan@lxorguk.ukuu.org.uk> > > Subject: Re: [PATCH] drivers:staging: sources for ST core > > To: pavan_savoy@ti.com > > Cc: marcel@holtmann.org, gregkh@suse.de, linux-kernel@vger.kernel.org, pavan_savoy@yahoo.co.in > > Date: Tuesday, 30 March, 2010, 4:52 PM > > > +/* all debug macros go in here > > */ > > > +#define ST_DRV_ERR(fmt, arg...)? printk(KERN_ERR > > "(stc):"fmt"\n" , ## arg) > > > +#if defined(DEBUG)??? > > ??? /* limited debug messages */ > > > +#define ST_DRV_DBG(fmt, arg...)? > > printk(KERN_INFO "(stc):"fmt"\n" , ## arg) > > > +#define ST_DRV_VER(fmt, arg...) > > > +#elif defined(VERBOSE)??? > > ??? /* very verbose */ > > > +#define ST_DRV_DBG(fmt, arg...)? > > printk(KERN_INFO "(stc):"fmt"\n" , ## arg) > > > +#define ST_DRV_VER(fmt, arg...)? > > printk(KERN_INFO "(stc):"fmt"\n" , ## arg) > > > +#else /* error msgs only */ > > > +#define ST_DRV_DBG(fmt, arg...) > > > +#define ST_DRV_VER(fmt, arg...) > > > +#endif > > > > As Greg said earlier - needs to be using the standard debug > > macros > > Agree - It's all there because of the organization's coding standards. > Will correct it.
Might you also want to correct your organization's coding standards to follow the correct Linux kernel ones? That way you will not have this problem in the future.
thanks,
greg k-h
| |