All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
	linux-pci@vger.kernel.org,
	"Krzysztof Wilczyński" <kwilczynski@kernel.org>,
	"Niklas Cassel" <cassel@kernel.org>
Subject: [pci:slot-reset 1/1] drivers/pci/controller/dwc/pcie-dw-rockchip.c:721:58: error: use of undeclared identifier 'PCIE_CLIENT_GENERAL_CON'
Date: Thu, 15 May 2025 23:24:15 +0800	[thread overview]
Message-ID: <202505152337.AoKvnBmd-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git slot-reset
head:   ebf9d2fae99254fc37f49384b769f363e676018d
commit: ebf9d2fae99254fc37f49384b769f363e676018d [1/1] PCI: dw-rockchip: Add support for slot reset on link down event
config: arm64-randconfig-002-20250515 (https://download.01.org/0day-ci/archive/20250515/202505152337.AoKvnBmd-lkp@intel.com/config)
compiler: clang version 21.0.0git (https://github.com/llvm/llvm-project f819f46284f2a79790038e1f6649172789734ae8)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250515/202505152337.AoKvnBmd-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202505152337.AoKvnBmd-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/pci/controller/dwc/pcie-dw-rockchip.c:721:58: error: use of undeclared identifier 'PCIE_CLIENT_GENERAL_CON'
     721 |         rockchip_pcie_writel_apb(rockchip, PCIE_CLIENT_RC_MODE, PCIE_CLIENT_GENERAL_CON);
         |                                                                 ^
   1 error generated.


vim +/PCIE_CLIENT_GENERAL_CON +721 drivers/pci/controller/dwc/pcie-dw-rockchip.c

   679	
   680	static int rockchip_pcie_rc_reset_slot(struct pci_host_bridge *bridge,
   681					       struct pci_dev *pdev)
   682	{
   683		struct pci_bus *bus = bridge->bus;
   684		struct dw_pcie_rp *pp = bus->sysdata;
   685		struct dw_pcie *pci = to_dw_pcie_from_pp(pp);
   686		struct rockchip_pcie *rockchip = to_rockchip_pcie(pci);
   687		struct device *dev = rockchip->pci.dev;
   688		u32 val;
   689		int ret;
   690	
   691		dw_pcie_stop_link(pci);
   692		clk_bulk_disable_unprepare(rockchip->clk_cnt, rockchip->clks);
   693		rockchip_pcie_phy_deinit(rockchip);
   694	
   695		ret = reset_control_assert(rockchip->rst);
   696		if (ret)
   697			return ret;
   698	
   699		ret = rockchip_pcie_phy_init(rockchip);
   700		if (ret)
   701			goto disable_regulator;
   702	
   703		ret = reset_control_deassert(rockchip->rst);
   704		if (ret)
   705			goto deinit_phy;
   706	
   707		ret = rockchip_pcie_clk_init(rockchip);
   708		if (ret)
   709			goto deinit_phy;
   710	
   711		ret = pp->ops->init(pp);
   712		if (ret) {
   713			dev_err(dev, "host init failed: %d\n", ret);
   714			goto deinit_clk;
   715		}
   716	
   717		/* LTSSM enable control mode. */
   718		val = HIWORD_UPDATE_BIT(PCIE_LTSSM_ENABLE_ENHANCE);
   719		rockchip_pcie_writel_apb(rockchip, val, PCIE_CLIENT_HOT_RESET_CTRL);
   720	
 > 721		rockchip_pcie_writel_apb(rockchip, PCIE_CLIENT_RC_MODE, PCIE_CLIENT_GENERAL_CON);
   722	
   723		ret = dw_pcie_setup_rc(pp);
   724		if (ret) {
   725			dev_err(dev, "failed to setup RC: %d\n", ret);
   726			goto deinit_clk;
   727		}
   728	
   729		/* Unmask DLL up/down indicator and hot reset/link-down reset IRQ. */
   730		val = HIWORD_UPDATE(PCIE_RDLH_LINK_UP_CHGED | PCIE_LINK_REQ_RST_NOT_INT, 0);
   731		rockchip_pcie_writel_apb(rockchip, val, PCIE_CLIENT_INTR_MASK_MISC);
   732	
   733		ret = dw_pcie_start_link(pci);
   734		if (ret)
   735			goto deinit_clk;
   736	
   737		/* Ignore errors, the link may come up later. */
   738		dw_pcie_wait_for_link(pci);
   739		dev_dbg(dev, "slot reset completed\n");
   740		return ret;
   741	
   742	deinit_clk:
   743		clk_bulk_disable_unprepare(rockchip->clk_cnt, rockchip->clks);
   744	deinit_phy:
   745		rockchip_pcie_phy_deinit(rockchip);
   746	disable_regulator:
   747		if (rockchip->vpcie3v3)
   748			regulator_disable(rockchip->vpcie3v3);
   749	
   750		return ret;
   751	}
   752	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

             reply	other threads:[~2025-05-15 15:24 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-15 15:24 kernel test robot [this message]
2025-05-15 15:54 ` 回复: [pci:slot-reset 1/1] drivers/pci/controller/dwc/pcie-dw-rockchip.c:721:58: error: use of undeclared identifier 'PCIE_CLIENT_GENERAL_CON' Hans Zhang
2025-05-15 15:54   ` Hans Zhang
2025-05-15 16:24   ` Krzysztof Wilczy��ski
2025-05-15 22:42     ` Wilfred Mallawa
2025-05-16  9:34     ` Niklas Cassel
2025-05-16 10:36       ` Krzysztof Wilczy��ski
2025-05-16 12:29         ` Niklas Cassel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202505152337.AoKvnBmd-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=cassel@kernel.org \
    --cc=kwilczynski@kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=llvm@lists.linux.dev \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=wilfred.mallawa@wdc.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.