Re: [net-next 1/6] ptp: add ptp virtual clock driver framework
From: kernel test robot <hidden>
Date: 2021-05-07 10:43:21
Also in:
oe-kbuild-all
Hi Yangbo, I love your patch! Yet something to improve: [auto build test ERROR on 9d31d2338950293ec19d9b095fbaa9030899dcb4] url: https://github.com/0day-ci/linux/commits/Yangbo-Lu/ptp-support-virtual-clocks-for-multiple-domains/20210507-164927 base: 9d31d2338950293ec19d9b095fbaa9030899dcb4 config: microblaze-randconfig-s032-20210507 (attached as .config) compiler: microblaze-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # apt-get install sparse # sparse version: v0.6.3-341-g8af24329-dirty # https://github.com/0day-ci/linux/commit/1f46e22fa0f24ac9acde10ca897266e0bac0f367 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Yangbo-Lu/ptp-support-virtual-clocks-for-multiple-domains/20210507-164927 git checkout 1f46e22fa0f24ac9acde10ca897266e0bac0f367 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' W=1 ARCH=microblaze If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <redacted> All error/warnings (new ones prefixed by >>): In file included from include/linux/spi/spi.h:17, from include/linux/iio/common/st_sensors.h:14, from include/linux/iio/common/st_sensors_i2c.h:14, from drivers/iio/common/st_sensors/st_sensors_i2c.c:16:
quoted
include/linux/ptp_clock_kernel.h:71:22: error: field 'cc' has incomplete type
71 | struct cyclecounter cc;
| ^~quoted
include/linux/ptp_clock_kernel.h:356:1: error: expected identifier or '(' before '{' token
356 | { return NULL; }
| ^quoted
include/linux/ptp_clock_kernel.h:358:6: warning: no previous prototype for 'ptp_clock_domain_tstamp' [-Wmissing-prototypes]
358 | void ptp_clock_domain_tstamp(struct device *dev, u64 *tstamp, u8 domain)
| ^~~~~~~~~~~~~~~~~~~~~~~
include/linux/ptp_clock_kernel.h:355:38: warning: 'ptp_get_pclock_info' declared 'static' but never defined [-Wunused-function]
355 | static inline struct ptp_clock_info *ptp_get_pclock_info(const struct cyclecounter *cc);
| ^~~~~~~~~~~~~~~~~~~
vim +/cc +71 include/linux/ptp_clock_kernel.h
50
51 /**
52 * struct ptp_vclock_cc - ptp virtual clock cycle counter info
53 *
54 * @cc: cyclecounter structure
55 * @refresh_interval: time interval to refresh time counter, to avoid 64-bit
56 * overflow during delta conversion. For example, with
57 * cc.mult value 2^28, there are 36 bits left of cycle
58 * counter. With 1 ns counter resolution, the overflow time
59 * is 2^36 ns which is 68.7 s. The refresh_interval may be
60 * (60 * HZ) less than 68.7 s.
61 * @mult_num: parameter for cc.mult adjustment calculation, see below
62 * @mult_dem: parameter for cc.mult adjustment calculation, see below
63 *
64 * scaled_ppm to adjustment(mult_adj) of cc.mult
65 *
66 * mult_adj = mult * (ppb / 10^9)
67 * = mult * (scaled_ppm * 1000 / 2^16) / 10^9
68 * = scaled_ppm * mult_num / mult_dem
69 */
70 struct ptp_vclock_cc {
> 71 struct cyclecounter cc;
72 unsigned long refresh_interval;
73 u32 mult_num;
74 u32 mult_dem;
75 };
76
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Attachments
- .config.gz [application/gzip] 23499 bytes