[kbuild-all] [intel-linux-intel-lts:5.4/yocto 63/66] drivers/ptp/ptp-intel-pmc-tgpio.c:388:6: warning: variable 'new_start_ns' is used uninitialized whenever 'if' condition is false

tree: https://github.com/intel/linux-intel-lts.git 5.4/yocto head: 7e5be1eaf9b5088cda67df5351c2c12572db341b commit: 357fc48f29d865a0760168fd13f28f512487eb00 [63/66] drivers/ptp: Add PMC Time-Aware GPIO Driver config: x86_64-randconfig-r023-20200811 (attached as .config) compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 4f2ad15db535873dda9bfe248a2771023b64a43c) 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 # install x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu git checkout 357fc48f29d865a0760168fd13f28f512487eb00 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All warnings (new ones prefixed by >>): if (on || perout->flags & PTP_PEROUT_ONE_SHOT) { ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/ptp/ptp-intel-pmc-tgpio.c:396:26: note: uninitialized use occurs here (tgpio, perout->index, new_start_ns, new_period_ns, ^~~~~~~~~~~~ drivers/ptp/ptp-intel-pmc-tgpio.c:388:2: note: remove the 'if' if its condition is always true if (on || perout->flags & PTP_PEROUT_ONE_SHOT) { ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/ptp/ptp-intel-pmc-tgpio.c:385:2: note: variable 'new_start_ns' is declared here struct timespec64 new_start_ns; ^ if (on || perout->flags & PTP_PEROUT_ONE_SHOT) { ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/ptp/ptp-intel-pmc-tgpio.c:396:40: note: uninitialized use occurs here (tgpio, perout->index, new_start_ns, new_period_ns, ^~~~~~~~~~~~~ drivers/ptp/ptp-intel-pmc-tgpio.c:388:2: note: remove the 'if' if its condition is always true if (on || perout->flags & PTP_PEROUT_ONE_SHOT) { ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/ptp/ptp-intel-pmc-tgpio.c:386:2: note: variable 'new_period_ns' is declared here struct timespec64 new_period_ns; ^ static const struct acpi_device_id intel_pmc_acpi_match[] = { ^ 3 warnings generated. vim +388 drivers/ptp/ptp-intel-pmc-tgpio.c 381 382 static int intel_pmc_tgpio_config_output 383 (struct intel_pmc_tgpio_t *tgpio, struct ptp_perout_request *perout, int on) 384 { 385 struct timespec64 new_start_ns; 386 struct timespec64 new_period_ns; 387 389 new_start_ns = ptp_clock_time_to_ts64(perout->start); 390 new_period_ns = ptp_clock_time_to_ts64(perout->period); 391 new_period_ns = ktime_to_timespec64 392 (ktime_divns(timespec64_to_ktime(new_period_ns), 2)); 393 } 394 395 return _intel_pmc_tgpio_config_output 396 (tgpio, perout->index, new_start_ns, new_period_ns, 397 perout->flags, on); 398 } 399 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org