On Wed, 8 Aug 2018 17:29:27 +0800 "Liu, Changcheng" <changcheng....@intel.com> wrote:
> Track snd_soc_dapm_widget:id which is used as dapm > sequence index in dapm_down_seq/dapm_up_seq. It's > useful for checking dapm seq after tracking it. > > Signed-off-by: Liu Changcheng <changcheng....@intel.com> > > diff --git a/include/trace/events/asoc.h b/include/trace/events/asoc.h > index 40c300f..0bc935b 100644 > --- a/include/trace/events/asoc.h > +++ b/include/trace/events/asoc.h > @@ -92,16 +92,18 @@ DECLARE_EVENT_CLASS(snd_soc_dapm_widget, > > TP_STRUCT__entry( > __string( name, w->name ) > + __field( int, id ) > __field( int, val ) > ), > > TP_fast_assign( > __assign_str(name, w->name); > + __entry->id = w->id; > __entry->val = val; > ), > > - TP_printk("widget=%s val=%d", __get_str(name), > - (int)__entry->val) > + TP_printk("widget=%s dapm_id=%d val=%d", __get_str(name), > + (int)__entry->id, (int)__entry->val) Not sure why it was there before, but the (int) typecast isn't needed in either case. __field(int, val) makes __entry->val of type int. Same for id. -- Steve > ); > > DEFINE_EVENT(snd_soc_dapm_widget, snd_soc_dapm_widget_power,