[PATCH v2 1/3] hwmon: (lm90) Add power control
From: broonie@kernel.org (Mark Brown)
Date: 2013-08-08 19:28:11
Also in:
linux-tegra, lkml
On Thu, Aug 08, 2013 at 11:30:30AM -0600, Stephen Warren wrote:
On 08/08/2013 02:42 AM, Guenter Roeck wrote:
quoted
If the regulator subsystem is not configured, you especially don't need or want to pollute the log with an error message.
DT parsing errors should be reported. However, if there's nothing to parse, it's not an error.
So, I think this should report an error message *if* there is a DT property that defines the regulator to use. If there's no property, just use no regulator. If there is a property, it had better be possible to parse it.
On a DT system you should get this behaviour simply by paying attention to the error code from the regualtor API - the core should complain loudly if the DT is incomprehensible. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130808/5d4dd604/attachment.sig>