[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150525173244.GE21577@sirena.org.uk>
Date: Mon, 25 May 2015 18:32:44 +0100
From: Mark Brown <broonie@...nel.org>
To: Tomeu Vizoso <tomeu.vizoso@...labora.com>
Cc: linux-arm-kernel@...ts.infradead.org,
Stéphane Marchesin
<stephane.marchesin@...il.com>,
Thierry Reding <thierry.reding@...il.com>,
Dmitry Torokhov <dmitry.torokhov@...il.com>,
Alexander Holler <holler@...oftware.de>,
Grant Likely <grant.likely@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Liam Girdwood <lgirdwood@...il.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 10/21] regulator: core: Probe regulators on demand
On Mon, May 25, 2015 at 04:53:14PM +0200, Tomeu Vizoso wrote:
> When looking up a regulator through its DT node, ensure that the
> corresponding device has been registered.
I'm sorry but I'm going to need a much better changelog to review this
patch. I can't entirely tell what the intended effect is supposed to
be, the subject line says we're probing things and the body says we're
registering things which aren't the same thing at all.
> if (node) {
> + of_platform_device_ensure(node);
This function doesn't appear in git so I'm guessing that some other
patch in the series adds it. See my reply to the first patch...
The obvious questions here based on the name are why we're doing
something specific to platform devices and why this isn't something
we're abstracting in the driver core (or at least generic firmware code)
- we're going to have the same thing with ACPI.
Download attachment "signature.asc" of type "application/pgp-signature" (474 bytes)
Powered by blists - more mailing lists