From: kernel test robot <lkp@intel.com> To: Yangbo Lu <yangbo.lu@nxp.com>, netdev@vger.kernel.org Cc: kbuild-all@lists.01.org, Yangbo Lu <yangbo.lu@nxp.com>, "David S . Miller" <davem@davemloft.net>, Richard Cochran <richardcochran@gmail.com>, Claudiu Manoil <claudiu.manoil@nxp.com>, Jakub Kicinski <kuba@kernel.org> Subject: Re: [net-next 1/6] ptp: add ptp virtual clock driver framework Date: Fri, 7 May 2021 18:42:24 +0800 [thread overview] Message-ID: <202105071819.W13hB8y6-lkp@intel.com> (raw) In-Reply-To: <20210507085756.20427-2-yangbo.lu@nxp.com> [-- Attachment #1: Type: text/plain, Size: 3807 bytes --] 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 <lkp@intel.com> 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: >> include/linux/ptp_clock_kernel.h:71:22: error: field 'cc' has incomplete type 71 | struct cyclecounter cc; | ^~ >> include/linux/ptp_clock_kernel.h:356:1: error: expected identifier or '(' before '{' token 356 | { return NULL; } | ^ >> 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 [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 23499 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [net-next 1/6] ptp: add ptp virtual clock driver framework Date: Fri, 07 May 2021 18:42:24 +0800 [thread overview] Message-ID: <202105071819.W13hB8y6-lkp@intel.com> (raw) In-Reply-To: <20210507085756.20427-2-yangbo.lu@nxp.com> [-- Attachment #1: Type: text/plain, Size: 3887 bytes --] 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 <lkp@intel.com> 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: >> include/linux/ptp_clock_kernel.h:71:22: error: field 'cc' has incomplete type 71 | struct cyclecounter cc; | ^~ >> include/linux/ptp_clock_kernel.h:356:1: error: expected identifier or '(' before '{' token 356 | { return NULL; } | ^ >> 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(a)lists.01.org [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 23499 bytes --]
next prev parent reply other threads:[~2021-05-07 10:43 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-07 8:57 [net-next 0/6] ptp: support virtual clocks for multiple domains Yangbo Lu 2021-05-07 8:57 ` [net-next 1/6] ptp: add ptp virtual clock driver framework Yangbo Lu 2021-05-07 10:42 ` kernel test robot [this message] 2021-05-07 10:42 ` kernel test robot 2021-05-07 11:26 ` kernel test robot 2021-05-07 11:26 ` kernel test robot 2021-05-07 8:57 ` [net-next 2/6] ptp: support virtual clock and domain via sysfs Yangbo Lu 2021-05-07 8:57 ` [net-next 3/6] ptp_qoriq: export ptp clock reading function for cyclecounter Yangbo Lu 2021-05-07 8:57 ` [net-next 4/6] enetc_ptp: support ptp virtual clock Yangbo Lu 2021-05-07 19:58 ` Claudiu Manoil 2021-05-07 20:50 ` kernel test robot 2021-05-07 20:50 ` kernel test robot 2021-05-07 8:57 ` [net-next 5/6] enetc: store ptp device pointer Yangbo Lu 2021-05-07 8:57 ` [net-next 6/6] enetc: support PTP domain timestamp conversion Yangbo Lu 2021-05-08 19:17 ` [net-next 0/6] ptp: support virtual clocks for multiple domains Richard Cochran 2021-05-10 3:04 ` Y.b. Lu 2021-05-10 23:18 ` Richard Cochran 2021-05-11 10:40 ` Y.b. Lu 2021-05-11 15:49 ` Richard Cochran 2021-05-14 6:41 ` Y.b. Lu 2021-05-14 17:44 ` Richard Cochran 2021-05-17 2:46 ` Y.b. Lu
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=202105071819.W13hB8y6-lkp@intel.com \ --to=lkp@intel.com \ --cc=claudiu.manoil@nxp.com \ --cc=davem@davemloft.net \ --cc=kbuild-all@lists.01.org \ --cc=kuba@kernel.org \ --cc=netdev@vger.kernel.org \ --cc=richardcochran@gmail.com \ --cc=yangbo.lu@nxp.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.