Messages in this thread | | | Date | Mon, 31 Jan 2022 12:15:07 +0800 | From | kernel test robot <> | Subject | Re: [PATCH 1/5] clk: mux: Declare u32 *table parameter as const |
| |
Hi "Jonathan,
I love your patch! Yet something to improve:
[auto build test ERROR on clk/clk-next] [also build test ERROR on linux/master linus/master v5.17-rc2 next-20220128] [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]
url: https://github.com/0day-ci/linux/commits/Jonathan-Neusch-fer/clk-Declare-mux-tables-as-const-u32/20220129-175243 base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next config: arm-randconfig-r004-20220131 (https://download.01.org/0day-ci/archive/20220131/202201311204.ggE2FHa7-lkp@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project f1c18acb07aa40f42b87b70462a6d1ab77a4825c) 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 arm cross compiling tool for clang build # apt-get install binutils-arm-linux-gnueabi # https://github.com/0day-ci/linux/commit/b97ffeed127cccf9159b9de1e9a1527b963460c4 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Jonathan-Neusch-fer/clk-Declare-mux-tables-as-const-u32/20220129-175243 git checkout b97ffeed127cccf9159b9de1e9a1527b963460c4 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/clk/nxp/clk-lpc18xx-cgu.c:460:6: warning: variable 'stat' set but not used [-Wunused-but-set-variable] u32 stat, ctrl; ^ >> drivers/clk/nxp/clk-lpc18xx-cgu.c:545:37: error: passing 'const u32 *' (aka 'const unsigned int *') to parameter of type 'u32 *' (aka 'unsigned int *') discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers] lpc18xx_fill_parent_names(parents, clk->mux.table, clk->n_parents); ^~~~~~~~~~~~~~ drivers/clk/nxp/clk-lpc18xx-cgu.c:526:65: note: passing argument to parameter 'id' here static void lpc18xx_fill_parent_names(const char **parent, u32 *id, int size) ^ drivers/clk/nxp/clk-lpc18xx-cgu.c:567:37: error: passing 'const u32 *' (aka 'const unsigned int *') to parameter of type 'u32 *' (aka 'unsigned int *') discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers] lpc18xx_fill_parent_names(parents, clk->mux.table, clk->n_parents); ^~~~~~~~~~~~~~ drivers/clk/nxp/clk-lpc18xx-cgu.c:526:65: note: passing argument to parameter 'id' here static void lpc18xx_fill_parent_names(const char **parent, u32 *id, int size) ^ drivers/clk/nxp/clk-lpc18xx-cgu.c:592:37: error: passing 'const u32 *' (aka 'const unsigned int *') to parameter of type 'u32 *' (aka 'unsigned int *') discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers] lpc18xx_fill_parent_names(parents, clk->mux.table, clk->n_parents); ^~~~~~~~~~~~~~ drivers/clk/nxp/clk-lpc18xx-cgu.c:526:65: note: passing argument to parameter 'id' here static void lpc18xx_fill_parent_names(const char **parent, u32 *id, int size) ^ 1 warning and 3 errors generated.
vim +545 drivers/clk/nxp/clk-lpc18xx-cgu.c
b04e0b8fd5443b Joachim Eastwood 2015-05-28 533 b04e0b8fd5443b Joachim Eastwood 2015-05-28 534 static struct clk *lpc18xx_cgu_register_div(struct lpc18xx_cgu_src_clk_div *clk, b04e0b8fd5443b Joachim Eastwood 2015-05-28 535 void __iomem *base, int n) b04e0b8fd5443b Joachim Eastwood 2015-05-28 536 { b04e0b8fd5443b Joachim Eastwood 2015-05-28 537 void __iomem *reg = base + LPC18XX_CGU_IDIV_CTRL(n); b04e0b8fd5443b Joachim Eastwood 2015-05-28 538 const char *name = clk_src_names[clk->clk_id]; b04e0b8fd5443b Joachim Eastwood 2015-05-28 539 const char *parents[CLK_SRC_MAX]; b04e0b8fd5443b Joachim Eastwood 2015-05-28 540 b04e0b8fd5443b Joachim Eastwood 2015-05-28 541 clk->div.reg = reg; b04e0b8fd5443b Joachim Eastwood 2015-05-28 542 clk->mux.reg = reg; b04e0b8fd5443b Joachim Eastwood 2015-05-28 543 clk->gate.reg = reg; b04e0b8fd5443b Joachim Eastwood 2015-05-28 544 b04e0b8fd5443b Joachim Eastwood 2015-05-28 @545 lpc18xx_fill_parent_names(parents, clk->mux.table, clk->n_parents); b04e0b8fd5443b Joachim Eastwood 2015-05-28 546 b04e0b8fd5443b Joachim Eastwood 2015-05-28 547 return clk_register_composite(NULL, name, parents, clk->n_parents, b04e0b8fd5443b Joachim Eastwood 2015-05-28 548 &clk->mux.hw, &clk_mux_ops, b04e0b8fd5443b Joachim Eastwood 2015-05-28 549 &clk->div.hw, &clk_divider_ops, c23a5847695dbd Joachim Eastwood 2015-10-24 550 &clk->gate.hw, &lpc18xx_gate_ops, 0); b04e0b8fd5443b Joachim Eastwood 2015-05-28 551 } b04e0b8fd5443b Joachim Eastwood 2015-05-28 552
--- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
| |