Hi Geert, On Thu, May 15, 2025 at 6:59 AM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > > On Thu, 15 May 2025 at 07:04, kernel test robot <lkp@xxxxxxxxx> wrote: > > kernel test robot noticed the following build warnings: > > > > [auto build test WARNING on tip/timers/core] > > [also build test WARNING on robh/for-next linus/master v6.15-rc6 next-20250514] > > [cannot apply to daniel-lezcano/clockevents/next] > > [If your patch is applied to the wrong git tree, kindly drop us a note. > > And when submitting patch, we suggest to use '--base' as documented in > > https://git-scm.com/docs/git-format-patch#_base_tree_information] > > > > url: https://github.com/intel-lab-lkp/linux/commits/Prabhakar/dt-bindings-timer-renesas-ostm-Document-RZ-V2N-R9A09G056-support/20250506-223636 > > base: tip/timers/core > > patch link: https://lore.kernel.org/r/20250506103152.109525-3-prabhakar.mahadev-lad.rj%40bp.renesas.com > > patch subject: [PATCH v3 2/2] clocksource/drivers/renesas-ostm: Unconditionally enable reprobe support > > config: hexagon-randconfig-001-20250513 (https://download.01.org/0day-ci/archive/20250515/202505151255.rCHp8Bvu-lkp@xxxxxxxxx/config) > > compiler: clang version 21.0.0git (https://github.com/llvm/llvm-project f819f46284f2a79790038e1f6649172789734ae8) > > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250515/202505151255.rCHp8Bvu-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/202505151255.rCHp8Bvu-lkp@xxxxxxxxx/ > > > > All warnings (new ones prefixed by >>): > > > > >> drivers/clocksource/renesas-ostm.c:235:34: warning: unused variable 'ostm_of_table' [-Wunused-const-variable] > > 235 | static const struct of_device_id ostm_of_table[] = { > > | ^~~~~~~~~~~~~ > > 1 warning generated. > > > > > > vim +/ostm_of_table +235 drivers/clocksource/renesas-ostm.c > > > > 3a3e9f23c2cae9 Biju Das 2021-11-12 234 > > 3a3e9f23c2cae9 Biju Das 2021-11-12 @235 static const struct of_device_id ostm_of_table[] = { > > 3a3e9f23c2cae9 Biju Das 2021-11-12 236 { .compatible = "renesas,ostm", }, > > 3a3e9f23c2cae9 Biju Das 2021-11-12 237 { /* sentinel */ } > > 3a3e9f23c2cae9 Biju Das 2021-11-12 238 }; > > 3a3e9f23c2cae9 Biju Das 2021-11-12 239 > > The table is unused if CONFIG_OF=n due to > > .of_match_table = of_match_ptr(ostm_of_table), > Thanks for the pointer, I'll add a __maybe_unused compiler attribute and send a new version. Cheers, Prabhakar > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds
