1 Mar
2011
1 Mar
'11
9:17 p.m.
Mark Brown wrote:
static int __init cs4270_init(void) {
- pr_info("Cirrus Logic CS4270 ALSA SoC Codec Driver\n");
- return i2c_add_driver(&cs4270_i2c_driver);
}
I'm not fond of this at all.
Can I get a justification for this change? On my boards, the CS4270 driver is compiled in-kernel, not as a module, and so when the kernel boots, the driver announces itself when it's loaded. I like this a lot, which is why I always put a pr_info() in the __init function of my modules.
--
Timur Tabi
Linux kernel developer at Freescale