Hi Aradhya, Thank you for the patch! Yet something to improve: [auto build test ERROR on clk/clk-next] [also build test ERROR on linus/master v6.2 next-20230303] [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/Aradhya-Bhatia/dt-bindings-clock-Add-binding-documentation-for-TI-AM62-DSS-Clock/20230213-200203 base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next patch link: https://lore.kernel.org/r/20230213115954.553-3-a-bhatia1%40ti.com patch subject: [PATCH v2 2/2] clk: keystone: Add support AM62 DSS clock divider config: arm-randconfig-r046-20230302 (https://download.01.org/0day-ci/archive/20230304/202303040131.k7AUxAg5-lkp@xxxxxxxxx/config) compiler: arm-linux-gnueabi-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/2ddb7301b11092b73d4a16af06362954f3d5f714 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Aradhya-Bhatia/dt-bindings-clock-Add-binding-documentation-for-TI-AM62-DSS-Clock/20230213-200203 git checkout 2ddb7301b11092b73d4a16af06362954f3d5f714 # 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=arm olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/clk/ mm// net/ If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Link: https://lore.kernel.org/oe-kbuild-all/202303040131.k7AUxAg5-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): In file included from arch/arm/include/asm/div64.h:107, from include/linux/math.h:6, from include/linux/math64.h:6, from include/linux/time.h:6, from include/linux/stat.h:19, from include/linux/module.h:13, from drivers/clk/keystone/clk-am62-dss.c:5: drivers/clk/keystone/clk-am62-dss.c: In function 'ti_am62_dss_clk_recalc_rate': include/asm-generic/div64.h:222:35: warning: comparison of distinct pointer types lacks a cast 222 | (void)(((typeof((n)) *)0) == ((uint64_t *)0)); \ | ^~ drivers/clk/keystone/clk-am62-dss.c:24:9: note: in expansion of macro 'do_div' 24 | do_div(rate, priv->div); | ^~~~~~ In file included from include/linux/build_bug.h:5, from include/linux/container_of.h:5, from include/linux/list.h:5, from include/linux/module.h:12: include/asm-generic/div64.h:234:32: warning: right shift count >= width of type [-Wshift-count-overflow] 234 | } else if (likely(((n) >> 32) == 0)) { \ | ^~ include/linux/compiler.h:77:45: note: in definition of macro 'likely' 77 | # define likely(x) __builtin_expect(!!(x), 1) | ^ drivers/clk/keystone/clk-am62-dss.c:24:9: note: in expansion of macro 'do_div' 24 | do_div(rate, priv->div); | ^~~~~~ include/asm-generic/div64.h:238:36: error: passing argument 1 of '__div64_32' from incompatible pointer type [-Werror=incompatible-pointer-types] 238 | __rem = __div64_32(&(n), __base); \ | ^~~~ | | | long unsigned int * drivers/clk/keystone/clk-am62-dss.c:24:9: note: in expansion of macro 'do_div' 24 | do_div(rate, priv->div); | ^~~~~~ arch/arm/include/asm/div64.h:24:45: note: expected 'uint64_t *' {aka 'long long unsigned int *'} but argument is of type 'long unsigned int *' 24 | static inline uint32_t __div64_32(uint64_t *n, uint32_t base) | ~~~~~~~~~~^ drivers/clk/keystone/clk-am62-dss.c: In function 'clk_hw_unregister_am62_dss_clk': >> drivers/clk/keystone/clk-am62-dss.c:99:9: error: implicit declaration of function 'kfree'; did you mean 'kvfree'? [-Werror=implicit-function-declaration] 99 | kfree(priv); | ^~~~~ | kvfree cc1: some warnings being treated as errors vim +99 drivers/clk/keystone/clk-am62-dss.c 91 92 static void clk_hw_unregister_am62_dss_clk(struct clk_hw *hw) 93 { 94 struct ti_am62_dss_clk *priv; 95 96 priv = to_ti_am62_dss_clk(hw); 97 98 clk_hw_unregister(hw); > 99 kfree(priv); 100 } 101 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests
