On 17/07/12 14:35, Mark Brown wrote:
On Tue, Jul 17, 2012 at 02:30:01PM +0100, Lee Jones wrote:
On 17/07/12 14:06, Mark Brown wrote:
It's not just about having generic bindings, it's also about having
bindings which have some abstraction and hope of reusability. An awful
lot of bindings are just straight dumps of Linux data structures into
the device tree which don't make a terribly great deal of sense as
bindings.
The Device Tree should supply any platform configuration which the
driver needs in order to correctly setup for a particular machine.
This is exactly what the platform_data structure did before, hence
is is reasonable to assume that whatever information resides in that
structure would be required in the Device Tree.
An *awful* lot of what people are trying to put into platform data is
nothing to do with that, it's just the generic data the driver needs to
be able to understand the hardware at all. Things like the MFD
breakdown, random parameters of the hardware which you can infer from
the device name and so on.
I can only speak from a personal perspective on that one. I do _try_ not
to put anything in there (platform data or Device Tree), which does not
belong. I have no idea how successful that notion was however.
I'm sure sure this is relevant in the current case though, as the i2c
properties proposed here are platform specific. What we're discussing is
some consolidation of property names, which I do support in theory. What
I fear is that this driver will lack Device Tree functionality for yet
another kernel version if it isn't resolved quickly.
Wolfram, are you theorising about these the multiple use of these
properties, or do you have some examples? I think we would do well to
work on these, or else they're just going to lie dormant and not get done.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
M: +44 77 88 633 515
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/