From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v3 0/3] Qualcomm Resource Power Manager driver Date: Wed, 18 Jun 2014 08:53:40 -0700 Message-ID: <7hzjha8b97.fsf@paris.lan> References: <1402944372-31901-1-git-send-email-bjorn.andersson@sonymobile.com> <7hvbrzbh1u.fsf@paris.lan> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: (Bjorn Andersson's message of "Tue, 17 Jun 2014 10:15:10 -0700") Sender: linux-arm-msm-owner@vger.kernel.org To: Bjorn Andersson Cc: Lee Jones , Bjorn Andersson , Rob Herring , Mark Rutland , Liam Girdwood , Mark Brown , Kumar Gala , Josh Cartwright , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , linux-arm-msm , Paul Walmsley List-Id: devicetree@vger.kernel.org Bjorn Andersson writes: > On Tue, Jun 17, 2014 at 10:07 AM, Kevin Hilman wrote: >> +Paul Walmsley >> >> Bjorn Andersson writes: >> >>> This series adds a regulator driver for the Resource Power Manager found in >>> Qualcomm 8660, 8960 and 8064 based devices. >>> >>> The RPM driver exposes resources to its child devices, that can be accessed to >>> implement drivers for the regulators, clocks and bus frequency control that's >>> owned by the RPM in these devices. >>> >>> Changes since v2: >>> - Fix copy-paste error in dt binding >>> - Correct incomplete move from mfd to soc >>> - Correct const mistake in regulator driver >>> >>> Changes since v1: >>> - Moved rpm driver to drivers/soc >> >> I'm not sure I follow the motivation for having this under drivers/soc? >> > Hi Kevin, > > I've made the argument that to me this is conceptually a black box > handling regulators, clocks and other stuff; hence similar to a PMIC, > which would fit nicely into drivers/mfd. > > I still think this is the case and now that I look back I didn't get > any pushback from Lee Jones so maybe the move was premature? Yes, IMO, the move was premature, but hopefully the drivers/soc folks can chime in an clarify the criteria for inclusion there. Kevin