Thread (50 messages) 50 messages, 6 authors, 2012-06-12
STALE5110d

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

From: Linus Walleij <hidden>
Date: 2012-05-30 00:09:27

On Tue, May 29, 2012 at 12:50 PM, Lee Jones [off-list ref] wrote:
[Me]
quoted
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help