On Fri, Sep 16, 2011 at 05:22:00PM +0100, Dave Martin wrote: > This patch enables device-tree-based probing of the pata-generic > platform driver across all architectures: > > * make the pata_of_generic module depend on OF instead of PPC_OF; > * supply some missing inclues; > * replace endianness-sensitive raw access to device tree data > with of_property_read_u32() calls. > > Signed-off-by: Dave Martin <dave.martin@xxxxxxxxxx> > --- > v2: correct sense of the check of_property_read_u32(dn, "pio-mode", > &pio_mode). Somehow I posted an old version of this patch, depite > having already fixed this... > > Tested on ARM Versatile Express, with my soon-to-be-posted device > tree support patches. > > I'm not in a position to build/test this for powerpc easily -- > if anyone is able to do that, it would be appreciated. What driver is normally used for versatile express pata? This driver is kind of legacy in that it was created when there was a split between platform_device and of_platform_devices. But that split was a bad idea and the same driver should be used regardless of whether or not DT is enabled. pata_of_platform.c really should be removed. So, instead of removing the PPC_OF restriction from pata_of_platform.c, you should look at adding DT support to the existing binding. Bonus points if you can roll pata_of_platform.c support into pata_platform.c (assuming that is the correct driver). g. -- 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