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 31118C433FE for ; Sat, 27 Nov 2021 11:26:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239009AbhK0LaG (ORCPT ); Sat, 27 Nov 2021 06:30:06 -0500 Received: from mga11.intel.com ([192.55.52.93]:52024 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233200AbhK0L2F (ORCPT ); Sat, 27 Nov 2021 06:28:05 -0500 X-IronPort-AV: E=McAfee;i="6200,9189,10180"; a="233249438" X-IronPort-AV: E=Sophos;i="5.87,269,1631602800"; d="scan'208";a="233249438" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Nov 2021 03:24:51 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,269,1631602800"; d="scan'208";a="498707234" Received: from lkp-server02.sh.intel.com (HELO 9e1e9f9b3bcb) ([10.239.97.151]) by orsmga007.jf.intel.com with ESMTP; 27 Nov 2021 03:24:48 -0800 Received: from kbuild by 9e1e9f9b3bcb with local (Exim 4.92) (envelope-from ) id 1mqvp1-0009Q8-DK; Sat, 27 Nov 2021 11:24:47 +0000 Date: Sat, 27 Nov 2021 19:23:48 +0800 From: kernel test robot To: Felix Fietkau , linux-arm-kernel@lists.infradead.org, Rob Herring , Michael Turquette , Stephen Boyd Cc: kbuild-all@lists.01.org, john@phrozen.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org Subject: Re: [PATCH v4 07/12] clk: en7523: Add clock driver for Airoha EN7523 SoC Message-ID: <202111271912.FCpB3UGa-lkp@intel.com> References: <20211125110738.41028-8-nbd@nbd.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211125110738.41028-8-nbd@nbd.name> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Felix, I love your patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [also build test WARNING on clk/clk-next v5.16-rc2 next-20211126] [cannot apply to linusw-gpio/for-next] [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/Felix-Fietkau/dt-bindings-Add-vendor-prefix-for-Airoha/20211125-200806 base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next config: powerpc64-randconfig-s032-20211126 (https://download.01.org/0day-ci/archive/20211127/202111271912.FCpB3UGa-lkp@intel.com/config) compiler: powerpc64-linux-gcc (GCC) 11.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # apt-get install sparse # sparse version: v0.6.4-dirty # https://github.com/0day-ci/linux/commit/ff9f74e46043aecee2d7373f978a99ce9a3f1e27 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Felix-Fietkau/dt-bindings-Add-vendor-prefix-for-Airoha/20211125-200806 git checkout ff9f74e46043aecee2d7373f978a99ce9a3f1e27 # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/clk/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot sparse warnings: (new ones prefixed by >>) >> drivers/clk/clk-en7523.c:202:27: sparse: sparse: incorrect type in initializer (different address spaces) @@ expected void *np_base @@ got void [noderef] __iomem *base @@ drivers/clk/clk-en7523.c:202:27: sparse: expected void *np_base drivers/clk/clk-en7523.c:202:27: sparse: got void [noderef] __iomem *base >> drivers/clk/clk-en7523.c:206:29: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:206:29: sparse: expected void const volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:206:29: sparse: got void * >> drivers/clk/clk-en7523.c:208:29: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:208:29: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:208:29: sparse: got void * drivers/clk/clk-en7523.c:213:29: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:213:29: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:213:29: sparse: got void * drivers/clk/clk-en7523.c:217:29: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:217:29: sparse: expected void const volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:217:29: sparse: got void * drivers/clk/clk-en7523.c:220:37: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:220:37: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:220:37: sparse: got void * drivers/clk/clk-en7523.c:222:36: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:222:36: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:222:36: sparse: got void * drivers/clk/clk-en7523.c:224:37: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:224:37: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:224:37: sparse: got void * drivers/clk/clk-en7523.c:229:29: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:229:29: sparse: expected void const volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:229:29: sparse: got void * drivers/clk/clk-en7523.c:230:37: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:230:37: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:230:37: sparse: got void * drivers/clk/clk-en7523.c:232:36: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:232:36: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:232:36: sparse: got void * drivers/clk/clk-en7523.c:241:27: sparse: sparse: incorrect type in initializer (different address spaces) @@ expected void *np_base @@ got void [noderef] __iomem *base @@ drivers/clk/clk-en7523.c:241:27: sparse: expected void *np_base drivers/clk/clk-en7523.c:241:27: sparse: got void [noderef] __iomem *base drivers/clk/clk-en7523.c:244:29: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:244:29: sparse: expected void const volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:244:29: sparse: got void * drivers/clk/clk-en7523.c:246:29: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void * @@ drivers/clk/clk-en7523.c:246:29: sparse: expected void volatile [noderef] __iomem *addr drivers/clk/clk-en7523.c:246:29: sparse: got void * vim +202 drivers/clk/clk-en7523.c 198 199 static int en7523_pci_enable(struct clk_hw *hw) 200 { 201 struct en_clk_gate *cg = container_of(hw, struct en_clk_gate, hw); > 202 void *np_base = cg->base; 203 u32 val, mask; 204 205 /* Need to pull device low before reset */ > 206 val = readl(np_base + REG_PCI_CONTROL); 207 val &= ~(REG_PCI_CONTROL_PERSTOUT1 | REG_PCI_CONTROL_PERSTOUT); > 208 writel(val, np_base + REG_PCI_CONTROL); 209 usleep_range(1000, 2000); 210 211 /* Enable PCIe port 1 */ 212 val |= REG_PCI_CONTROL_REFCLK_EN1; 213 writel(val, np_base + REG_PCI_CONTROL); 214 usleep_range(1000, 2000); 215 216 /* Reset to default */ 217 val = readl(np_base + REG_RESET_CONTROL); 218 mask = REG_RESET_CONTROL_PCIE1 | REG_RESET_CONTROL_PCIE2 | 219 REG_RESET_CONTROL_PCIEHB; 220 writel(val & ~mask, np_base + REG_RESET_CONTROL); 221 usleep_range(1000, 2000); 222 writel(val | mask, np_base + REG_RESET_CONTROL); 223 msleep(100); 224 writel(val & ~mask, np_base + REG_RESET_CONTROL); 225 usleep_range(5000, 10000); 226 227 /* Release device */ 228 mask = REG_PCI_CONTROL_PERSTOUT1 | REG_PCI_CONTROL_PERSTOUT; 229 val = readl(np_base + REG_PCI_CONTROL); 230 writel(val & ~mask, np_base + REG_PCI_CONTROL); 231 usleep_range(1000, 2000); 232 writel(val | mask, np_base + REG_PCI_CONTROL); 233 msleep(250); 234 235 return 0; 236 } 237 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org