Re: [PATCH 1/2] regulator: support operating mode in the device treestructure

From: Mark Brown
Date: Wed May 08 2013 - 07:12:31 EST


On Tue, May 07, 2013 at 11:46:12PM +0000, Kim, Milo wrote:

> It's my intention to guarantee same operations in the DT as the regulator
> constraints in the platform side.
> In case a regulator has specific 'valid_modes_mask', the operating mode is
> unable to be set with the device tree.
> For some regulator drivers such as LP8720/5 and LP8788, this patch would be
> useful because there is no difference between configuration in the platform side
> and the device tree.

I understand what the patch is doing but there's no reason why the
device tree binding needs to map directly onto our internal
configuration and indeed in cases where we didn't make ideal choices
that's not what we want to do. Providing a binding that corresponds to
what the chip itself calls the configuration seems more useful.

Attachment: signature.asc
Description: Digital signature