On Wed, 2007-01-24 at 20:20 -0500, Jeff Garzik wrote: > Akira Iguchi wrote: > > This is the patch for PATA controller of Celleb. > > It depends on the previous "add another IRQ calls" patch. > > > > Because this driver needs special taskfile accesses, there is > > a copy of ata_std_softreset(). ata_dev_try_classify() is exported > > so that it can be used in this function. > > > > Signed-off-by: Kou Ishizaki <kou.ishizaki@xxxxxxxxxxxxx> > > Signed-off-by: Akira Iguchi <akira2.iguchi@xxxxxxxxxxxxx> > > The driver looks OK, the only thing I wonder is: can you use the iomap > interface to eliminate the rest of the duplicated code? > > See Tejun's latest patchset "[PATCHSET] Managed device resources, take > #3" which completes the conversion of libata to using the lib/iomap.c > interface. That would be a bit nasty :-) The powerpc kernel can have any number of platforms compiled into the same kernel. Some of the base things, like PCI IOs, iomap, etc... are common to all platforms. In this case, what they want is hook only accesses on one given device, one one given platform (or rather, on all platforms using that device, which is even worse, it's not even platform specific, at least not in theory). While we do now provide a mecanism to add hooks to all PCI MMIO and IO accessors (to workaround HW bugs on some chipsets), again, it would be a a serious abuse to have a driver hook on that to change the access method on a given device :-) How much copied code remains ? I wonder if the accesses to nsect/lba for example can be done using the taskfile accessors, possibly with a new flag indicating to only read/write those 2 registers ? Ben. - 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
