Hi all, After merging the chrome-platform tree, today's linux-next build (arm multi_v7_defconfig) produced this warning: In file included from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h: In function 'trace_raw_output_cros_ec_sensorhub_timestamp': drivers/platform/chrome/./cros_ec_trace.h:95:12: warning: format '%lld' expects argument of type 'long long int', but argument 3 has type 'u32' {aka 'unsigned int'} [-Wformat=] 95 | TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/trace/trace_events.h:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS' 399 | trace_event_printf(iter, print); \ | ^~~~~ include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS' 80 | PARAMS(print)); \ | ^~~~~~ drivers/platform/chrome/./cros_ec_trace.h:74:1: note: in expansion of macro 'TRACE_EVENT' 74 | TRACE_EVENT(cros_ec_sensorhub_timestamp, | ^~~~~~~~~~~ drivers/platform/chrome/./cros_ec_trace.h:95:2: note: in expansion of macro 'TP_printk' 95 | TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ^~~~~~~~~ In file included from include/trace/trace_events.h:433, from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h:95:25: note: format string is defined here 95 | TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ~~~~~^ | | | long long int | %12d In file included from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h:95:12: warning: format '%lld' expects argument of type 'long long int', but argument 4 has type 'u32' {aka 'unsigned int'} [-Wformat=] 95 | TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/trace/trace_events.h:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS' 399 | trace_event_printf(iter, print); \ | ^~~~~ include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS' 80 | PARAMS(print)); \ | ^~~~~~ drivers/platform/chrome/./cros_ec_trace.h:74:1: note: in expansion of macro 'TRACE_EVENT' 74 | TRACE_EVENT(cros_ec_sensorhub_timestamp, | ^~~~~~~~~~~ drivers/platform/chrome/./cros_ec_trace.h:95:2: note: in expansion of macro 'TP_printk' 95 | TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ^~~~~~~~~ In file included from include/trace/trace_events.h:433, from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h:95:45: note: format string is defined here 95 | TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ~~~~~^ | | | long long int | %12d In file included from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h: In function 'trace_raw_output_cros_ec_sensorhub_data': drivers/platform/chrome/./cros_ec_trace.h:125:12: warning: format '%lld' expects argument of type 'long long int', but argument 4 has type 'u32' {aka 'unsigned int'} [-Wformat=] 125 | TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/trace/trace_events.h:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS' 399 | trace_event_printf(iter, print); \ | ^~~~~ include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS' 80 | PARAMS(print)); \ | ^~~~~~ drivers/platform/chrome/./cros_ec_trace.h:105:1: note: in expansion of macro 'TRACE_EVENT' 105 | TRACE_EVENT(cros_ec_sensorhub_data, | ^~~~~~~~~~~ drivers/platform/chrome/./cros_ec_trace.h:125:2: note: in expansion of macro 'TP_printk' 125 | TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ^~~~~~~~~ In file included from include/trace/trace_events.h:433, from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h:125:43: note: format string is defined here 125 | TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld", | ~~~~~^ | | | long long int | %12d In file included from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h: In function 'trace_raw_output_cros_ec_sensorhub_filter': drivers/platform/chrome/./cros_ec_trace.h:156:12: warning: format '%d' expects argument of type 'int', but argument 7 has type 's64' {aka 'long long int'} [-Wformat=] 156 | TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/trace/trace_events.h:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS' 399 | trace_event_printf(iter, print); \ | ^~~~~ include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS' 80 | PARAMS(print)); \ | ^~~~~~ drivers/platform/chrome/./cros_ec_trace.h:135:1: note: in expansion of macro 'TRACE_EVENT' 135 | TRACE_EVENT(cros_ec_sensorhub_filter, | ^~~~~~~~~~~ drivers/platform/chrome/./cros_ec_trace.h:156:2: note: in expansion of macro 'TP_printk' 156 | TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld", | ^~~~~~~~~ In file included from include/trace/trace_events.h:433, from include/trace/define_trace.h:102, from drivers/platform/chrome/cros_ec_trace.h:178, from drivers/platform/chrome/cros_ec_trace.c:215: drivers/platform/chrome/./cros_ec_trace.h:156:80: note: format string is defined here 156 | TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld", | ~^ | | | int | %lld Introduced by commit d453ceb6549a ("platform/chrome: sensorhub: Add trace events for sample") -- Cheers, Stephen Rothwell
Attachment:
pgpEPMTZ7m186.pgp
Description: OpenPGP digital signature