lkml.org 
[lkml]   [2010]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/3] Staging: rt2860: include KERN_* in printk
On Sun, Dec 12, 2010 at 11:04:28AM -0800, Joe Perches wrote:
> On Sun, 2010-12-12 at 18:56 +0100, L. Alberto Giménez wrote:
> > Fix checkpatch complains
> > diff --git a/drivers/staging/rt2860/common/ee_efuse.c b/drivers/staging/rt2860/common/ee_efuse.c
> []
> > @@ -281,8 +281,8 @@ int set_eFusedump_Proc(struct rt_rtmp_adapter *pAd, char *arg)
> >
> > eFuseReadPhysical(pAd, &InBuf[0], 4, &InBuf[2], 2);
> > if (i % 4 == 0)
> > - printk("\nBlock %x:", i / 8);
> > - printk("%04x ", InBuf[2]);
> > + printk(KERN_DEBUG "\nBlock %x:", i / 8);
> > + printk(KERN_DEBUG "%04x ", InBuf[2]);
> > }
> > return TRUE;
> > }
>
> Not quite. Use:
>
> (before the for)
> printk(KERN_DEBUG "Block 0: ");
>
> for(...)
> []
> if (i && i % 4 == 0) {
> printk(KERN_CONT "\n");
> printk(KERN_DEBUG "Block %x: ", i / 8);
> }
> printk(KERN_CONT " %04x", InBuf[2]);
> }
> printk(KERN_CONT "\n");
>

This is dead code anyway. Just remove the whole set_eFusedump_Proc()
function.

regards,
dan carpenter
--
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/

\
 
 \ /
  Last update: 2010-12-13 07:01    [W:1.133 / U:0.184 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site