> -----Original Message----- > From: Joe Perches [mailto:j...@perches.com] > Sent: Wednesday, June 19, 2013 6:42 AM > To: Kirsher, Jeffrey T > Cc: a...@canonical.com; Keller, Jacob E; linux-kernel@vger.kernel.org; > net...@vger.kernel.org > Subject: Re: [net] checkpatch: modify logFunction regular expression to > allow repeated front tags > > On Wed, 2013-06-19 at 03:38 -0700, Jeff Kirsher wrote: > > From: Jacob Keller <jacob.e.kel...@intel.com> > > > > The previous logFunction regular expression allowed names for log > functions like > > dev_warn or e_dbg and so forth, but some log functions are of the > forms > > similar to e_dev_warn, (adding an additional tag in front of a standard > name > > like dev_warn). > > Perhaps names like foo_bar_baz_dev_warn are a bit long. > > Maybe instead of > + (?:[a-z0- > 9]+_)+(?:printk|emerg|alert|crit|err|warning|warn|notice|info|debug > |dbg|vdbg|devel|cont|WARN)(?:_ratelimited|_once|)| > use > + (?:[a-z0- > 9]+_){0,2}(?:printk|emerg|alert|crit|err|warning|warn|notice|info|de > bug|dbg|vdbg|devel|cont|WARN)(?:_ratelimited|_once|)| >
That sounds good to me. - Jake -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
