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 8E5B1C982D0 for ; Thu, 17 Sep 2026 20:28:46 +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:Subject:References:In-Reply-To:Message-Id:Cc:To:From: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=dQSnXPOErAUYlxrzO2RZ74MubmylXYaCQrdeyKrYUi0=; b=OD+yARAT+mMHTzmZuWNyxy392c U3rPesFbXy5tidu88xFK8aWcBO2RAjjkxHGMI73/pQZroWEbsllxZIQiFGd7F1IPDnpL3A27Hyb7t GUVCwr5VOmxbgCoAHbgigv1OA0SApgoJik+eqVCL/5Yz8Jc2cKME8E4FPJU4pnI7b00CQSp9Kgcrk jRxiylMPcfftsh+z2Ce9WxQh/pwIF58sfIcl7Z+x0+ue6Dxa/XWkwnNySOcTyX5jGCkPORt3PtJWn AJ2h5Sga7nKh97ToIoytQoXUFU6IT4JNgBrPQ0L4/QGXqwscIW/lw051YWHCa2TMOAW1Y1oS239eS cQRZtXvg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x7Iim-0000000CQXG-07xK; Thu, 17 Sep 2026 20:28:40 +0000 Received: from fout-b2-smtp.messagingengine.com ([202.12.124.145]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x7Iii-0000000CQWi-1zYq for linux-arm-kernel@lists.infradead.org; Thu, 17 Sep 2026 20:28:38 +0000 Received: from ams-compute-02.internal (ams-compute-02.internal [10.64.2.62]) by mailfout.stl.internal (Postfix) with ESMTP id 8117C1D001AF; Thu, 17 Sep 2026 16:28:30 -0400 (EDT) Received: from ams-imap-03 ([10.64.2.23]) by ams-compute-02.internal (MEProxy); Thu, 17 Sep 2026 16:28:32 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm3; t=1789676909; x=1789763309; bh=dQSnXPOErAUYlxrzO2RZ74MubmylXYaCQrdeyKrYUi0=; b= Ea3MteqdaDCBlN04NxRlJlkODSOOYD+CK6GsrzkpMI7mi3M1QxTSIm6BnogsqSF2 W902iaE/+wZQyGuL/RsXQ3faHAtBGHqtauHRvYQ1PZmGG7JfrkUN1Y8yTJiqBfRa 1En0OpIP5V54+iWVJvrQz8ryzXoQKT/nHh6xg1gVIflUJvIp/b2NrlV8wTwK+Q7T FDtQCCU6VppN2rN1emglOAVq+/YAat3BS2c02LArBdqFTxr+mZo3kRNxQGQ6KheL lw8YUkvmd8KrpwdDMiPbF1oRNt5mk8KOpI8E6Y7aZESj73fY1pyeVSSTMKa6wfnc Ouvf5p7TlOPoyNdRM796Yg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1789676909; x= 1789763309; bh=dQSnXPOErAUYlxrzO2RZ74MubmylXYaCQrdeyKrYUi0=; b=K N8LLazJ+p+3TRA33qKr2KsVcSuuEVcKUCN8mNmLteGWh+lZwYrDpPJci8WDxuhZW KiZ92bBws7yGrAgOdFkjSd50jkmIT328cJ/OtB/LVN1awCZcdfOQzam8G7lMQAUs VlKW99pvcVfDSBi6MrBSjDmijQAM9DFLzfHmiHeiu3ui1IVVb1ievhTGjNCI9v2Z lEi0ipNmDibND25LutHGkgebXAP6n4oOZRULN5Y4aojP6LQXVgsJswXVEU3PoatL UtqNL8tHcBx/lW0z7s74FpGrrolqMVkkGtK2PP+QgXE/T8qiZDAJ2LQ6Gu6BXhXv fLML3vOOikyXiUJB/k3hg== X-ME-Sender: X-ME-Proxy-Cause: dmFkZTFRv9h6v7nu+lxcC4g6+Rx4j3wacmqFRZ5xl2uHzVCxCgxkdidoTnjLkraylrTKIw Kfewlt0nvNtJDjt78HcdE5Zs+n5mFUcVSdVdEMfUXLYCOX9FgGhzm+Z1Zni31uw1M/bKS8 hk3T07yUDXfnaq+7Ta6YNC0mMZCIsCOjeCm3tCosnxdHFfF1mVaV8Z6UziSPjmvozwVNec DWh7FpRs0Lc0Z0nXPcE1lbHC63Amak8WUHmoToeq2uMPI/PlpCxc/H/KB53Z7GMPX3oPRk nySy385vR5QBgq+rRS6mNKz6Rg/RBWTdy8maYn1ZlymNF17r3NiMeLueOspzW9rJBuywPD wI8CSI2dgyzBVm3n08bUxoO8WrTw4enFSbGoym75e7Pj/FY7qVht7A13t37gQwS28DxqaG fHe9TD+oqqEH6rgEIigCTEgRZQS4iv/Mvn5MshvajnjWrXgVt45lIqlFOIP582Gm0dfs2P eiir43Ama9RWuHitH89+m8ock1fFKAtED+TDBjlCauTcX7Aj5+/PDuBfr2p0ahYTaRuPVY EFPWkUXhOMSeYaYqb7qH6emycJ3ofk9UfSCOtzByKyWxYfqTAAWyC8ez6tCAvKhlKps02l C/Yf2Qi/URm9NWBjJQI+zAIZ44ng9kgZ2e6tDWvfVxjGb3ZDD6ICVDHqs/BQ X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.ams.internal (Postfix, from userid 501) id 8E27B32A0084; Thu, 17 Sep 2026 16:28:24 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface MIME-Version: 1.0 X-ThreadId: A1v-knPnKEnN Date: Thu, 17 Sep 2026 22:27:54 +0200 From: "Arnd Bergmann" To: "Bjorn Helgaas" , "Semih Baskan" Cc: "Lorenzo Pieralisi" , =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= , "Manivannan Sadhasivam" , "Rob Herring" , bhelgaas@google.com, "Ray Jui" , "Scott Branden" , bcm-kernel-feedback-list@broadcom.com, =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= , =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= , "Florian Fainelli" , linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, "Rosen Penev" , rani.hod@gmail.com Message-Id: <7c2a8582-280e-471a-b4b0-037b2cda6ad8@app.fastmail.com> In-Reply-To: <20260917183602.GA1036092@bhelgaas> References: <20260917183602.GA1036092@bhelgaas> Subject: Re: [PATCH] PCI: iproc: Use pci_alloc_host_bridge() on BCMA Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260917_132836_953003_EA27431C X-CRM114-Status: GOOD ( 14.92 ) 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 On Thu, Sep 17, 2026, at 20:36, Bjorn Helgaas wrote: > On Thu, Sep 17, 2026 at 09:02:23PM +0300, Semih Baskan wrote: >> On Thu, Sep 17, 2026 at 10:59:29AM -0500, Bjorn Helgaas wrote: > > Then it seems like we should remove the "brcm,iproc-pcie" compatible > from DTs for platforms where bcma can enumerate it via EROM. Those > platforms can enable CONFIG_PCIE_IPROC_BCMA. They still need the PCIe > controller node in DT so they can use the wifi nodes below it, but if > there's no "brcm,iproc-pcie" compatible, pcie-iproc-platform shouldn't > do anything. > > Arnd, Rafa=C5=82, jump in if I'm in the weeds here. It feels wrong to me to change the devicetee file when that arguably describes the device correctly, and we still need the node for the bcma bus probe in the end. Normally, there should be a list of compatible strings so the driver can make a more informed decision, e.g. the platform driver could skip a device if it identifies the compatible string as one that can be probed using the bcma bus when that driver is also enabled. Unfortunately, it appears that all of these only have the generic "brcm,iproc-pcie" string (or only the paxc variant), so that doesn't work here. Would it be help to change the probe order so the bcma driver always comes before the platform driver? Arnd