* checkpatch.pl and assembly #defines
@ 2012-12-04 12:59 Paul E. McKenney
2012-12-04 20:44 ` Andy Whitcroft
0 siblings, 1 reply; 2+ messages in thread
From: Paul E. McKenney @ 2012-12-04 12:59 UTC (permalink / raw)
To: apw; +Cc: linux-kernel
Hello, Andy,
The current checkpatch.pl complains about cpp macros defined as follows:
#define callit call my_function
Of course, if you do put parentheses around the definition, your assembler
will complain. Could this complaint be downgraded to a warning? Or is
there some way to recognize assembly cpp macros?
Thanx, Paul
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: checkpatch.pl and assembly #defines
2012-12-04 12:59 checkpatch.pl and assembly #defines Paul E. McKenney
@ 2012-12-04 20:44 ` Andy Whitcroft
0 siblings, 0 replies; 2+ messages in thread
From: Andy Whitcroft @ 2012-12-04 20:44 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: linux-kernel
On Tue, Dec 04, 2012 at 04:59:23AM -0800, Paul E. McKenney wrote:
> Hello, Andy,
>
> The current checkpatch.pl complains about cpp macros defined as follows:
>
> #define callit call my_function
>
> Of course, if you do put parentheses around the definition, your assembler
> will complain. Could this complaint be downgraded to a warning? Or is
> there some way to recognize assembly cpp macros?
Well all checkpatch output is warnings anyhow. Where are these being
placed, perhaps there is a clue in that at least.
-apw
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-12-04 20:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-04 12:59 checkpatch.pl and assembly #defines Paul E. McKenney
2012-12-04 20:44 ` Andy Whitcroft
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).