[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 10 Oct 2014 12:53:59 +0100
From: Mark Brown <broonie@...nel.org>
To: Grant Likely <grant.likely@...aro.org>
Cc: PERIER Romain <romain.perier@...il.com>,
devicetree <devicetree@...r.kernel.org>,
Liam Girdwood <lgirdwood@...il.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Heiko Stübner <heiko@...ech.de>,
"sameo@...ux.intel.com" <sameo@...ux.intel.com>,
Lee Jones <lee.jones@...aro.org>, robh <robh@...nel.org>,
Sebastian Reichel <sre@...nel.org>,
Dmitry Eremin-Solenikov <dbaryshkov@...il.com>,
David Woodhouse <dwmw2@...radead.org>,
Anton Vorontsov <anton@...msg.org>,
Laxman Dewangan <ldewangan@...dia.com>,
"linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>,
"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>
Subject: Re: [RFC PATCH v2 1/4] regulator: Add helper function to get
"poweroff-source" property
On Fri, Oct 10, 2014 at 12:47:08PM +0100, Grant Likely wrote:
> This is an extremely simple wrapper. It may be best to simply make it
> a static inline. It is also generic enough that I don't have a problem
> with it living in include/linux/of.h; but of_regulator.h would be fine
> too.
I think of.h might be a better idea - it's not something that needs to
be regulator specific, system power controllers could provide the same
functionality but may not offer any regulator control to Linux.
Download attachment "signature.asc" of type "application/pgp-signature" (474 bytes)
Powered by blists - more mailing lists