From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCHv8 4/5] twl4030: add support for external voltage get/set Date: Tue, 14 Feb 2012 11:16:23 -0800 Message-ID: <87fwed6xa0.fsf@ti.com> References: <1323444589-19940-1-git-send-email-t-kristo@ti.com> <1323444589-19940-5-git-send-email-t-kristo@ti.com> <871ur8weds.fsf@ti.com> <20120110033012.GM30766@opensource.wolfsonmicro.com> <87pqertwjt.fsf@ti.com> <20120110191012.GI7164@opensource.wolfsonmicro.com> <1329223666.4102.319.camel@sokoban> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from na3sys009aog122.obsmtp.com ([74.125.149.147]:60516 "EHLO na3sys009aog122.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757012Ab2BNTQa (ORCPT ); Tue, 14 Feb 2012 14:16:30 -0500 Received: by mail-pz0-f47.google.com with SMTP id n34so207574dal.6 for ; Tue, 14 Feb 2012 11:16:29 -0800 (PST) In-Reply-To: <1329223666.4102.319.camel@sokoban> (Tero Kristo's message of "Tue, 14 Feb 2012 14:47:46 +0200") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: t-kristo@ti.com Cc: Mark Brown , Benoit Cousson , linux-omap@vger.kernel.org, Liam Girdwood , Samuel Ortiz Tero Kristo writes: > On Tue, 2012-01-10 at 19:10 +0000, Mark Brown wrote: >> On Tue, Jan 10, 2012 at 07:19:18AM -0800, Kevin Hilman wrote: >> >> > Yes, some of the control still goes via the normal path (although I >> > forget which, maybe Benoit can remind us), so I think it's best to add >> > the HW control part to each regulator that might uses it. >> >> > Ideally this could be facilitated by adding the extentions to the >> > regulator core so the amount of code needed for each regulator driver >> > would be minimal. >> >> I think the original version of the patch was something along those >> lines but it was just a general facility which ignored the regulator >> driver entirely which didn't feel well integrated. The discussion >> suggested that this wasn't something that'd work with other regulators >> so a per-driver solution seemed OK. > > Coming back to this patch now as I have time to look at it, what is the > general opinion, is it acceptable to patch the regulator core to add > support for the external controller or should I just resend the latest > version with changes Mark suggested? This will probably mean that once > we add new regulator drivers (e.g. pmics) we may need to duplicate the > external controller support here. How about you keep it in the regulator driver for now, and when we need to abstract it out, we make the case for it then. Kevin