On Thursday 19 November 2009 15:33:51 Alan Cox wrote: > On Thu, 19 Nov 2009 15:17:14 +0100 > Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx> wrote: > > > On Thursday 19 November 2009 14:16:36 Alan Cox wrote: > > > > Only pata_hpt37x has broken cable detection while pata_hpt3x2n is > > > > okay. ;) > > > > > > I think you have that backwards according to the vendor drivers. 3x2n > > > looks like it needs adjusting. > > > > lol, just fix the damn pata_hpt37x driver _lacking_ ->cable_detect method.. > > It doesn't need one. It still sets the cable type in the pre reset > function as the older drivers that haven't converted to a cable_detect > method do. If you know about other drivers still using ->pre_reset for cable detection please let us know because they need fixing ASAP. ->cable_detect method is there for a reason, by knowingly not using it you already have a buggy cable detection (since ->pre_reset ignores the mandatory by spec part of cable detection which is probing slave before master) and make it more likely to cause breakages in the future when some other developer do a change under assumption that all host drivers use API correctly. -- Bartlomiej Zolnierkiewicz -- 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