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 14508CCFA05 for ; Fri, 7 Nov 2025 10:28:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:References:To:From:Subject: Cc:Message-Id:Date:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=7QoIJKFxiJPfoncWqx8yT2nDk5qmOifjKtASflaey4c=; b=MLox18838RaRJ7 Yynmgmo9AF7sy1DYDtOi0+C/g/Vs8BMtT589dLuF+ONess8gtUsRqGG9ZkJNrUtdd+2fj813+QJn6 ZQRABaPkXZhhAfheONc2SY4YwdvWfg3CRjsqo1fMEJ4xnsQBH8Gq85ptaatv1XdsZxwdqVkMzOucG 2rEihDB/bNCDMB5Z6Cr6k55SjdB+kYcK05u013DRe8cuoS2a7pw+ERFjR72bpkJtDnu5jnP3ooJ2B kNR65FtqyUcegc9PFOTsXqEGI7dFo3AmbXX54rLsznPO/Ia/oU5mnIGb+9XifYTi24Ntf3YpLQ4Rm Hv8+hrdA7pDNod1qktMw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vHJhJ-0000000H4Ye-1EYX; Fri, 07 Nov 2025 10:28:01 +0000 Received: from out-176.mta1.migadu.com ([95.215.58.176]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vHJhF-0000000H4YC-2DXP for linux-rockchip@lists.infradead.org; Fri, 07 Nov 2025 10:27:59 +0000 Mime-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cknow-tech.com; s=key1; t=1762511271; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=WXxxLdb7XheOy75RFTu6qpzQ1CR6/qzHUiLBzTJbIfo=; b=LTXY4niJ2Q4v9blFZDDIDMhmT6YuANss37XVrUPjaWkBRYbGPvukGIrKWMjfMtjCz9X4y2 D0icX131qTv+kScD53gosbMyEh9nKGD7IdwIhnbkx9kmiVcrc3fXJVV8w0StZO3YlSuV6u N0fDkWwwzBQPhuNam3hUStygGtLEykiDTBETx/DGwquha8F2AIUK55iJQQjrfXkvH8ri19 JWvm9c3kuu8nOS8c0V9jSfOMNKS9NziiYCuireCSED3VE6TV87pkiPaoZ/tU7QArHk1qe1 UkRkKnx8n4UE05b8wzlOpUbvsSK+LAJmaMEErMTaPXGlZ7HCyBphXEv2Ab2MQw== Date: Fri, 07 Nov 2025 11:27:45 +0100 Message-Id: Cc: "Bjorn Helgaas" , , "Shawn Lin" , "Lorenzo Pieralisi" , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , "Manivannan Sadhasivam" , "Rob Herring" , "Bjorn Helgaas" , "Heiko Stuebner" , , , , , "Krzysztof Kozlowski" , "Conor Dooley" , "Johan Jonker" Subject: Re: [RFC PATCH 2/2] PCI: rockchip-host: drop wait on PERST# toggle X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: "Diederik de Haas" To: "Geraldo Nascimento" References: <20251103181038.GA1814635@bhelgaas> In-Reply-To: X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251107_022758_149579_B3FE9551 X-CRM114-Status: GOOD ( 15.15 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Hi Geraldo, On Wed Nov 5, 2025 at 10:22 PM CET, Geraldo Nascimento wrote: > On Wed, Nov 05, 2025 at 10:06:53AM +0100, Diederik de Haas wrote: >> I have a Samsung PM981 (without the 'a') and AFAICT it works fine. >> I had noticed that the PERST# (pinctrl) was missing, but 'ep-gpios' >> was/is new to me and I hadn't had time to research that properly yet. >> Good to see you already found it yourself :-) > > Would you mind testing the following DT change to make sure your PM981 > continues to work fine? I just learned the the PCIe system on rk3399 is indeed different from the systems where I use it with (rk3568 & rk3588). And 'ep-gpios' is only used with rk3399 based devices (in the Rockchip dts tree), which explains why I hadn't seen that before. Which in turn means I can't test your proposed change. I guess I was triggered by RFC patch 2 which said 'a known quirky device' while my Samsung PM981's are working fine ... but with DW based IP. You did specify in your cover letter the connection with Rockchip PCI IP, which apparently can make a (lot of?) difference. Me finding the PERST# pinctrl in a few minutes and we finding improvements in RockPro64's PCI 'config' recently, indicated to me that a new look into the dts definition may be warranted, before changing ``drivers/pci/controller/pcie-rockchip-host.c`` for everyone. Cheers, Diederik > Thank you, > Geraldo Nascimento > > --- > > diff --git a/arch/arm64/boot/dts/rockchip/rk3399pro-vmarc-som.dtsi b/arch/arm64/boot/dts/rockchip/rk3399pro-vmarc-som.dtsi > index aa70776e898a..b3d19dce539f 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3399pro-vmarc-som.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk3399pro-vmarc-som.dtsi > @@ -383,7 +383,7 @@ &pcie_phy { > }; > > &pcie0 { > - ep-gpios = <&gpio0 RK_PB4 GPIO_ACTIVE_HIGH>; > + ep-gpios = <&gpio0 RK_PB4 (GPIO_ACTIVE_HIGH | GPIO_SINGLE_ENDED)>; > num-lanes = <4>; > pinctrl-0 = <&pcie_clkreqnb_cpm>; > pinctrl-names = "default"; _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip