On Tue, 18 Nov 2008, David Brownell wrote: > --- > drivers/i2c/busses/i2c-gpio.c | 10 ++ > drivers/i2c/i2c-core.c | 155 ++++++++++++++++++++++++++++++++++++++++ > include/linux/i2c.h | 14 +++ > 3 files changed, 179 insertions(+) Can this be made optional? Seeing as nothing uses it yet and it increases a brunch of core structs' sizes. > + smbalert = platform_get_resource_byname(pdev, IORESOURCE_IRQ, > + "smbalert#"); The handler for this irq uses the name "smbus_alert", maybe the resource and the handler should use the same name?