Re: [PATCH] i2c: Add message transfer tracepoints for I2C and SMBUS

From: David Howells
Date: Fri Dec 13 2013 - 12:05:20 EST


Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> > Would it be possible to print this as a string rather than a number?
> > The protocol numbers are completely arbitrary, so the reader would have
> > to open i2c.h each time to figure out what is what.
>
> That can be done in the TP_printk() with:
>
> ".. %s ..", ..,
> __print_symbolic(__entry->protocol,
> { I2C_SMBUS_QUICK , "QUICK" },
> { I2C_SMBUS_BYTE , "BYTE" },
> { I2C_SMBUS_BYTE_DATA , "BYTE_DATA" },
> [...]
> { I2C_SMBUS_I2C_BLOCK_DATA, "I2C_BLOCK_DATA"}), ...

What happens if the number isn't in the table?

David
--
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/