[PATCHv2,4/5] ASoC: Intel: Add trace support for generic SST IPC messages.

Commit Message

Liam Girdwood Feb. 17, 2014, 1:32 p.m. UTC

Provide a trace mechanism for debugging generic SST IPC messages.

Signed-off-by: Liam Girdwood <[e-mail]>
---
 include/trace/events/sst.h | 148 +++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 148 insertions(+)
 create mode 100644 include/trace/events/sst.h

Comments

Mark Brown Feb. 18, 2014, 1:36 a.m. UTC | #1

On Mon, Feb 17, 2014 at 01:32:09PM +0000, Liam Girdwood wrote:

>  include/trace/events/sst.h | 148 +++++++++++++++++++++++++++++++++++++++++++++

intel-sst perhaps?  It just seems like something someone else might use
as a name.

Liam Girdwood Feb. 18, 2014, 8:52 a.m. UTC | #2

On Tue, 2014-02-18 at 10:36 +0900, Mark Brown wrote:
> On Mon, Feb 17, 2014 at 01:32:09PM +0000, Liam Girdwood wrote:
> 
> >  include/trace/events/sst.h | 148 +++++++++++++++++++++++++++++++++++++++++++++
> 
> intel-sst perhaps?  It just seems like something someone else might use
> as a name.

True, it's a little generic. I also have trace/events/hswadsp.h for
Haswell IPC but that's quite specific so should be OK.

Liam

Takashi Iwai Feb. 18, 2014, 9:34 a.m. UTC | #3

At Tue, 18 Feb 2014 08:52:12 +0000,
Liam Girdwood wrote:
> 
> On Tue, 2014-02-18 at 10:36 +0900, Mark Brown wrote:
> > On Mon, Feb 17, 2014 at 01:32:09PM +0000, Liam Girdwood wrote:
> > 
> > >  include/trace/events/sst.h | 148 +++++++++++++++++++++++++++++++++++++++++++++
> > 
> > intel-sst perhaps?  It just seems like something someone else might use
> > as a name.
> 
> True, it's a little generic.

There are enough TLAs for SST, someone might want to implement a Samoa
Standard Time driver ;)
    http://en.wikipedia.org/wiki/SST


Takashi

Patch

diff --git a/include/trace/events/sst.h b/include/trace/events/sst.h
new file mode 100644
index 0000000..1a0d6b3
--- /dev/null
+++ b/include/trace/events/sst.h
@@ -0,0 +1,148 @@ 
+#undef TRACE_SYSTEM
+#define TRACE_SYSTEM sst
+
+#if !defined(_TRACE_SST_H) || defined(TRACE_HEADER_MULTI_READ)
+#define _TRACE_SST_H
+
+#include <linux/types.h>
+#include <linux/ktime.h>
+#include <linux/tracepoint.h>
+
+DECLARE_EVENT_CLASS(sst_ipc_msg,
+
+	TP_PROTO(unsigned int val),
+
+	TP_ARGS(val),
+
+	TP_STRUCT__entry(
+		__field(	unsigned int,	val		)
+	),
+
+	TP_fast_assign(
+		__entry->val = val;
+	),
+
+	TP_printk("0x%8.8x", (unsigned int)__entry->val)
+);
+
+DEFINE_EVENT(sst_ipc_msg, sst_ipc_msg_tx,
+
+	TP_PROTO(unsigned int val),
+
+	TP_ARGS(val)
+
+);
+
+DEFINE_EVENT(sst_ipc_msg, sst_ipc_msg_rx,
+
+	TP_PROTO(unsigned int val),
+
+	TP_ARGS(val)
+
+);
+
+DECLARE_EVENT_CLASS(sst_ipc_mailbox,
+
+	TP_PROTO(unsigned int offset, unsigned int val),
+
+	TP_ARGS(offset, val),
+
+	TP_STRUCT__entry(
+		__field(	unsigned int,	offset		)
+		__field(	unsigned int,	val		)
+	),
+
+	TP_fast_assign(
+		__entry->offset = offset;
+		__entry->val = val;
+	),
+
+	TP_printk(" 0x%4.4x = 0x%8.8x",
+		(unsigned int)__entry->offset, (unsigned int)__entry->val)
+);
+
+DEFINE_EVENT(sst_ipc_mailbox, sst_ipc_inbox_rdata,
+
+	TP_PROTO(unsigned int offset, unsigned int val),
+
+	TP_ARGS(offset, val)
+
+);
+
+DEFINE_EVENT(sst_ipc_mailbox, sst_ipc_inbox_wdata,
+
+	TP_PROTO(unsigned int offset, unsigned int val),
+
+	TP_ARGS(offset, val)
+
+);
+
+DEFINE_EVENT(sst_ipc_mailbox, sst_ipc_outbox_rdata,
+
+	TP_PROTO(unsigned int offset, unsigned int val),
+
+	TP_ARGS(offset, val)
+
+);
+
+DEFINE_EVENT(sst_ipc_mailbox, sst_ipc_outbox_wdata,
+
+	TP_PROTO(unsigned int offset, unsigned int val),
+
+	TP_ARGS(offset, val)
+
+);
+
+DECLARE_EVENT_CLASS(sst_ipc_mailbox_info,
+
+	TP_PROTO(unsigned int size),
+
+	TP_ARGS(size),
+
+	TP_STRUCT__entry(
+		__field(	unsigned int,	size		)
+	),
+
+	TP_fast_assign(
+		__entry->size = size;
+	),
+
+	TP_printk("Mailbox bytes 0x%8.8x", (unsigned int)__entry->size)
+);
+
+DEFINE_EVENT(sst_ipc_mailbox_info, sst_ipc_inbox_read,
+
+	TP_PROTO(unsigned int size),
+
+	TP_ARGS(size)
+
+);
+
+DEFINE_EVENT(sst_ipc_mailbox_info, sst_ipc_inbox_write,
+
+	TP_PROTO(unsigned int size),
+
+	TP_ARGS(size)
+
+);
+
+DEFINE_EVENT(sst_ipc_mailbox_info, sst_ipc_outbox_read,
+
+	TP_PROTO(unsigned int size),
+
+	TP_ARGS(size)
+
+);
+
+DEFINE_EVENT(sst_ipc_mailbox_info, sst_ipc_outbox_write,
+
+	TP_PROTO(unsigned int size),
+
+	TP_ARGS(size)
+
+);
+
+#endif /* _TRACE_SST_H */
+
+/* This part must be outside protection */
+#include <trace/define_trace.h>