Gradually build up is fine, but that doesn't mean to do it one way first andchange it later. This only results in feedback like mine - I2C=y justdidn't make sense in the previous patch. Please keep in mind that doingthings one way first and then changing them later only wastes people's time.It would be much better to mention in the previous patch that the symbol isdeclared bool because a subsequent patch introduces an interrupt controllerdriver which needs to be built-in.

Thanks,Guenter

>> It does explain the I2C=y, but I really think it should be bool from>> the start if it ends up>> there.> > Ok.> >>>>> depends on I2C=y>>> depends on OF>>> select REGMAP_I2C>>> select REGMAP_IRQ>>> - select SL28CPLD_IRQ>>> + select SL28CPLD_INTC>>>> What is the point of introducing SL28CPLD_IRQ in the first place ?> > oh damn. this is a left-over which slipped through. There should just> be a SL28CPLD_INTC.> > -michael> >>>>> select MFD_CORE>>> help>>> This option enables support for the board management controller>>>