[lkml]   [2021]   [Jul]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] use 64bit timer for hpet
Hi "zhaoyan.liao",

Thank you for the patch! Yet something to improve:

[auto build test ERROR on tip/x86/core]
[also build test ERROR on v5.13 next-20210701]
[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]

base: 2c88d45edbb89029c1190bb3b136d2602f057c98
config: i386-randconfig-c021-20210628 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
git remote add linux-review
git fetch --no-tags linux-review zhaoyan-liao/use-64bit-timer-for-hpet/20210702-161515
git checkout 905ed014f601086571e608c13b14319c2f56a95c
# save the attached .config to linux build tree
make W=1 ARCH=i386

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <>

All errors (new ones prefixed by >>):

arch/x86/kernel/hpet.c: In function 'hpet_readq':
>> arch/x86/kernel/hpet.c:88:9: error: implicit declaration of function 'readq'; did you mean 'readl'? [-Werror=implicit-function-declaration]
88 | return readq(hpet_virt_address + a);
| ^~~~~
| readl
arch/x86/kernel/hpet.c: In function 'hpet_writeq':
>> arch/x86/kernel/hpet.c:93:2: error: implicit declaration of function 'writeq'; did you mean 'writel'? [-Werror=implicit-function-declaration]
93 | writeq(d, hpet_virt_address + a);
| ^~~~~~
| writel
cc1: some warnings being treated as errors

vim +88 arch/x86/kernel/hpet.c

86 inline unsigned long hpet_readq(unsigned int a)
87 {
> 88 return readq(hpet_virt_address + a);
89 }
91 static inline void hpet_writeq(unsigned long d, unsigned int a)
92 {
> 93 writeq(d, hpet_virt_address + a);
94 }

0-DAY CI Kernel Test Service, Intel Corporation
[unhandled content-type:application/gzip]
 \ /
  Last update: 2021-07-02 17:58    [W:0.083 / U:1.516 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site