8 May
2012
8 May
'12
2:26 p.m.
On Tue, May 08, 2012 at 02:52:25PM +0300, Peter Ujfalusi wrote:
+static const struct of_device_id twl6040_codec_of_match[] = {
- {.compatible = "ti,twl6040-codec", },
- { },
+}; +MODULE_DEVICE_TABLE(of, twl6040_codec_of_match);
Why are we loading MFD components using device tree? It seems like we're doing something very wrong if we need people to explicitly write this stuff out in the device tree, the whole MFD thing is purely a Linux implementation detail, as is the way the interrupt controller has been structured. I'd really not expect to see a specific node like this, especially not one that does nothing but device registration.