Re: [PATCH v1 2/2] clocking-wizard: Add support for versal clocking wizard

kernel test robot
2023-04-18T16:58:07Z

[auto build test WARNING on clk/clk-next]
[also build test WARNING on next-20230417]
[cannot apply to xilinx-xlnx/master robh/for-next linus/master v6.3-rc7]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Shubhrajyoti-Datta/dt-bindings-clocking-wizard-add-versal-compatible/20230418-183046
base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
patch link: https://lore.kernel.org/r/20230418102855.6791-3-shubhrajyoti.datta%40amd.com
patch subject: [PATCH v1 2/2] clocking-wizard: Add support for versal clocking wizard
config: arc-randconfig-r043-20230417 (https://download.01.org/0day-ci/archive/20230419/202304190011.BMtXurPJ-lkp@intel.com/config)
compiler: arc-elf-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/143916412aa6a419e20fa37059004329e300ca95
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Shubhrajyoti-Datta/dt-bindings-clocking-wizard-add-versal-compatible/20230418-183046
git checkout 143916412aa6a419e20fa37059004329e300ca95
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arc olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash drivers/clk/xilinx/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202304190011.BMtXurPJ-lkp@intel.com/

All warnings (new ones prefixed by >>):

drivers/clk/xilinx/clk-xlnx-clock-wizard.c: In function 'clk_wzrd_get_divisors':
>> drivers/clk/xilinx/clk-xlnx-clock-wizard.c:102:41: warning: unsigned conversion from 'long long int' to 'long unsigned int' changes value from '4320000000' to '25032704' [-Woverflow]
102 | #define VER_WZRD_VCO_MAX 4320000000
| ^~~~~~~~~~
drivers/clk/xilinx/clk-xlnx-clock-wizard.c:332:26: note: in expansion of macro 'VER_WZRD_VCO_MAX'
332 | vcomax = VER_WZRD_VCO_MAX;
| ^~~~~~~~~~~~~~~~

vim +102 drivers/clk/xilinx/clk-xlnx-clock-wizard.c

88
89 #define WZRD_M_MIN 2
90 #define WZRD_M_MAX 128
91 #define WZRD_D_MIN 1
92 #define WZRD_D_MAX 106
93 #define WZRD_VCO_MIN 800000000
94 #define WZRD_VCO_MAX 1600000000
95 #define WZRD_O_MIN 1
96 #define WZRD_O_MAX 128
97 #define VER_WZRD_M_MIN 4
98 #define VER_WZRD_M_MAX 432
99 #define VER_WZRD_D_MIN 1
100 #define VER_WZRD_D_MAX 123
101 #define VER_WZRD_VCO_MIN 2160000000
> 102 #define VER_WZRD_VCO_MAX 4320000000
103 #define VER_WZRD_O_MIN 2
104 #define VER_WZRD_O_MAX 511
105 #define WZRD_MIN_ERR 20000
106 #define WZRD_FRAC_POINTS 1000
107