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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 42581CCFA1A for ; Mon, 10 Nov 2025 02:30:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:To:Subject:Cc:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ST4GcK2McWrhDhZAl75VtQG+4FHxpdlWOF4ND05GHw4=; b=eEDeCDoCeLBtfY+f5YPww5W3e0 UG1NTN6GKyHELFDBA2yyDHtJs1VoGHMNBVqc1+gWlKSe5H+78I3o170zzFvAFizpP+cLgbwgiI/9J TevHFuAod0SWFlWkMj5nB68n91hsja0b+ynFuDLd12v4qQ2VSbB/txpCRrPoB3PbTrio64uwdPonw gSTcymFKKKzszdCD6Wf/ZSd3w0OsqDyQLRqHGl7nUa7Fj2lv5C5Os2oEfUiEy+O5IknwHfT0TY+bz bKYlVqdNL2x5VpTRiCrbAyXcbIQC0pAUa9q5mRUW/BZDY8aqYDaZCFrLkIExv6F1bLYTMz4twyhs4 Mgmsk6bw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vIHfz-00000004cUI-0q29; Mon, 10 Nov 2025 02:30:39 +0000 Received: from mail-m32122.qiye.163.com ([220.197.32.122]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vIHfv-00000004cTi-2oPO; Mon, 10 Nov 2025 02:30:37 +0000 Received: from [172.16.12.129] (unknown [58.22.7.114]) by smtp.qiye.163.com (Hmail) with ESMTP id 28f6e9869; Mon, 10 Nov 2025 10:30:24 +0800 (GMT+08:00) Message-ID: <780a4209-f89f-43a9-9364-331d3b77e61e@rock-chips.com> Date: Mon, 10 Nov 2025 10:30:20 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: shawn.lin@rock-chips.com, Damien Le Moal , Anand Moon , linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Dragan Simic , Lorenzo Pieralisi , =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= , Rob Herring , Bjorn Helgaas , Heiko Stuebner , mani@kernel.org, Niklas Cassel Subject: Re: [RESEND] Re: [PATCH] PCI: dw-rockchip: Skip waiting for link up To: FUKAUMI Naoki References: <20250113-rockchip-no-wait-v1-1-25417f37b92f@kernel.org> <1E8E4DB773970CB5+5a52c9e1-01b8-4872-99b7-021099f04031@radxa.com> <6e87b611-13ea-4d89-8dbf-85510dd86fa6@rock-chips.com> <55EB0E5F655F3AFC+136b89fd-98d4-42af-a99d-a0bb05cc93f3@radxa.com> From: Shawn Lin In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-HM-Tid: 0a9a6b99abfe09cckunm39d6557412b970a X-HM-MType: 1 X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFDSUNOT01LS0k3V1ktWUFJV1kPCRoVCBIfWUFZQh9IHVZIGBhMSh0aGkJJHxpWFRQJFh oXVRMBExYaEhckFA4PWVdZGBILWUFZTkNVSUlVTFVKSk9ZV1kWGg8SFR0UWUFZT0tIVUpLSEpKQk xVSktLVUpCS0tZBg++ DKIM-Signature: a=rsa-sha256; b=GUxFPHmh5MX8wp7RrOmxB7Fy+w6B+Zbl+TkUp3xCyX6leuKuiZCEFDdiKy0akiHYJJ+ykpT8yYLRVhjtplDwDi210e+zX1I9nI8LBPfoA4nn5T50/H5L3/kwWPOBHjGFG2hrGK534NEh9XfEtJtZdcDhKuiJWgTB2uW9Gc7jEx4=; c=relaxed/relaxed; s=default; d=rock-chips.com; v=1; bh=ST4GcK2McWrhDhZAl75VtQG+4FHxpdlWOF4ND05GHw4=; h=date:mime-version:subject:message-id:from; X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251109_183036_185287_C7BCB7BE X-CRM114-Status: GOOD ( 26.14 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Fukaumi 在 2025/11/10 星期一 7:26, FUKAUMI Naoki 写道: > (RESEND: fix mani's email address) > > Hi Niklas, > > On 11/9/25 21:28, Niklas Cassel wrote: >> On Sun, Nov 09, 2025 at 01:42:23PM +0900, FUKAUMI Naoki wrote: >>> Hi Niklas, >>> >>> On 11/8/25 22:27, Niklas Cassel wrote: >>> (snip)> (And btw. please test with the latest 6.18-rc, as, from >>> experience, >>> the >>>> ASPM problems in earlier RCs can result in some weird problems that are >>>> not immediately deduced to be caused by the ASPM enablement.) >>> >>> Here is dmesg from v6.18-rc4: >>>   https://gist.github.com/RadxaNaoki/40e1d049bff4f1d2d4773a5ba0ed9dff >> >> Same problem as before: >> [    1.732538] pci_bus 0004:43: busn_res: can not insert [bus 43-41] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.732645] pci_bus 0004:43: busn_res: [bus 43-41] end is updated >> to 43 >> [    1.732651] pci_bus 0004:43: busn_res: can not insert [bus 43] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.732661] pci 0004:42:00.0: devices behind bridge are unusable >> because [bus 43] cannot be assigned for them >> [    1.732840] pci_bus 0004:44: busn_res: can not insert [bus 44-41] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.732947] pci_bus 0004:44: busn_res: [bus 44-41] end is updated >> to 44 >> [    1.732952] pci_bus 0004:44: busn_res: can not insert [bus 44] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.732962] pci 0004:42:02.0: devices behind bridge are unusable >> because [bus 44] cannot be assigned for them >> [    1.733134] pci_bus 0004:45: busn_res: can not insert [bus 45-41] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.733246] pci_bus 0004:45: busn_res: [bus 45-41] end is updated >> to 45 >> [    1.733255] pci_bus 0004:45: busn_res: can not insert [bus 45] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.733266] pci 0004:42:06.0: devices behind bridge are unusable >> because [bus 45] cannot be assigned for them >> [    1.733438] pci_bus 0004:46: busn_res: can not insert [bus 46-41] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.733544] pci_bus 0004:46: busn_res: [bus 46-41] end is updated >> to 46 >> [    1.733550] pci_bus 0004:46: busn_res: can not insert [bus 46] >> under [bus 42-41] (conflicts with (null) [bus 42-41]) >> [    1.733560] pci 0004:42:0e.0: devices behind bridge are unusable >> because [bus 46] cannot be assigned for them >> [    1.733571] pci_bus 0004:42: busn_res: [bus 42-41] end is updated >> to 46 >> [    1.733575] pci_bus 0004:42: busn_res: can not insert [bus 42-46] >> under [bus 41] (conflicts with (null) [bus 41]) >> [    1.733585] pci 0004:41:00.0: devices behind bridge are unusable >> because [bus 42-46] cannot be assigned for them >> [    1.733596] pcieport 0004:40:00.0: bridge has subordinate 41 but >> max busn 46 >> >> >> Seems like the ASM2806 switch, for some reason, is not ready. >> >> One change that Diederik pointed out is that in the "good" case, >> the link is always in Gen1 speed. >> >> Perhaps you could build with CONFIG_PCI_QUIRKS=y and try this patch: >> >> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c >> index 214ed060ca1b..ac134d95a97f 100644 >> --- a/drivers/pci/quirks.c >> +++ b/drivers/pci/quirks.c >> @@ -96,6 +96,7 @@ int pcie_failed_link_retrain(struct pci_dev *dev) >>   { >>       static const struct pci_device_id ids[] = { >>           { PCI_VDEVICE(ASMEDIA, 0x2824) }, /* ASMedia ASM2824 */ >> +        { PCI_VDEVICE(ASMEDIA, 0x2806) }, /* ASMedia ASM2806 */ >>           {} >>       }; >>       u16 lnksta, lnkctl2; > > It doesn't help with either probing behind the bridge or the link speed. > >> If that does not work, perhaps you could try this patch >> (assuming that all Rock 5C:s have a ASM2806 on pcie2x1l2): > > ROCK 5C has a PCIe FPC connector and I'm using Dual 2.5G Router HAT. >  https://radxa.com/products/rock5/5c#techspec >  https://radxa.com/products/accessories/dual-2-5g-router-hat > > Regarding the link speed, I initially suspected the FPC connector and/or > cable might be the issue. However, I tried the Dual 2.5G Router HAT with > the ROCK 5A (which uses a different cable), and I got the same result. > > BTW, the link speed varies between 2Gb/s and 4Gb/s depending on the > reboot. (with or without quirk) Could you please help check this patch? --- a/drivers/pci/controller/dwc/pcie-dw-rockchip.c +++ b/drivers/pci/controller/dwc/pcie-dw-rockchip.c @@ -454,6 +454,8 @@ static irqreturn_t rockchip_pcie_rc_sys_irq_thread(int irq, void *arg) struct dw_pcie *pci = &rockchip->pci; struct dw_pcie_rp *pp = &pci->pp; struct device *dev = pci->dev; + struct pci_bus *child, *root_bus = NULL; + struct pci_dev *bridge; u32 reg; reg = rockchip_pcie_readl_apb(rockchip, PCIE_CLIENT_INTR_STATUS_MISC); @@ -462,12 +464,21 @@ static irqreturn_t rockchip_pcie_rc_sys_irq_thread(int irq, void *arg) dev_dbg(dev, "PCIE_CLIENT_INTR_STATUS_MISC: %#x\n", reg); dev_dbg(dev, "LTSSM_STATUS: %#x\n", rockchip_pcie_get_ltssm(rockchip)); + list_for_each_entry(child, &pp->bridge->bus->children, node) { + if (child->parent == pp->bridge->bus) { + root_bus = child; + bridge = root_bus->self; + break; + } + } + if (reg & PCIE_RDLH_LINK_UP_CHGED) { if (rockchip_pcie_link_up(pci)) { msleep(PCIE_RESET_CONFIG_WAIT_MS); dev_dbg(dev, "Received Link up event. Starting enumeration!\n"); /* Rescan the bus to enumerate endpoint devices */ pci_lock_rescan_remove(); + pci_stop_and_remove_bus_device(bridge); pci_rescan_bus(pp->bridge->bus); pci_unlock_rescan_remove(); } > > Best regards, > > -- > FUKAUMI Naoki > Radxa Computer (Shenzhen) Co., Ltd. > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts b/arch/ >> arm64/boot/dts/rockchip/rk3588s-rock-5c.dts >> index dd7317bab613..26f8539d934a 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts >> @@ -452,6 +452,7 @@ &pcie2x1l2 { >>       pinctrl-0 = <&pcie20x1_2_perstn_m0>; >>       reset-gpios = <&gpio3 RK_PD1 GPIO_ACTIVE_HIGH>; >>       vpcie3v3-supply = <&pcie2x1l2_3v3>; >> +    max-link-speed = <1>; >>       status = "okay"; >>   }; >> >> >> >> Kind regards, >> Niklas >> > >