[PATCH 15/16] ARM: ux500: Add regulator support for nomadik-i2c into the db8500 Device Tree

[Linus Walleij]

On Tue, May 29, 2012 at 12:50 PM, Lee Jones <lee.jones at linaro.org> wrote:
> [Me]
>> I'm told not to put DT bindings for things in place if they are
>> incorrect, since they get set in stone... and I'm told we should
>> probably remove this one regulator from the device rather.
>> I'm just uncertain.
>
> Nothing is set in stone. I'm always happy to do clean-ups and code
> corrections. :)

So assume this one device tree gets deployed, and some poor windows
phone developer start scratching his head about the strange voltage
domain regulator for the I2C device, when in reality there is no such
thing, there is only an APE voltage domain regulator common for all
these devices.

So with device trees the fit between physical reality and what we
model in the kernel needs to be as close as possible or we will
cause trouble.

That's the implications it will have unless we get it right, and that's
why I'm hesitant.

Yours,
Linus Walleij