spi/trace: drop useless and wrong (but harmless) casts
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 21 Dec 2018 10:35:35 +0000 (11:35 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 7 Jan 2019 18:31:14 +0000 (18:31 +0000)
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>
include/trace/events/spi.h

index 277bb9d..6eb014f 100644 (file)
@@ -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,