Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932896AbbLPBUM (ORCPT ); Tue, 15 Dec 2015 20:20:12 -0500 Received: from ring0.de ([5.45.101.7]:59539 "EHLO ring0.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752985AbbLPBUJ (ORCPT ); Tue, 15 Dec 2015 20:20:09 -0500 X-Greylist: delayed 521 seconds by postgrey-1.27 at vger.kernel.org; Tue, 15 Dec 2015 20:20:08 EST X-Spam-Report: * -0.0 NO_RELAYS Informational: message was not relayed via SMTP * -1.9 BAYES_00 BODY: Spamwahrscheinlichkeit nach Bayes-Test: 0-1% * [score: 0.0000] * -0.7 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's * domain * -0.7 DKIM_VALID Message has at least one valid DKIM or DK signature * 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily * valid * -0.0 NO_RECEIVED Informational: message has no Received headers X-Spam-Relay-Country: Date: Wed, 16 Dec 2015 02:11:11 +0100 From: Sebastian Reichel To: Eric Anholt Cc: Florian Fainelli , "Rafael J. Wysocki" , linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Stephen Warren , Lee Jones , Kevin Hilman , Ulf Hansson , Greg Kroah-Hartman , Alexander Aring , devicetree@vger.kernel.org, linux-pm@vger.kernel.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell Subject: Re: [PATCH v3 2/4] ARM: bcm2835: add rpi power domain driver Message-ID: <20151216011110.GA13147@earth> References: <1450215622-27394-1-git-send-email-eric@anholt.net> <1450215622-27394-3-git-send-email-eric@anholt.net> <567093CD.1020208@gmail.com> <87oadrmfmi.fsf@eliezer.anholt.net> <5670AAB2.1080102@gmail.com> <87mvtbkyd0.fsf@eliezer.anholt.net> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="82I3+IH0IqGh5yIs" Content-Disposition: inline In-Reply-To: <87mvtbkyd0.fsf@eliezer.anholt.net> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3008 Lines: 78 --82I3+IH0IqGh5yIs Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Tue, Dec 15, 2015 at 04:53:31PM -0800, Eric Anholt wrote: > >>> What motivated the location of this power domain driver in > >>> arch/arm/mach-bcm? Should not we have this in drivers/power/ or > >>> somewhere in drivers/ at the very least? > >>=20 > >> ls stronly suggests that power contains drivers for power supplies and > >> batteries, not power domains. Indeed it's used for fuel gauges and chargers, but also for reboot/powerdown and adaptive voltage scaling, so another subdirectory for power-domains wouldn't be that odd. > >> There are 6 power domain drivers in > >> arch/arm, 3 in drivers/clk, and 3 in drivers/soc. > > > > If we ever have to support a different architecture which happens to use > > a similar power domain, then we want it to be in a location which makes > > it easy for sharing it in the first place. As it stands today, it does > > not seem useful to me to have this code in arch/arm/mach-bcm/ at all. > > > > Maybe there is room from a drivers/power/domains/ of some kind? I like the idea, but let's include generic power domain maintainers in this discussion, as I suggested here (I got a power domain driver patch for drivers/power just a few days ago): https://lkml.org/lkml/2015/12/15/815 Also somebody would have to step up to maintain that directory. > The great thing about git is that moving code is easy, even after it's > first committed. > > The subsystem maintainer didn't comment on the code's location in v1 or > v2, and I think they probably have the final say on that. Whatever they > want, where there is currently a genpd driver, is fine with me. sounds reasonable. P.S.: Thanks for taking bringing RPI support upstream :) -- Sebastian --82I3+IH0IqGh5yIs Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBCgAGBQJWcLosAAoJENju1/PIO/qayHQP/3LA9qewnqfA4VNNZiM2JGSL G3oMD9a7lLRgIwTa5H+SKewdbzsXMeR/uNRfw+wt+jj6WDzfNopDK/zjW8IbHCIC qpS/bLX1XyeE1Jdkecusn2U3OazTnPH174z8KzfURReM+WMyLyp6WA4zuD9GIqSJ BF9yGFBqaaAxNnnDx4z+ZzIqFLVseyAsQKY13AjxU9NbLtugRnpN8h8VAwdGiLIG 36e4aWwcroSqsfguP8Jb10tgGN29PWaRFk93zOAW2Hs3w82Fg4YYZHz9sXJr/5Ue 2IguacUAbjg+rKu4Imyx1PqZo4AbH+L/7hwIwZ/nTtb15povcKCHMO009JIS086B B4XFyRUEWHFUpUOQQgnS1xKdkC+56TAb2WZUQyclz1jWnqQ9Tvqw6MnUUvoaBIuS hleChC52E0ALbxHhGyyY7FB1HsuYRaTGO2G8cKKICbVkrTCiDAdc2DFOL2694vZW St6ww2FaT6zcdjHxA1eWE7HzY9hQZ4CBCaJS+jBA5RjzwSaRwZ48c93ceqme6AVK zbUavziTYoMo8jTf7I+d4XtwH+Fnsa3H4Eoh+ZPpK+RQA5g8JCqA6iVMHwxnQ/R9 d4DfiRlrSvyEIa/AeT7fbcnRoa6D7hOe2yub1YlUuZYRha8oNZHl0EBaZXZZv4Cw SMctL1g8DOvX9oa59VGM =Z91h -----END PGP SIGNATURE----- --82I3+IH0IqGh5yIs-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/