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 CDB63C44507 for ; Wed, 15 Jul 2026 11:56:18 +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-Type:MIME-Version: References:In-Reply-To:Subject:Cc:To:From:Message-ID:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=u9wJHKiV23P2GLbMmzVOeIGyReKDaO4AJFuZast6R08=; b=YJOZYbFtp7IwKRH1QHrDCApXtI niZ/UOMi0JXx5SLR3+9BrLf+h3cHo9qjPGAPN0gITT3rBGZGuNB1nvLgKC6yqZpg2tQ3+t1IUmdtA 1/pnQrFCq/ALAqrGy9v9ukwcI6GvVxhQ3pFo+Em36jQuQvcqNEbM1DbYJl8ngx9vr++LlBBOcWoM6 MuAchPJ7+WDW4GIAMxJwbJbD3SmnQ+b2FK2NWldEHrJlbnflUaqrSGQSQQMl4bc5dx7dxRA1P1Ttl DUuf3uN9CDoyBubzpJNGNzfxsIjp2FKmTWyLuS5eivaYbl91SL/DnCiRiLhu2+xWuFnZLzjiZSTxP q4dBYhsQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wjyDk-0000000EeTY-1daU; Wed, 15 Jul 2026 11:56:12 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wjyDj-0000000EeTA-0tPy for linux-arm-kernel@lists.infradead.org; Wed, 15 Jul 2026 11:56:11 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 94B5D42A26; Wed, 15 Jul 2026 11:56:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 715131F000E9; Wed, 15 Jul 2026 11:56:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1784116570; bh=u9wJHKiV23P2GLbMmzVOeIGyReKDaO4AJFuZast6R08=; h=Date:From:To:Cc:Subject:In-Reply-To:References; b=hWcnDRifBQOBuXKk7Y87WvSAYB0GUR0mBPWzDtZQ8cwFCRiSAFRIGymSi3j7BdxaX YF8ONpbBKamJiY7YUrkPSfoxN3wxywi1v6xeb+O5TBzRUZ3PTfWUgx4IE4bZ1PTjq0 L+hw3UdfX3PZc3q7shYUzh9hEV+mXKLiYKjjDg8f7b7NndG6K/IESn64N1XNmNNFvD cSb5FEAFpitcYYFqTpuPoywBnJM9uuxEKSMses2GU6y5JG77b7+B/yCcpAp4plhXv5 hH6RuiJsk8RoQqWR01c26lGBDuJFyz088Q8tmYiw7QUiEIZFnsMntWimisVumy6R/V Uy8ipp1x1fdqg== Received: from sofa.misterjones.org ([185.219.108.64] helo=goblin-girl.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1wjyDg-00000005H0X-19Tp; Wed, 15 Jul 2026 11:56:08 +0000 Date: Wed, 15 Jul 2026 12:56:07 +0100 Message-ID: <86bjc8o4a0.wl-maz@kernel.org> From: Marc Zyngier To: Yureka Lilian Cc: Manivannan Sadhasivam , Lorenzo Pieralisi , Krzysztof =?UTF-8?B?V2lsY3p5xYRza2k=?= , Rob Herring , Bjorn Helgaas , Sven Peter , Janne Grunau , Neal Gompa , Krzysztof Kozlowski , Conor Dooley , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org Subject: Re: [PATCH 1/2] PCI: apple: Integrate pwrctrl API In-Reply-To: <76243051-e391-45d2-b47a-d4a904f0a36f@cyberchaos.dev> References: <20260707-apple-pcie-pwren-v1-0-5a281b182fe2@cyberchaos.dev> <20260707-apple-pcie-pwren-v1-1-5a281b182fe2@cyberchaos.dev> <86a4s3p2vy.wl-maz@kernel.org> <8a80a170-a508-4a9e-8090-27cfcedb1cd4@cyberchaos.dev> <865x2qq4ts.wl-maz@kernel.org> <76243051-e391-45d2-b47a-d4a904f0a36f@cyberchaos.dev> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/30.1 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: yureka@cyberchaos.dev, mani@kernel.org, lpieralisi@kernel.org, kwilczynski@kernel.org, robh@kernel.org, bhelgaas@google.com, sven@kernel.org, j@jannau.net, neal@gompa.dev, krzk+dt@kernel.org, conor+dt@kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false 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 Tue, 14 Jul 2026 17:57:11 +0100, Yureka Lilian wrote: > > > On 7/9/26 14:29, Manivannan Sadhasivam wrote: > > On Tue, Jul 07, 2026 at 02:58:39PM +0100, Marc Zyngier wrote: > >> Works fine is one thing. Being consistent with the way the rest of the > >> driver works is another. pci_pwrctrl_create_device() and > >> pci_pwrctrl_power_on_device() appear to do exactly what would be > >> required for a single port. They just needs to be exported made > >> global/exported. > >> > > I'm fine with exporting pci_pwrctrl_create_device() and > > pci_pwrctrl_power_on_device() and calling them with per-port np. > > > > - Mani > > In theory I do agree this would work to power on the slots, and I can > send a v2 of the patch doing it that way, but I don't quite understand > yet how this fits better with the existing driver code: If this is the > case, what is the difference between pcie-apple and pcie-qcom and why > should pcie-qcom use the recursive function but pcie-apple not? Consistency is important to maintaining any piece of code, and I do not want to deal with two management models simultaneously. Furthermore, the qcom stuff is bolted on the side the DWC train wreck, has to deal with the existing way the DWC driver works, and I see no reason to be dragged down to that level of horror. Yes, this is an arbitrary decision. Thanks, M. -- Without deviation from the norm, progress is not possible.