drivers/gpu/drm/gma500/oaktrail_crtc.c:659:30: sparse: sparse: symbol 'mrst_clock_funcs' was not declared. Should it be static?

From: kernel test robot
Date: Sat Jan 06 2024 - 16:55:48 EST


Hi Thomas,

First bad commit (maybe != root cause):

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 52b1853b080a082ec3749c3a9577f6c71b1d4a90
commit: e40f97ef12772f8eb04b6a155baa1e0e2e8f3ecc drm/gma500: Drop DRM_GMA600 config option
date: 2 years, 11 months ago
config: i386-randconfig-r132-20240106 (https://download.01.org/0day-ci/archive/20240107/202401070528.nCUuFKLa-lkp@xxxxxxxxx/config)
compiler: ClangBuiltLinux clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240107/202401070528.nCUuFKLa-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/202401070528.nCUuFKLa-lkp@xxxxxxxxx/

sparse warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/gma500/oaktrail_crtc.c:659:30: sparse: sparse: symbol 'mrst_clock_funcs' was not declared. Should it be static?

vim +/mrst_clock_funcs +659 drivers/gpu/drm/gma500/oaktrail_crtc.c

1b082ccf590110 Alan Cox 2011-11-03 657
ac6113ebb70d4b Patrik Jakobsson 2013-11-06 658 /* Not used yet */
ac6113ebb70d4b Patrik Jakobsson 2013-11-06 @659 const struct gma_clock_funcs mrst_clock_funcs = {

:::::: The code at line 659 was first introduced by commit
:::::: ac6113ebb70d4bc7018db4e73f923653347da743 drm/gma500/mrst: Add SDVO clock calculation

:::::: TO: Patrik Jakobsson <patrik.r.jakobsson@xxxxxxxxx>
:::::: CC: Patrik Jakobsson <patrik.r.jakobsson@xxxxxxxxx>

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki