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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2EB7FC433F5 for ; Wed, 3 Nov 2021 21:53:40 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id EAF9460E08 for ; Wed, 3 Nov 2021 21:53:39 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org EAF9460E08 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From: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=yg56PabeKpLtNhDXl5voVW8b8s6ey5by9vNb51bvJeg=; b=wCmSMg1O9TND52ebhlj5Yt1QTl KG9MS5XYdgbt8civOlJ3DdBnXfD/euifcpvP22Dn6HT1etoh5n0pHJuvp3bVaw6LZzxcudFEEMYq1 Sm/g1r6RCBKippLJGZWoiEbxPvEk7yW4JXvm3QpbzeNwqDFbQ4yu7Uk2A6sXdFCZC8QBxMMmPlfFu gtpwLsWPaDHTDE/MnBwhAhj1CXUfzVQFPyN/w4MFTzXaLZ6PJjPMDBrDWhDtDxYgs1sNDcPd1PAkW WJewFZknGQmwvfPQQoU1MBapI/5It48SGLxM/JIq1A9bO8Z7lF6jf6n2gCjYrNA0YP9CJEW5PMOtO bTn5+qTA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1miOBI-006jej-3X; Wed, 03 Nov 2021 21:52:28 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1miOBF-006jdr-3Q; Wed, 03 Nov 2021 21:52:26 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id ECC73610EA; Wed, 3 Nov 2021 21:52:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1635976344; bh=V/b4RPe+KlKMwlq2NVIwE02LwYXgoYqfcsIIQPEMlik=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GXRdL45mPMhb9fHhdFwxq1r9oH2XDVICseRyZZ0nq1Q5YtLD56tMiVwffbDB9hsgB //QND5nBe77QrgUkzGrfF+3W09PjkvHo0Eiyf2RCOUgSgrBDbrBzuon85T8JVbMM6b Wbi2twOSQSlG+662LO05WulrrUApWF33skbei5Mtv5FPTEU3G4MJ9Dm9Vt3QehkbTc Hb8NvLuXUS8yBYpzhF3vh9ofblDU4anXGJWRS0OflhfTg3C82Dsbnd034nq+ZWjjl4 lYzD+0eMzTyCQq+HFWVWMQhYDNHOCBsvzzcp7Bs7NS7cnTFzYECOsAIlEf09PA0Wqd g6/+jbhMgolgg== Date: Wed, 3 Nov 2021 21:52:18 +0000 From: Mark Brown To: Jim Quinlan Cc: Jim Quinlan , linux-pci@vger.kernel.org, Nicolas Saenz Julienne , Rob Herring , bcm-kernel-feedback-list@broadcom.com, Florian Fainelli , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , "moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE" , "moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE" , open list Subject: Re: [PATCH v7 5/7] PCI: brcmstb: Add control of subdevice voltage regulators Message-ID: References: <20211103184939.45263-1-jim2101024@gmail.com> <20211103184939.45263-6-jim2101024@gmail.com> MIME-Version: 1.0 In-Reply-To: X-Cookie: Thank God I'm an atheist. X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211103_145225_191792_3DD3D502 X-CRM114-Status: GOOD ( 13.61 ) 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: , Content-Type: multipart/mixed; boundary="===============6243344752328020000==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6243344752328020000== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="9AXClGSRiYH1zTCu" Content-Disposition: inline --9AXClGSRiYH1zTCu Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Nov 03, 2021 at 04:34:34PM -0400, Jim Quinlan wrote: > On Wed, Nov 3, 2021 at 4:25 PM Jim Quinlan wrote: > > I did it to squelch the "supply xxxxx not found, using dummy > > regulator" output. I'll change it. > Now I remember: if I know there are no vpciexxx-supplly props in the > DT, I can skip executing all of the buik regulator calls entirely, as > well as walking the PCI bus as in brcm_regulators_off(). > Do you consider this a valid reason? No, the whole point in the core code providing dummy supplies is that it removes the complexity introduced by client drivers trying to guess if there's supplies available or not. If they do that then we end up with a bunch of code duplication and issues if there's any changes or extensions to the generic bindings. --9AXClGSRiYH1zTCu Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmGDBJEACgkQJNaLcl1U h9Dkjwf/QMk7OC27Jh/s6EzEAKCqj6WwCPuFlh1Vz8bufodISNcXsKUqGWTxJNiM FjoP//ePLiCy2DaXo4KIrgPREdIDAPBiIWvO+NvSH9ZBLu/SiAJXj0hAHyhkmdz+ wPKuJJ/ybkVCclDPe7ulcMIaScGg/MD5fDmRDHEenY0NPK5RO9IXVMlNQCxMIoQW /0L9Zmxy8jHWUJXXKP53eIHngmuwVvnaQxQDHNFy7gl+HCe0VE7EcmTRDXamTKeO P6Oh3U5o/XfhQEeTtBYS9X9ZhHTGc/S+/Biwg2hoi5JmSVb3ypoIvjRLXZwub2FL rBXMLpU9bR+q553FS0AzhAZJYGGZcQ== =seGl -----END PGP SIGNATURE----- --9AXClGSRiYH1zTCu-- --===============6243344752328020000== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6243344752328020000==--