> pata_cs5536.c relies on Geode Machine Specific Registers to configure > the ATA function and uses the correct PIO timings for the chip. Any reason for not supporting both methods ? Though I guess as you say msr is sufficient. > +#ifndef CONFIG_PATA_CS5536 /* Temporary */ !defined(FOO) && ! defined(FOO_MODULE) otherwise ACK Alan - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html