From: kernel test robot <lkp@intel.com>
To: Jacob Keller <jacob.e.keller@intel.com>, netdev@vger.kernel.org
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
Jacob Keller <jacob.e.keller@intel.com>,
"K. Y. Srinivasan" <kys@microsoft.com>,
Haiyang Zhang <haiyangz@microsoft.com>,
Stephen Hemminger <sthemmin@microsoft.com>,
Wei Liu <wei.liu@kernel.org>, Dexuan Cui <decui@microsoft.com>,
Tom Lendacky <thomas.lendacky@amd.com>,
Shyam Sundar S K <Shyam-sundar.S-k@amd.com>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Siva Reddy Kallam <siva.kallam@broadcom.com>,
Prashant Sreedharan <prashant@broadcom.com>,
Michael Chan <mchan@broadcom.com>,
Yisen Zhuang <yisen.zhuang@huawei.com>,
Salil Mehta <salil.mehta@huawei.com>,
Jesse Brandeburg <jesse.brandeburg@intel.com>,
Tony Nguyen <anthony.l.nguyen@intel.com>,
Tariq Toukan <tariqt@nvidia.com>,
Saeed Mahameed <saeedm@nvidia.com>,
Leon Romanovsky <leon@kernel.org>,
Bryan Whitehead <bryan.whitehead@microchip.com>,
Sergey Shtylyov <s.shtylyov@omp.ru>,
Giuseppe Cavallaro <peppe.cavallaro@st.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Jose Abreu <joabreu@synopsys.com>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Richard Cochran <richardcochran@gmail.com>,
Vivek Thampi <vithampi@vmware.com>
Subject: Re: [net-next 10/14] ptp: lan743x: convert to .adjfine and diff_by_scaled_ppm
Date: Fri, 19 Aug 2022 10:47:20 +0800 [thread overview]
Message-ID: <202208191003.18DiEs1l-lkp@intel.com> (raw)
In-Reply-To: <20220818222742.1070935-11-jacob.e.keller@intel.com>
Hi Jacob,
I love your patch! Yet something to improve:
[auto build test ERROR on 9017462f006c4b686cb1e1e1a3a52ea8363076e6]
url: https://github.com/intel-lab-lkp/linux/commits/Jacob-Keller/ptp-convert-drivers-to-adjfine/20220819-063154
base: 9017462f006c4b686cb1e1e1a3a52ea8363076e6
config: i386-randconfig-a015 (https://download.01.org/0day-ci/archive/20220819/202208191003.18DiEs1l-lkp@intel.com/config)
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project aed5e3bea138ce581d682158eb61c27b3cfdd6ec)
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/intel-lab-lkp/linux/commit/d3c6eac5778f2ce74e7d6d7be90a60f616551718
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jacob-Keller/ptp-convert-drivers-to-adjfine/20220819-063154
git checkout d3c6eac5778f2ce74e7d6d7be90a60f616551718
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/net/ethernet/microchip/lan743x_ptp.c:368:12: error: redefinition of 'lan743x_ptpci_adjfine'
static int lan743x_ptpci_adjfine(struct ptp_clock_info *ptpci, long delta)
^
drivers/net/ethernet/microchip/lan743x_ptp.c:335:12: note: previous definition is here
static int lan743x_ptpci_adjfine(struct ptp_clock_info *ptpci, long scaled_ppm)
^
>> drivers/net/ethernet/microchip/lan743x_ptp.c:386:3: error: use of undeclared identifier 'lan743_rate_adj'; did you mean 'lan743x_rate_adj'?
lan743_rate_adj = (u32)diff;
^~~~~~~~~~~~~~~
lan743x_rate_adj
drivers/net/ethernet/microchip/lan743x_ptp.c:374:6: note: 'lan743x_rate_adj' declared here
u64 lan743x_rate_adj;
^
>> drivers/net/ethernet/microchip/lan743x_ptp.c:388:3: error: use of undeclared identifier 'lan74e_rage_adj'; did you mean 'lan743x_rate_adj'?
lan74e_rage_adj = (u32)diff | PTP_CLOCK_RATE_ADJ_DIR_;
^~~~~~~~~~~~~~~
lan743x_rate_adj
drivers/net/ethernet/microchip/lan743x_ptp.c:374:6: note: 'lan743x_rate_adj' declared here
u64 lan743x_rate_adj;
^
3 errors generated.
vim +386 drivers/net/ethernet/microchip/lan743x_ptp.c
367
368 static int lan743x_ptpci_adjfine(struct ptp_clock_info *ptpci, long delta)
369 {
370 struct lan743x_ptp *ptp =
371 container_of(ptpci, struct lan743x_ptp, ptp_clock_info);
372 struct lan743x_adapter *adapter =
373 container_of(ptp, struct lan743x_adapter, ptp);
374 u64 lan743x_rate_adj;
375 s32 delta_ppb;
376 u64 diff;
377
378 delta_ppb = scaled_ppm_to_ppb(delta);
379 if ((delta_ppb < (-LAN743X_PTP_MAX_FREQ_ADJ_IN_PPB)) ||
380 delta_ppb > LAN743X_PTP_MAX_FREQ_ADJ_IN_PPB) {
381 return -EINVAL;
382 }
383
384 /* diff_by_scaled_ppm returns true if the difference is negative */
385 if (diff_by_scaled_ppm(1ULL << 35, delta, &diff))
> 386 lan743_rate_adj = (u32)diff;
387 else
> 388 lan74e_rage_adj = (u32)diff | PTP_CLOCK_RATE_ADJ_DIR_;
389
390 lan743x_csr_write(adapter, PTP_CLOCK_RATE_ADJ,
391 lan743x_rate_adj);
392
393 return 0;
394 }
395
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-08-19 2:47 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-18 22:27 [net-next 00/14] ptp: convert drivers to .adjfine Jacob Keller
2022-08-18 22:27 ` [net-next 01/14] ptp: add missing documentation for parameters Jacob Keller
2022-08-18 22:27 ` [net-next 02/14] ptp: introduce helpers to adjust by scaled parts per million Jacob Keller
2022-08-18 22:27 ` [net-next 03/14] drivers: convert unsupported .adjfreq to .adjfine Jacob Keller
2022-08-18 22:27 ` [net-next 04/14] ptp_phc: convert to .adjfine and ptp_adj_scaled_ppm Jacob Keller
2022-08-18 22:27 ` [net-next 05/14] ptp_ixp46x: convert to .adjfine and adjust_by_scaled_ppm Jacob Keller
2022-08-18 22:27 ` [net-next 06/14] ptp: tg3: convert to .adjfine and diff_by_scaled_ppm Jacob Keller
2022-08-18 22:27 ` [net-next 07/14] ptp: hclge: convert to .adjfine and adjust_by_scaled_ppm Jacob Keller
2022-08-18 22:27 ` [net-next 08/14] ptp: mlx4: " Jacob Keller
2022-08-18 22:27 ` [net-next 09/14] ptp: mlx5: " Jacob Keller
2022-08-23 15:30 ` Gal Pressman
2022-08-25 6:31 ` Gal Pressman
2022-08-25 21:59 ` Keller, Jacob E
2022-08-18 22:27 ` [net-next 10/14] ptp: lan743x: convert to .adjfine and diff_by_scaled_ppm Jacob Keller
2022-08-19 2:47 ` kernel test robot [this message]
2022-08-21 15:22 ` kernel test robot
2022-08-22 17:10 ` Keller, Jacob E
2022-08-18 22:27 ` [net-next 11/14] ptp: ravb: convert to .adjfine and adjust_by_scaled_ppm Jacob Keller
2022-08-20 19:33 ` Sergey Shtylyov
2022-08-22 17:07 ` Keller, Jacob E
2022-08-18 22:27 ` [net-next 12/14] ptp: stmac: " Jacob Keller
2022-08-18 22:27 ` [net-next 13/14] ptp: cpts: " Jacob Keller
2022-08-18 22:27 ` [net-next 14/14] ptp: xgbe: " Jacob Keller
2022-08-19 16:05 ` Tom Lendacky
2022-08-19 21:04 ` Keller, Jacob E
2022-08-22 20:47 ` Tom Lendacky
2022-08-22 21:11 ` Keller, Jacob E
2022-08-19 7:39 ` [net-next 00/14] ptp: convert drivers to .adjfine Richard Cochran
2022-08-19 20:56 ` Keller, Jacob E
2022-08-23 16:28 ` Richard Cochran
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=202208191003.18DiEs1l-lkp@intel.com \
--to=lkp@intel.com \
--cc=Shyam-sundar.S-k@amd.com \
--cc=alexandre.torgue@foss.st.com \
--cc=anthony.l.nguyen@intel.com \
--cc=bryan.whitehead@microchip.com \
--cc=decui@microsoft.com \
--cc=edumazet@google.com \
--cc=haiyangz@microsoft.com \
--cc=jacob.e.keller@intel.com \
--cc=jesse.brandeburg@intel.com \
--cc=joabreu@synopsys.com \
--cc=kbuild-all@lists.01.org \
--cc=kuba@kernel.org \
--cc=kys@microsoft.com \
--cc=leon@kernel.org \
--cc=llvm@lists.linux.dev \
--cc=mchan@broadcom.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=peppe.cavallaro@st.com \
--cc=prashant@broadcom.com \
--cc=richardcochran@gmail.com \
--cc=s.shtylyov@omp.ru \
--cc=saeedm@nvidia.com \
--cc=salil.mehta@huawei.com \
--cc=siva.kallam@broadcom.com \
--cc=sthemmin@microsoft.com \
--cc=tariqt@nvidia.com \
--cc=thomas.lendacky@amd.com \
--cc=vithampi@vmware.com \
--cc=wei.liu@kernel.org \
--cc=yisen.zhuang@huawei.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.