Unverified Commit 983f6ae9 authored Dec 21, 2018 by Uwe Kleine-König Committed by Mark Brown Jan 07, 2019

spi/trace: drop useless and wrong (but harmless) casts

bus_num, chip_select and len are already ints, so there is no gain in
casting them to int. xfer is a pointer to a struct spi_transfer. Casting
that to struct spi_message * is wrong but as only the pointer value is
used for the %p format specifier no harm is done.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Mark Brown <broonie@kernel.org>

parent 560b097c

include/trace/events/spi.h

+3 −4

Original line number	Diff line number	Diff line
		@@ -129,10 +129,9 @@ DECLARE_EVENT_CLASS(spi_transfer,
		__entry->len = xfer->len;
		),

		TP_printk("spi%d.%d %p len=%d", (int)__entry->bus_num,
		(int)__entry->chip_select,
		(struct spi_message *)__entry->xfer,
		(int)__entry->len)
		TP_printk("spi%d.%d %p len=%d",
		__entry->bus_num, __entry->chip_select,
		__entry->xfer, __entry->len)
		);

		DEFINE_EVENT(spi_transfer, spi_transfer_start,