>> static int applesmc_init_smcreg_try(void)>> {>> struct applesmc_registers *s = &smcreg;>> + u8 tmp[4];> > Why tmp[4] if you only read one byte ?Alignment optimization, perhaps? :-) I'll change it.Thanks,Henrik