[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110503143046.GR1762@opensource.wolfsonmicro.com>
Date: Tue, 3 May 2011 15:30:47 +0100
From: Mark Brown <broonie@...nsource.wolfsonmicro.com>
To: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
Cc: LKML <linux-kernel@...r.kernel.org>,
Steven Rostedt <rostedt@...dmis.org>,
Ingo Molnar <mingo@...e.hu>,
Thomas Gleixner <tglx@...utronix.de>,
Frederic Weisbecker <fweisbec@...il.com>, lrg@...mlogic.co.uk
Subject: Re: [RFC patch 31/32] trace event regulator remove semicolons
On Mon, May 02, 2011 at 05:11:54PM -0400, Mathieu Desnoyers wrote:
> Part of the gradual TRACE_EVENT() semicolon removal. Enables creation of array
> of events, thus saving space for trace event probes.
>
> Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
> CC: Steven Rostedt <rostedt@...dmis.org>
> CC: Frederic Weisbecker <fweisbec@...il.com>
> CC: Ingo Molnar <mingo@...e.hu>
> CC: Thomas Gleixner <tglx@...utronix.de>
> CC: Mark Brown <broonie@...nsource.wolfsonmicro.com>
Acked-by: Mark Brown <broonie@...nsource.wolfsonmicro.com>
and adding Liam.
> ---
> include/trace/events/regulator.h | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> Index: linux-2.6-lttng/include/trace/events/regulator.h
> ===================================================================
> --- linux-2.6-lttng.orig/include/trace/events/regulator.h
> +++ linux-2.6-lttng/include/trace/events/regulator.h
> @@ -27,7 +27,7 @@ DECLARE_EVENT_CLASS(regulator_basic,
>
> TP_printk("name=%s", __get_str(name))
>
> -);
> +)
>
> DEFINE_EVENT(regulator_basic, regulator_enable,
>
> @@ -35,7 +35,7 @@ DEFINE_EVENT(regulator_basic, regulator_
>
> TP_ARGS(name)
>
> -);
> +)
>
> DEFINE_EVENT(regulator_basic, regulator_enable_delay,
>
> @@ -43,7 +43,7 @@ DEFINE_EVENT(regulator_basic, regulator_
>
> TP_ARGS(name)
>
> -);
> +)
>
> DEFINE_EVENT(regulator_basic, regulator_enable_complete,
>
> @@ -51,7 +51,7 @@ DEFINE_EVENT(regulator_basic, regulator_
>
> TP_ARGS(name)
>
> -);
> +)
>
> DEFINE_EVENT(regulator_basic, regulator_disable,
>
> @@ -59,7 +59,7 @@ DEFINE_EVENT(regulator_basic, regulator_
>
> TP_ARGS(name)
>
> -);
> +)
>
> DEFINE_EVENT(regulator_basic, regulator_disable_complete,
>
> @@ -67,7 +67,7 @@ DEFINE_EVENT(regulator_basic, regulator_
>
> TP_ARGS(name)
>
> -);
> +)
>
> /*
> * Events that take a range of numerical values, mostly for voltages
> @@ -93,7 +93,7 @@ DECLARE_EVENT_CLASS(regulator_range,
>
> TP_printk("name=%s (%d-%d)", __get_str(name),
> (int)__entry->min, (int)__entry->max)
> -);
> +)
>
> DEFINE_EVENT(regulator_range, regulator_set_voltage,
>
> @@ -101,7 +101,7 @@ DEFINE_EVENT(regulator_range, regulator_
>
> TP_ARGS(name, min, max)
>
> -);
> +)
>
>
> /*
> @@ -125,7 +125,7 @@ DECLARE_EVENT_CLASS(regulator_value,
>
> TP_printk("name=%s, val=%u", __get_str(name),
> (int)__entry->val)
> -);
> +)
>
> DEFINE_EVENT(regulator_value, regulator_set_voltage_complete,
>
> @@ -133,7 +133,7 @@ DEFINE_EVENT(regulator_value, regulator_
>
> TP_ARGS(name, value)
>
> -);
> +)
>
> #endif /* _TRACE_POWER_H */
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists