Re: Checkpatch error on trace events macros

2013-07-29 Thread Li Zefan
On 2013/7/30 11:10, Joe Perches wrote: > On Tue, 2013-07-30 at 11:04 +0800, Li Zefan wrote: >> On 2013/7/30 10:36, Joe Perches wrote: >>> On Tue, 2013-07-30 at 10:06 +0800, Li Zefan wrote: On 2013/7/30 9:58, Joe Perches wrote: > So what are these TRACE_ defines that need > excluding fr

Re: Checkpatch error on trace events macros

2013-07-29 Thread Joe Perches
On Tue, 2013-07-30 at 11:04 +0800, Li Zefan wrote: > On 2013/7/30 10:36, Joe Perches wrote: > > On Tue, 2013-07-30 at 10:06 +0800, Li Zefan wrote: > >> On 2013/7/30 9:58, Joe Perches wrote: > >>> So what are these TRACE_ defines that need > >>> excluding from the "complex values" check? > >>> > >>>

Re: Checkpatch error on trace events macros

2013-07-29 Thread Li Zefan
On 2013/7/30 10:36, Joe Perches wrote: > On Tue, 2013-07-30 at 10:06 +0800, Li Zefan wrote: >> On 2013/7/30 9:58, Joe Perches wrote: >>> So what are these TRACE_ defines that need >>> excluding from the "complex values" check? >>> >>> Anything other than >>> >>> TRACE_SYSTEM >>> TRACE_INCLUDE_FILE

Re: Checkpatch error on trace events macros

2013-07-29 Thread Joe Perches
On Tue, 2013-07-30 at 10:06 +0800, Li Zefan wrote: > On 2013/7/30 9:58, Joe Perches wrote: > > So what are these TRACE_ defines that need > > excluding from the "complex values" check? > > > > Anything other than > > > > TRACE_SYSTEM > > TRACE_INCLUDE_FILE > > TRACE_INCLUDE_PATH > > > > ? > >

Re: Checkpatch error on trace events macros

2013-07-29 Thread Li Zefan
On 2013/7/30 9:58, Joe Perches wrote: > On Tue, 2013-07-30 at 09:30 +0800, Li Zefan wrote: >> On 2013/7/30 3:52, Sarah Sharp wrote: >>> Hi Andy and Joe, >>> >>> Checkpatch is complaining when code adds new trace events macros: >>> >>> sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/

Re: Checkpatch error on trace events macros

2013-07-29 Thread Joe Perches
On Tue, 2013-07-30 at 09:30 +0800, Li Zefan wrote: > On 2013/7/30 3:52, Sarah Sharp wrote: > > Hi Andy and Joe, > > > > Checkpatch is complaining when code adds new trace events macros: > > > > sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/.to-apply > > Applying: xhci: add trace

Re: Checkpatch error on trace events macros

2013-07-29 Thread Li Zefan
On 2013/7/30 3:52, Sarah Sharp wrote: > Hi Andy and Joe, > > Checkpatch is complaining when code adds new trace events macros: > > sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/.to-apply > Applying: xhci: add traces for debug messages in xhci_address_device() > ERROR: Macros wit

Re: Checkpatch error on trace events macros

2013-07-29 Thread Joe Perches
On Mon, 2013-07-29 at 14:23 -0700, Sarah Sharp wrote: > On Mon, Jul 29, 2013 at 01:02:44PM -0700, Joe Perches wrote: > > On Mon, 2013-07-29 at 12:52 -0700, Sarah Sharp wrote: [] > > > sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/.to-apply > > > Applying: xhci: add traces for debu

Re: Checkpatch error on trace events macros

2013-07-29 Thread Sarah Sharp
On Mon, Jul 29, 2013 at 01:02:44PM -0700, Joe Perches wrote: > On Mon, 2013-07-29 at 12:52 -0700, Sarah Sharp wrote: > > Hi Andy and Joe, > > Hi Sarah. > > > Checkpatch is complaining when code adds new trace events macros: > > > > sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/

Re: Checkpatch error on trace events macros

2013-07-29 Thread Joe Perches
On Mon, 2013-07-29 at 12:52 -0700, Sarah Sharp wrote: > Hi Andy and Joe, Hi Sarah. > Checkpatch is complaining when code adds new trace events macros: > > sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/.to-apply > Applying: xhci: add traces for debug messages in xhci_address_dev

Checkpatch error on trace events macros

2013-07-29 Thread Sarah Sharp
Hi Andy and Joe, Checkpatch is complaining when code adds new trace events macros: sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/.to-apply Applying: xhci: add traces for debug messages in xhci_address_device() ERROR: Macros with complex values should be enclosed in parenthesis #