From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B1BFEC4332F for ; Wed, 24 Nov 2021 10:47:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241591AbhKXKu4 (ORCPT ); Wed, 24 Nov 2021 05:50:56 -0500 Received: from mga18.intel.com ([134.134.136.126]:41222 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240440AbhKXKuz (ORCPT ); Wed, 24 Nov 2021 05:50:55 -0500 X-IronPort-AV: E=McAfee;i="6200,9189,10177"; a="222129462" X-IronPort-AV: E=Sophos;i="5.87,260,1631602800"; d="scan'208";a="222129462" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Nov 2021 02:47:46 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,260,1631602800"; d="scan'208";a="607149526" Received: from lkp-server02.sh.intel.com (HELO 9e1e9f9b3bcb) ([10.239.97.151]) by orsmga004.jf.intel.com with ESMTP; 24 Nov 2021 02:47:43 -0800 Received: from kbuild by 9e1e9f9b3bcb with local (Exim 4.92) (envelope-from ) id 1mppoU-0004fD-HG; Wed, 24 Nov 2021 10:47:42 +0000 Date: Wed, 24 Nov 2021 18:46:49 +0800 From: kernel test robot To: Bjorn Andersson , Michael Turquette , Stephen Boyd , Taniya Das , Vinod Koul Cc: kbuild-all@lists.01.org, linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] clk: qcom: clk-alpha-pll: Don't reconfigure running Trion Message-ID: <202111241836.A2WvIKic-lkp@intel.com> References: <20211123161630.123222-1-bjorn.andersson@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211123161630.123222-1-bjorn.andersson@linaro.org> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Bjorn, I love your patch! Yet something to improve: [auto build test ERROR on clk/clk-next] [also build test ERROR on v5.16-rc2 next-20211124] [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/Bjorn-Andersson/clk-qcom-clk-alpha-pll-Don-t-reconfigure-running-Trion/20211124-001628 base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next config: m68k-allmodconfig (https://download.01.org/0day-ci/archive/20211124/202111241836.A2WvIKic-lkp@intel.com/config) compiler: m68k-linux-gcc (GCC) 11.2.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/0day-ci/linux/commit/1c6539db17125d4d4eaf17c4071063fe8a7e2ca6 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Bjorn-Andersson/clk-qcom-clk-alpha-pll-Don-t-reconfigure-running-Trion/20211124-001628 git checkout 1c6539db17125d4d4eaf17c4071063fe8a7e2ca6 # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=m68k If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): drivers/clk/qcom/clk-alpha-pll.c: In function 'clk_trion_pll_configure': >> drivers/clk/qcom/clk-alpha-pll.c:1437:17: error: implicit declaration of function 'pr_dbg'; did you mean 'pr_debug'? [-Werror=implicit-function-declaration] 1437 | pr_dbg("Trion PLL is already enabled, skipping configuration\n"); | ^~~~~~ | pr_debug cc1: some warnings being treated as errors vim +1437 drivers/clk/qcom/clk-alpha-pll.c 1421 1422 /** 1423 * clk_lucid_pll_configure - configure the lucid pll 1424 * 1425 * @pll: clk alpha pll 1426 * @regmap: register map 1427 * @config: configuration to apply for pll 1428 */ 1429 void clk_trion_pll_configure(struct clk_alpha_pll *pll, struct regmap *regmap, 1430 const struct alpha_pll_config *config) 1431 { 1432 /* 1433 * If the bootloader left the PLL enabled it's likely that there are 1434 * RCGs that will lock up if we disable the PLL below. 1435 */ 1436 if (trion_pll_is_enabled(pll, regmap)) { > 1437 pr_dbg("Trion PLL is already enabled, skipping configuration\n"); 1438 return; 1439 } 1440 1441 clk_alpha_pll_write_config(regmap, PLL_L_VAL(pll), config->l); 1442 regmap_write(regmap, PLL_CAL_L_VAL(pll), TRION_PLL_CAL_VAL); 1443 clk_alpha_pll_write_config(regmap, PLL_ALPHA_VAL(pll), config->alpha); 1444 clk_alpha_pll_write_config(regmap, PLL_CONFIG_CTL(pll), 1445 config->config_ctl_val); 1446 clk_alpha_pll_write_config(regmap, PLL_CONFIG_CTL_U(pll), 1447 config->config_ctl_hi_val); 1448 clk_alpha_pll_write_config(regmap, PLL_CONFIG_CTL_U1(pll), 1449 config->config_ctl_hi1_val); 1450 clk_alpha_pll_write_config(regmap, PLL_USER_CTL(pll), 1451 config->user_ctl_val); 1452 clk_alpha_pll_write_config(regmap, PLL_USER_CTL_U(pll), 1453 config->user_ctl_hi_val); 1454 clk_alpha_pll_write_config(regmap, PLL_USER_CTL_U1(pll), 1455 config->user_ctl_hi1_val); 1456 clk_alpha_pll_write_config(regmap, PLL_TEST_CTL(pll), 1457 config->test_ctl_val); 1458 clk_alpha_pll_write_config(regmap, PLL_TEST_CTL_U(pll), 1459 config->test_ctl_hi_val); 1460 clk_alpha_pll_write_config(regmap, PLL_TEST_CTL_U1(pll), 1461 config->test_ctl_hi1_val); 1462 1463 regmap_update_bits(regmap, PLL_MODE(pll), PLL_UPDATE_BYPASS, 1464 PLL_UPDATE_BYPASS); 1465 1466 /* Disable PLL output */ 1467 regmap_update_bits(regmap, PLL_MODE(pll), PLL_OUTCTRL, 0); 1468 1469 /* Set operation mode to OFF */ 1470 regmap_write(regmap, PLL_OPMODE(pll), PLL_STANDBY); 1471 1472 /* Place the PLL in STANDBY mode */ 1473 regmap_update_bits(regmap, PLL_MODE(pll), PLL_RESET_N, PLL_RESET_N); 1474 } 1475 EXPORT_SYMBOL_GPL(clk_trion_pll_configure); 1476 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org