diff options
| author | Steven Rostedt <rostedt@goodmis.org> | 2014-06-13 03:44:20 +0400 |
|---|---|---|
| committer | Jiri Olsa <jolsa@kernel.org> | 2014-06-19 20:18:36 +0400 |
| commit | 24eda087ccbb32f556cb3f9f78be152312bf6cc4 (patch) | |
| tree | 3c72cb5fe658fbb66575672625a7e35b25fb584c /tools/lib/traceevent/plugin_cfg80211.c | |
| parent | a21e3a34bef9e9c177be61111ced23e71e25fd8a (diff) | |
| download | linux-24eda087ccbb32f556cb3f9f78be152312bf6cc4.tar.xz | |
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/traceevent/plugin_cfg80211.c')
| -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 @@ #include "event-parse.h" static unsigned long long -process___le16_to_cpup(struct trace_seq *s, - unsigned long long *args) +process___le16_to_cpup(struct trace_seq *s, unsigned long long *args) { uint16_t *val = (uint16_t *) (unsigned long) args[0]; return val ? (long long) le16toh(*val) : 0; |