netdev - RE: [PATCH v2 2/2] netdev: driver: ethernet: Add TI CPSW driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <EB1619762EAF8B4E97A227FB77B7E0293178F468@DBDE01.ent.ti.com>
Date:	Thu, 23 Feb 2012 12:47:55 +0000
From:	"N, Mugunthan V" <mugunthanvnm@...com>
To:	Joe Perches <joe@...ches.com>,
	"davem@...emloft.net" <davem@...emloft.net>
CC:	"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [PATCH v2 2/2] netdev: driver: ethernet: Add TI CPSW driver

Joe/David

> -----Original Message-----
> From: N, Mugunthan V
> Sent: Wednesday, February 22, 2012 10:13 PM
> To: 'Joe Perches'
> Cc: netdev@...r.kernel.org; davem@...emloft.net
> Subject: RE: [PATCH v2 2/2] netdev: driver: ethernet: Add TI CPSW driver
> 
> Joe
> 
> > -----Original Message-----
> > From: Joe Perches [mailto:joe@...ches.com]
> > Sent: Tuesday, February 21, 2012 4:34 AM
[...]
> > Add
> > #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> > before all #includes and remove the embedded prefix
> > from all pr_<level> uses.
> 
> Agreed, will make use of pr_fmt and remove embedded prefix in pr_<level>
> uses
Adding 
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
in the driver gives a checkpatch error as follows

#77: FILE: drivers/net/ethernet/ti/cpsw.c:16:
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt

total: 1 errors, 0 warnings, 1137 lines checked

Is it ok to ignore the checkpatch error as most of the drivers use it.

[...] 

With regards
Mugunthan V N

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html