diff options
author | Steven Rostedt <rostedt@goodmis.org> | 2014-06-12 19:44:20 -0400 |
---|---|---|
committer | Jiri Olsa <jolsa@kernel.org> | 2014-06-19 12:18:36 -0400 |
commit | 24eda087ccbb32f556cb3f9f78be152312bf6cc4 (patch) | |
tree | 3c72cb5fe658fbb66575672625a7e35b25fb584c /tools/lib | |
parent | a21e3a34bef9e9c177be61111ced23e71e25fd8a (diff) |
tools lib traceevent: Clean up format of args in cfg80211 plugin
While synchronizing what's in trace-cmd vs what's in perf, I came
across a change that was made when entering the cfg80211 plugin into
the tools/lib/traceevent directory. The function prototype went from:
static unsigned long long process___le16_to_cpup(struct trace_seq *s,
unsigned long long *args)
to:
static unsigned long long
process___le16_to_cpup(struct trace_seq *s,
unsigned long long *args)
I can understand the line break after the long long, but there's no
reason to keep args on a separate line.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Link: http://lkml.kernel.org/r/20140612194420.24073744@gandalf.local.home
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Diffstat (limited to 'tools/lib')
-rw-r--r-- | tools/lib/traceevent/plugin_cfg80211.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tools/lib/traceevent/plugin_cfg80211.c b/tools/lib/traceevent/plugin_cfg80211.c index c066b25905f8..4592d8438318 100644 --- a/tools/lib/traceevent/plugin_cfg80211.c +++ b/tools/lib/traceevent/plugin_cfg80211.c | |||
@@ -5,8 +5,7 @@ | |||
5 | #include "event-parse.h" | 5 | #include "event-parse.h" |
6 | 6 | ||
7 | static unsigned long long | 7 | static unsigned long long |
8 | process___le16_to_cpup(struct trace_seq *s, | 8 | process___le16_to_cpup(struct trace_seq *s, unsigned long long *args) |
9 | unsigned long long *args) | ||
10 | { | 9 | { |
11 | uint16_t *val = (uint16_t *) (unsigned long) args[0]; | 10 | uint16_t *val = (uint16_t *) (unsigned long) args[0]; |
12 | return val ? (long long) le16toh(*val) : 0; | 11 | return val ? (long long) le16toh(*val) : 0; |