On Mon, Sep 19, 2022 at 11:08:10AM +0800, Ren Zhijie wrote: > +#ifdef CONFIG_TRACING > static void > qla_trace_init(void) > { > @@ -2868,7 +2871,7 @@ qla_trace_uninit(void) > return; > trace_array_put(qla_trc_array); > } > - > +#endif FWIW, the qla2xxx driver uses a different pattern for ifdefs static void qla_trace_init(void) { #ifdef CONFIG_TRACING [...] #endif } This avoid to updated the callside with ifdefs.