From: kernel test robot <lkp@intel.com>
To: Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
davem@davemloft.net, joabreu@synopsys.com, kuba@kernel.org,
alexandre.torgue@st.com, peppe.cavallaro@st.com,
mcoquelin.stm32@gmail.com
Cc: kbuild-all@lists.01.org, netdev@vger.kernel.org,
boon.leong.ong@intel.com, weifeng.voon@intel.com,
vee.khee.wong@intel.com
Subject: Re: [PATCH v1 net-next 1/3] net: stmmac: separate the tas basetime calculation function
Date: Tue, 1 Jun 2021 18:58:32 +0800 [thread overview]
Message-ID: <202106011813.ZhUFgEvu-lkp@intel.com> (raw)
In-Reply-To: <20210601083813.1078-2-xiaoliang.yang_1@nxp.com>
[-- Attachment #1: Type: text/plain, Size: 2413 bytes --]
Hi Xiaoliang,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
url: https://github.com/0day-ci/linux/commits/Xiaoliang-Yang/net-stmmac-re-configure-tas-basetime-after-ptp-time-adjust/20210601-163025
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 7fc6d3abc0844a9b8ef67937af465a417af6e9e9
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/5abe8df9d55824e7eff47e48b66f57260b0bfe50
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Xiaoliang-Yang/net-stmmac-re-configure-tas-basetime-after-ptp-time-adjust/20210601-163025
git checkout 5abe8df9d55824e7eff47e48b66f57260b0bfe50
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/net/ethernet/stmicro/stmmac/stmmac_tc.c:716:19: warning: no previous prototype for 'stmmac_calc_tas_basetime' [-Wmissing-prototypes]
716 | struct timespec64 stmmac_calc_tas_basetime(ktime_t old_base_time,
| ^~~~~~~~~~~~~~~~~~~~~~~~
vim +/stmmac_calc_tas_basetime +716 drivers/net/ethernet/stmicro/stmmac/stmmac_tc.c
715
> 716 struct timespec64 stmmac_calc_tas_basetime(ktime_t old_base_time,
717 ktime_t current_time,
718 u64 cycle_time)
719 {
720 struct timespec64 time;
721
722 if (ktime_after(old_base_time, current_time)) {
723 time = ktime_to_timespec64(old_base_time);
724 } else {
725 s64 n;
726 ktime_t base_time;
727
728 n = div64_s64(ktime_sub_ns(current_time, old_base_time),
729 cycle_time);
730 base_time = ktime_add_ns(old_base_time,
731 (n + 1) * cycle_time);
732
733 time = ktime_to_timespec64(base_time);
734 }
735
736 return time;
737 }
738
---
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: 68164 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v1 net-next 1/3] net: stmmac: separate the tas basetime calculation function
Date: Tue, 01 Jun 2021 18:58:32 +0800 [thread overview]
Message-ID: <202106011813.ZhUFgEvu-lkp@intel.com> (raw)
In-Reply-To: <20210601083813.1078-2-xiaoliang.yang_1@nxp.com>
[-- Attachment #1: Type: text/plain, Size: 2475 bytes --]
Hi Xiaoliang,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
url: https://github.com/0day-ci/linux/commits/Xiaoliang-Yang/net-stmmac-re-configure-tas-basetime-after-ptp-time-adjust/20210601-163025
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 7fc6d3abc0844a9b8ef67937af465a417af6e9e9
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/5abe8df9d55824e7eff47e48b66f57260b0bfe50
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Xiaoliang-Yang/net-stmmac-re-configure-tas-basetime-after-ptp-time-adjust/20210601-163025
git checkout 5abe8df9d55824e7eff47e48b66f57260b0bfe50
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/net/ethernet/stmicro/stmmac/stmmac_tc.c:716:19: warning: no previous prototype for 'stmmac_calc_tas_basetime' [-Wmissing-prototypes]
716 | struct timespec64 stmmac_calc_tas_basetime(ktime_t old_base_time,
| ^~~~~~~~~~~~~~~~~~~~~~~~
vim +/stmmac_calc_tas_basetime +716 drivers/net/ethernet/stmicro/stmmac/stmmac_tc.c
715
> 716 struct timespec64 stmmac_calc_tas_basetime(ktime_t old_base_time,
717 ktime_t current_time,
718 u64 cycle_time)
719 {
720 struct timespec64 time;
721
722 if (ktime_after(old_base_time, current_time)) {
723 time = ktime_to_timespec64(old_base_time);
724 } else {
725 s64 n;
726 ktime_t base_time;
727
728 n = div64_s64(ktime_sub_ns(current_time, old_base_time),
729 cycle_time);
730 base_time = ktime_add_ns(old_base_time,
731 (n + 1) * cycle_time);
732
733 time = ktime_to_timespec64(base_time);
734 }
735
736 return time;
737 }
738
---
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: 68164 bytes --]
next prev parent reply other threads:[~2021-06-01 10:59 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-01 8:38 [PATCH v1 net-next 0/3] net: stmmac: re-configure tas basetime after ptp time adjust Xiaoliang Yang
2021-06-01 8:38 ` Xiaoliang Yang
2021-06-01 8:38 ` [PATCH v1 net-next 1/3] net: stmmac: separate the tas basetime calculation function Xiaoliang Yang
2021-06-01 8:38 ` Xiaoliang Yang
2021-06-01 10:58 ` kernel test robot [this message]
2021-06-01 10:58 ` kernel test robot
2021-06-01 8:38 ` [PATCH v1 net-next 2/3] net: stmmac: add mutex lock to protect est parameters Xiaoliang Yang
2021-06-01 8:38 ` Xiaoliang Yang
2021-06-01 8:38 ` [PATCH v1 net-next 3/3] net: stmmac: ptp: update tas basetime after ptp adjust Xiaoliang Yang
2021-06-01 8:38 ` Xiaoliang Yang
2021-06-01 11:21 ` kernel test robot
2021-06-01 11:21 ` kernel test robot
2021-06-02 10:18 ` Rui Sousa
2021-06-02 10:18 ` Rui Sousa
2021-06-09 9:03 ` Xiaoliang Yang
2021-06-09 9:03 ` Xiaoliang Yang
2021-06-16 17:12 ` Rui Sousa
2021-06-16 17:12 ` Rui Sousa
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=202106011813.ZhUFgEvu-lkp@intel.com \
--to=lkp@intel.com \
--cc=alexandre.torgue@st.com \
--cc=boon.leong.ong@intel.com \
--cc=davem@davemloft.net \
--cc=joabreu@synopsys.com \
--cc=kbuild-all@lists.01.org \
--cc=kuba@kernel.org \
--cc=mcoquelin.stm32@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=peppe.cavallaro@st.com \
--cc=vee.khee.wong@intel.com \
--cc=weifeng.voon@intel.com \
--cc=xiaoliang.yang_1@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: link
Be 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.