Re: [RFC patch 31/32] trace event regulator remove semicolons

From: Mark Brown
Date: Wed May 04 2011 - 04:57:46 EST


On Tue, May 03, 2011 at 07:10: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@xxxxxxxxxxxx>
> CC: Steven Rostedt <rostedt@xxxxxxxxxxx>
> CC: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> CC: Ingo Molnar <mingo@xxxxxxx>
> CC: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> CC: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>

Acked-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>

and adding Liam again.

> ---
> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/