From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 07961606A31C for ; Mon, 9 Nov 2026 20:59:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=UbQNeIjJ; dkim=fail reason="signature verification failed" (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=EeDIV6bs 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:MIME-Version:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Unsubscribe-Post:List-Owner; bh=gqB2wA6lZzwU30HdpAfDwrf1sf+CopajN/+H+ctEATg=; b=UbQNeIjJV71e52OdiMs/6sI2IU 44cwJPxGajCLDZdJzLnuDC6w/hIeMUZBd6KHx+ibUXS5mb62A7XqlHa2nvKhwCU+yPIMCTUKJb729 fK1bPawyMx6RrLJq8ZZZf9WdznN7p7fOu1RReJQUq9z9RsJcE9Ozw58hHRrhvBgNrwFNfGY64HmZy xU0gLhdpKnqbpstnNYdnuZ8mjIr9xzxQz3AtxtJq4on4uhaDcJCYCb0VPLzQYtCp42ghBl15NA1oe hots+356BIogFtMvALCotDeVYR6LuHrl5hGE5I00KC4YZ4BhtKQooZgA1oh0pSKOjRq1lMA1Bz2KI 7N8U9Ivg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xQWSR-00000001iEO-2CmT; Mon, 09 Nov 2026 20:59:15 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xQWSP-00000001iDy-3ltK; Mon, 09 Nov 2026 20:59:13 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id 0C47660F2F; Mon, 9 Nov 2026 20:59:13 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 878741F0089F; Mon, 9 Nov 2026 20:59:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794257952; bh=OKIk6rnFZ6VmNTZhn1YkBpy13BJSr4y4E6v2+iMZxUI=; h=Date:From:To:Cc:Subject:In-Reply-To; b=EeDIV6bseeGWOEYh7Znh4Kj8Zq8ivM4dw9jyixYC9Xos0t6GtWEMpY2uV6MzziW9s bIk/BUTpKRhLcRwesJmKQ8a9WrqxuoWL4kig82NAIqXZ/hPbAu5iQPzwlAcF5Fn/i0 GjrMWYmyIy4Hddv0PSM0HVrqHfGV4MDg86b7KrT+we5f3LX+rfot199K5+eLwNdXKj ljDlYfOBgFYxiFM7GwouzusMmJz6eu4QZxN316/1fistBQhgNZa1YRWYobJMynxU02 2JFIqLc7tSWBLm/b5lhmPEdEz7Vwtai3Pp2eysShu9dXkFo7+GExsCtyiVFU4PC8ch pq53+t1iZ99qw== Date: Mon, 9 Nov 2026 14:59:11 -0600 From: Bjorn Helgaas To: Igor Velkov Cc: Shawn Lin , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Manivannan Sadhasivam , Bjorn Helgaas , Heiko Stuebner , Rob Herring , linux-pci@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] PCI: rockchip-host: Enable async probe by default Message-ID: <20261109205911.GA292554@bhelgaas> X-Archived-By: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20261109204122.async-probe-reply@iav.lv> X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" X-Spam-Status: No, score=-3.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 On Mon, Nov 09, 2026 at 10:47:39PM +0200, Igor Velkov wrote: > On Mon, Nov 09, 2026 at 02:22:02PM -0600, Bjorn Helgaas wrote: > > I don't know what the SD card has to do with PCIe. Seems like there's > > some SD issue, and making PCIe async happens to avoid it? Sounds > > fragile to me. > > > > Other changes like c0e1eb441b1d justify it by allowing other drivers > > to load in parallel to reduce boot time, which is a good reason. But > > apparently here it makes a functional difference as well, which feels > > sketchy. E.g., what other drivers need to use async probe to avoid > > similar issues? > > You're right that the real problem is ordering: nothing makes dw_mmc wait > for the RK3399 io-domain driver, which sets the 1.8 V level of the SD > pins. [1] tried to add that dependency and was not merged. Async probe > only stops this driver from holding the deferred-probe worker for ~0.7 s; > any other slow probe in that window breaks UHS the same way. > > Is there a way to express this dependency that you would accept? > > [1] https://lore.kernel.org/all/20230904115816.1237684-1-s.hauer@pengutronix.de/ Mani's already applied it, so I think it's fine. I'm just trying to understand the commit message and figure out what the merge commit log and the eventual pull request should say. It sounds like "this makes SD cards work better" is somehow part of it, but it just feels kind of hand-wavey. _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip