On Wed, May 6, 2009 at 4:51 PM, Grant Likely <[email protected]> wrote: > On Wed, May 6, 2009 at 4:19 PM, Wolfgang Denk <[email protected]> wrote: >> Dear Grant Likely, >> >> In message <[email protected]> you >> wrote: >>> On Wed, May 6, 2009 at 2:15 PM, Wolfgang Denk <[email protected]> wrote: >>> > From: Piotr Ziecik <[email protected]> >>> > >>> > - Enabled I2C interrupts on MPC5121. >>> > - Updated Kconfig for i2c-mpc driver. >>> >>> I think this workaround belongs in the driver itself. >> >> Sorry, I don't get it. Which workaround? What exactly should I change? > > Sorry, I misread the patch. Never mind.
Actually, on 3rd reading, I think my first impression was correct (even if I was wrong about it being a workaround). This code in mpc512x_init_i2c() is only relevant for i2c busses (it isn't shared with any other drivers). Therefore, it belongs with the i2c bus itself. It does not belong in platform code. Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
