On 7/31/08, Grant Likely <[EMAIL PROTECTED]> wrote: > If you're careful, the table doesn't need to be huge. It can be > marked as initdata and conditionally compiled depending on which > architectures are compiled in. You should use .data in the driver's > of_device_id table to provide machine specific ops for setting > clocking to avoid a maze of if/else statements.
Does this look ok for the mpc5200 i2c struct? /* I2C Registers */ struct mpc52xx_i2c { u8 madr; /* I2C + 0x00 */ u8 reserved1[3]; /* I2C + 0x01 */ u8 mfdr; /* I2C + 0x04 */ u8 reserved2[3]; /* I2C + 0x05 */ u8 mcr; /* I2C + 0x08 */ u8 reserved3[3]; /* I2C + 0x09 */ u8 msr; /* I2C + 0x0c */ u8 reserved4[3]; /* I2C + 0x0d */ u8 mdr; /* I2C + 0x10 */ u8 reserved5[15]; /* I2C + 0x11 */ u8 interrupt; /* I2C + 0x20 */ u8 reserved6[3]; /* I2C + 0x21 */ u8 mifr; /* I2C + 0x24 */ }; -- Jon Smirl [EMAIL PROTECTED] _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev