Hi Krishna, kernel test robot noticed the following build errors: [auto build test ERROR on 3006adf3be79cde4d14b1800b963b82b6e5572e0] url: https://github.com/intel-lab-lkp/linux/commits/Krishna-chaitanya-chundru/bus-mhi-host-Add-tracing-support/20231111-101955 base: 3006adf3be79cde4d14b1800b963b82b6e5572e0 patch link: https://lore.kernel.org/r/20231111-ftrace_support-v3-1-f358d2911a74%40quicinc.com patch subject: [PATCH v3] bus: mhi: host: Add tracing support config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20231111/202311111237.zTm5NBLH-lkp@xxxxxxxxx/config) compiler: sh4-linux-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231111/202311111237.zTm5NBLH-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202311111237.zTm5NBLH-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): In file included from include/trace/define_trace.h:102, from drivers/bus/mhi/host/trace.h:224, from drivers/bus/mhi/host/init.c:24: drivers/bus/mhi/host/./trace.h: In function 'trace_event_raw_event_mhi_process_ctrl_ev_ring': >> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'}) 144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)]) | ^~ include/trace/trace_events.h:402:11: note: in definition of macro 'DECLARE_EVENT_CLASS' 402 | { assign; } \ | ^~~~~~ include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS' 44 | PARAMS(assign), \ | ^~~~~~ drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT' 123 | TRACE_EVENT(mhi_process_ctrl_ev_ring, | ^~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign' 138 | TP_fast_assign( | ^~~~~~~~~~~~~~ include/linux/compiler_types.h:413:9: note: in expansion of macro '__compiletime_assert' 413 | __compiletime_assert(condition, msg, prefix, suffix) | ^~~~~~~~~~~~~~~~~~~~ include/linux/compiler_types.h:425:9: note: in expansion of macro '_compiletime_assert' 425 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) | ^~~~~~~~~~~~~~~~~~~ include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert' 39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) | ^~~~~~~~~~~~~~~~~~ include/linux/bitfield.h:71:17: note: in expansion of macro 'BUILD_BUG_ON_MSG' 71 | BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \ | ^~~~~~~~~~~~~~~~ include/linux/bitfield.h:61:43: note: in expansion of macro '__unsigned_scalar_typeof' 61 | #define __bf_cast_unsigned(type, x) ((__unsigned_scalar_typeof(type))(x)) | ^~~~~~~~~~~~~~~~~~~~~~~~ include/linux/bitfield.h:72:34: note: in expansion of macro '__bf_cast_unsigned' 72 | __bf_cast_unsigned(_reg, ~0ull), \ | ^~~~~~~~~~~~~~~~~~ include/linux/bitfield.h:154:17: note: in expansion of macro '__BF_FIELD_CHECK' 154 | __BF_FIELD_CHECK(_mask, _reg, 0U, "FIELD_GET: "); \ | ^~~~~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~ include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu' 89 | #define le32_to_cpu __le32_to_cpu | ^~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE' 144 | __entry->state = MHI_TRE_GET_EV_STATE(rp); | ^~~~~~~~~~~~~~~~~~~~ >> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'}) 144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)]) | ^~ include/trace/trace_events.h:402:11: note: in definition of macro 'DECLARE_EVENT_CLASS' 402 | { assign; } \ | ^~~~~~ include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS' 44 | PARAMS(assign), \ | ^~~~~~ drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT' 123 | TRACE_EVENT(mhi_process_ctrl_ev_ring, | ^~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign' 138 | TP_fast_assign( | ^~~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~ include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu' 89 | #define le32_to_cpu __le32_to_cpu | ^~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE' 144 | __entry->state = MHI_TRE_GET_EV_STATE(rp); | ^~~~~~~~~~~~~~~~~~~~ In file included from include/trace/define_trace.h:103: drivers/bus/mhi/host/./trace.h: In function 'perf_trace_mhi_process_ctrl_ev_ring': >> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'}) 144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)]) | ^~ include/trace/perf.h:51:11: note: in definition of macro 'DECLARE_EVENT_CLASS' 51 | { assign; } \ | ^~~~~~ include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS' 44 | PARAMS(assign), \ | ^~~~~~ drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT' 123 | TRACE_EVENT(mhi_process_ctrl_ev_ring, | ^~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign' 138 | TP_fast_assign( | ^~~~~~~~~~~~~~ include/linux/compiler_types.h:413:9: note: in expansion of macro '__compiletime_assert' 413 | __compiletime_assert(condition, msg, prefix, suffix) | ^~~~~~~~~~~~~~~~~~~~ include/linux/compiler_types.h:425:9: note: in expansion of macro '_compiletime_assert' 425 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) | ^~~~~~~~~~~~~~~~~~~ include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert' 39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) | ^~~~~~~~~~~~~~~~~~ include/linux/bitfield.h:71:17: note: in expansion of macro 'BUILD_BUG_ON_MSG' 71 | BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \ | ^~~~~~~~~~~~~~~~ include/linux/bitfield.h:61:43: note: in expansion of macro '__unsigned_scalar_typeof' 61 | #define __bf_cast_unsigned(type, x) ((__unsigned_scalar_typeof(type))(x)) | ^~~~~~~~~~~~~~~~~~~~~~~~ include/linux/bitfield.h:72:34: note: in expansion of macro '__bf_cast_unsigned' 72 | __bf_cast_unsigned(_reg, ~0ull), \ | ^~~~~~~~~~~~~~~~~~ include/linux/bitfield.h:154:17: note: in expansion of macro '__BF_FIELD_CHECK' 154 | __BF_FIELD_CHECK(_mask, _reg, 0U, "FIELD_GET: "); \ | ^~~~~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~ include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu' 89 | #define le32_to_cpu __le32_to_cpu | ^~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE' 144 | __entry->state = MHI_TRE_GET_EV_STATE(rp); | ^~~~~~~~~~~~~~~~~~~~ >> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'}) 144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)]) | ^~ include/trace/perf.h:51:11: note: in definition of macro 'DECLARE_EVENT_CLASS' 51 | { assign; } \ | ^~~~~~ include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS' 44 | PARAMS(assign), \ | ^~~~~~ drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT' 123 | TRACE_EVENT(mhi_process_ctrl_ev_ring, | ^~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign' 138 | TP_fast_assign( | ^~~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~ include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu' 89 | #define le32_to_cpu __le32_to_cpu | ^~~~~~~~~~~~~ drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD' 159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0))) | ^~~~~~~~~~~~~~~~~ drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE' 144 | __entry->state = MHI_TRE_GET_EV_STATE(rp); | ^~~~~~~~~~~~~~~~~~~~ vim +144 drivers/bus/mhi/host/../common.h 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 122 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 123 /* Channel reset command */ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 124 #define MHI_TRE_CMD_RESET_PTR 0 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 125 #define MHI_TRE_CMD_RESET_DWORD0 0 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 126 #define MHI_TRE_CMD_RESET_DWORD1(chid) cpu_to_le32(FIELD_PREP(GENMASK(31, 24), chid) | \ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 127 FIELD_PREP(GENMASK(23, 16), \ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 128 MHI_CMD_RESET_CHAN)) 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 129 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 130 /* Channel stop command */ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 131 #define MHI_TRE_CMD_STOP_PTR 0 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 132 #define MHI_TRE_CMD_STOP_DWORD0 0 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 133 #define MHI_TRE_CMD_STOP_DWORD1(chid) cpu_to_le32(FIELD_PREP(GENMASK(31, 24), chid) | \ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 134 FIELD_PREP(GENMASK(23, 16), \ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 135 MHI_CMD_STOP_CHAN)) 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 136 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 137 /* Channel start command */ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 138 #define MHI_TRE_CMD_START_PTR 0 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 139 #define MHI_TRE_CMD_START_DWORD0 0 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 140 #define MHI_TRE_CMD_START_DWORD1(chid) cpu_to_le32(FIELD_PREP(GENMASK(31, 24), chid) | \ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 141 FIELD_PREP(GENMASK(23, 16), \ 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 142 MHI_CMD_START_CHAN)) 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 143 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 @144 #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)]) 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 145 #define MHI_TRE_GET_CMD_CHID(tre) FIELD_GET(GENMASK(31, 24), MHI_TRE_GET_DWORD(tre, 1)) 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 146 #define MHI_TRE_GET_CMD_TYPE(tre) FIELD_GET(GENMASK(23, 16), MHI_TRE_GET_DWORD(tre, 1)) 8485149c7a84bb Manivannan Sadhasivam 2022-03-01 147 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki