From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752568AbdAYTTp (ORCPT ); Wed, 25 Jan 2017 14:19:45 -0500 Received: from foss.arm.com ([217.140.101.70]:43434 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752351AbdAYTTn (ORCPT ); Wed, 25 Jan 2017 14:19:43 -0500 Date: Wed, 25 Jan 2017 19:21:35 +0000 From: Lorenzo Pieralisi To: Mark Brown Cc: Mark Rutland , Furquan Shaikh , "Rafael J. Wysocki" , "Rafael J . Wysocki" , Liam Girdwood , Tony Lindgren , Dmitry Torokhov , Len Brown , Greg Kroah-Hartman , Hanjun Guo , Will Deacon , Rob Herring , Sathyanarayana Nujella , Heikki Krogerus , Adam Thomson , Linus Walleij , Alexandre Courbot , linux-gpio@vger.kernel.org, ACPI Devel Maling List , Linux Kernel Mailing List , Linux OMAP Mailing List , Aaron Durbin , dlaurie@chromium.org Subject: Re: [PATCH 0/7] Implement generic regulator constraints parsing for ACPI and OF Message-ID: <20170125192135.GA8605@red-moon> References: <20170125000641.25520-1-furquan@chromium.org> <20170125124911.dxveow7bo3zw5jcc@sirena.org.uk> <20170125182319.GB25470@leverpostej> <20170125182955.uznr5ehpx4dabcon@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170125182955.uznr5ehpx4dabcon@sirena.org.uk> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 25, 2017 at 06:29:55PM +0000, Mark Brown wrote: > On Wed, Jan 25, 2017 at 06:23:20PM +0000, Mark Rutland wrote: > > On Wed, Jan 25, 2017 at 08:56:42AM -0800, Furquan Shaikh wrote: > > > > That is the reason why the recent change to add ACPI support to fixed > > > regulators was done > > > (https://github.com/torvalds/linux/blob/master/drivers/regulator/fixed.c#L100). > > > To be honest, I'm surprised this got merged. > > My understanding was that it was instantiated from another device as an > implementation detail of that device, letting it say "this GPIO should > be handled as a regulator". > > > Mark, this was added in this cycle; can we please rip that out for now? > > If it's instantiated directly we probably should. > > > We can certainly come up with something that allows drivers to support > > both, but trying to do this without updating drivers opens a huge set of > > problems. > > I think there's a reasonable chance that any ACPI specs could be written > in such a way as to allow transparent support in Linux, the main thing > I'd worry about is naming issues. I think that the difference between ACPI and DT firmware models, in particular in relation to power states handling (and what piece of SW is in charge of power management) is significant and goes beyond naming conventions, therefore the code (and reasoning behind it - ie to have an identical driver interface to a completely different FW model) in this series is just not acceptable, that's a plain shortcut. We will see how this should be implemented in ACPI, not with this code (and FW bindings). Thanks, Lorenzo