[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
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