From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH v7] regulator: fixed: Convert to use GPIO descriptor only Date: Fri, 12 Oct 2018 11:43:38 +0100 Message-ID: <20181012104338.GS30658@n2100.armlinux.org.uk> References: <20180906122436.25610-1-linus.walleij@linaro.org> <20181011090112eucas1p286d8c1edfc1a2a207d8a11c5ad7eb20e~cglSx9qcr2394623946eucas1p2y@eucas1p2.samsung.com> <1539272073.18645.12.camel@toradex.com> <1539337415.30485.21.camel@toradex.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Jon Hunter Cc: Marcel Ziswiler , "linus.walleij@linaro.org" , "linux-kernel@vger.kernel.org" , "robert.jarzmik@free.fr" , "aaro.koskinen@iki.fi" , "jacopo@jmondi.org" , "m.szyprowski@samsung.com" , "broonie@kernel.org" , "shc_work@mail.ru" , "haojian.zhuang@gmail.com" , "lgirdwood@gmail.com" , "rppt@linux.vnet.ibm.com" , "zonque@gmail.com" , "marc.zyngier@arm.com" , "philipp.zabel@gmail.com" , "linux-tegra@vger.kernel.org" List-Id: linux-tegra@vger.kernel.org On Fri, Oct 12, 2018 at 11:39:15AM +0100, Jon Hunter wrote: > We had the same situation for Tegra124 Jetson TK1 but I don't think that > adding a pseudo intermediate regulator is cleaner. If the GPIO controls > more than one regulator, I don't see why is it necessary to change the > DT. There are several other people reporting the same problem with > various different boards. So this does seem to be a common usage. Given that DT describes the hardware, not the software implementation, it must not change just because we move from GPIO numbers to GPIO descriptors. The existing DT description is reasonable, and introducing ficticious regulators in DT to work around the implementation is not reasonable. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up