2022-08-15 01:04:18

by kernel test robot

Subject: drivers/clocksource/timer-ti-dm.c:1039:34: warning: 'omap_timer_match' defined but not used

Hi Tony,

First bad commit (maybe != root cause):

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 568035b01cfb107af8d2e4bd2fb9aea22cf5b868
commit: ab0bbef3ae0f6b5a3b60671cd0124d0fc4fc2567 clocksource/drivers/timer-ti-dm: Make timer selectable for ARCH_K3
date: 3 weeks ago
config: x86_64-buildonly-randconfig-r004-20220815 (https://download.01.org/0day-ci/archive/20220815/[email protected]/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce (this is a W=1 build):
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ab0bbef3ae0f6b5a3b60671cd0124d0fc4fc2567
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout ab0bbef3ae0f6b5a3b60671cd0124d0fc4fc2567
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/clocksource/ drivers/soc/mediatek/ drivers/usb/misc/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <[email protected]>

All warnings (new ones prefixed by >>):

>> drivers/clocksource/timer-ti-dm.c:1039:34: warning: 'omap_timer_match' defined but not used [-Wunused-const-variable=]
1039 | static const struct of_device_id omap_timer_match[] = {
| ^~~~~~~~~~~~~~~~


vim +/omap_timer_match +1039 drivers/clocksource/timer-ti-dm.c

d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1038
9725f4451a9ccd arch/arm/plat-omap/dmtimer.c Jon Hunter 2012-05-14 @1039 static const struct of_device_id omap_timer_match[] = {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1040 {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1041 .compatible = "ti,omap2420-timer",
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1042 },
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1043 {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1044 .compatible = "ti,omap3430-timer",
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1045 .data = &omap3plus_pdata,
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1046 },
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1047 {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1048 .compatible = "ti,omap4430-timer",
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1049 .data = &omap3plus_pdata,
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1050 },
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1051 {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1052 .compatible = "ti,omap5430-timer",
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1053 .data = &omap3plus_pdata,
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1054 },
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1055 {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1056 .compatible = "ti,am335x-timer",
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1057 .data = &omap3plus_pdata,
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1058 },
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1059 {
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1060 .compatible = "ti,am335x-timer-1ms",
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1061 .data = &omap3plus_pdata,
d1c6ccfe3dbdd2 arch/arm/plat-omap/dmtimer.c Jon Hunter 2013-03-19 1062 },
8c0cabd7970c0d arch/arm/plat-omap/dmtimer.c Neil Armstrong 2015-10-22 1063 {
8c0cabd7970c0d arch/arm/plat-omap/dmtimer.c Neil Armstrong 2015-10-22 1064 .compatible = "ti,dm816-timer",
8c0cabd7970c0d arch/arm/plat-omap/dmtimer.c Neil Armstrong 2015-10-22 1065 .data = &omap3plus_pdata,
8c0cabd7970c0d arch/arm/plat-omap/dmtimer.c Neil Armstrong 2015-10-22 1066 },
9725f4451a9ccd arch/arm/plat-omap/dmtimer.c Jon Hunter 2012-05-14 1067 {},
9725f4451a9ccd arch/arm/plat-omap/dmtimer.c Jon Hunter 2012-05-14 1068 };
9725f4451a9ccd arch/arm/plat-omap/dmtimer.c Jon Hunter 2012-05-14 1069 MODULE_DEVICE_TABLE(of, omap_timer_match);
9725f4451a9ccd arch/arm/plat-omap/dmtimer.c Jon Hunter 2012-05-14 1070

:::::: The code at line 1039 was first introduced by commit
:::::: 9725f4451a9ccd159b1d13f63e05896cd9bce07d ARM: OMAP: Add DT support for timer driver

:::::: TO: Jon Hunter <[email protected]>
:::::: CC: Jon Hunter <[email protected]>

--
0-DAY CI Kernel Test Service
https://01.org/lkp

2022-08-15 12:07:51

by Tony Lindgren

Subject: Re: drivers/clocksource/timer-ti-dm.c:1039:34: warning: 'omap_timer_match' defined but not used

* kernel test robot <[email protected]> [220815 00:34]:
> Hi Tony,
>
> First bad commit (maybe != root cause):
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 568035b01cfb107af8d2e4bd2fb9aea22cf5b868
> commit: ab0bbef3ae0f6b5a3b60671cd0124d0fc4fc2567 clocksource/drivers/timer-ti-dm: Make timer selectable for ARCH_K3
> date: 3 weeks ago
> config: x86_64-buildonly-randconfig-r004-20220815 (https://download.01.org/0day-ci/archive/20220815/[email protected]/config)
> compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
> reproduce (this is a W=1 build):
> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ab0bbef3ae0f6b5a3b60671cd0124d0fc4fc2567
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git fetch --no-tags linus master
> git checkout ab0bbef3ae0f6b5a3b60671cd0124d0fc4fc2567
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/clocksource/ drivers/soc/mediatek/ drivers/usb/misc/
>
> If you fix the issue, kindly add following tag where applicable
> Reported-by: kernel test robot <[email protected]>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/clocksource/timer-ti-dm.c:1039:34: warning: 'omap_timer_match' defined but not used [-Wunused-const-variable=]
> 1039 | static const struct of_device_id omap_timer_match[] = {
> | ^~~~~~~~~~~~~~~~

Seems this is best fixes with __maybe_unsued for omap_timer_match[], I'll
send a patch.

Regards,

Tony