Hello. Bartlomiej Zolnierkiewicz wrote: [...]
Damn, I should have re-check ATA specs before writing this. :)
Yeah, never hurts... but takes time. ;-)
I was thinking of checking if the drive really supports IORDY before issuing a command to set PIO mode (and just skipping the command if there's no IORDY -- well, maybe adding an extra check that the passed mode is acceptable to the drive, i.e. <= its default one). Should be quite simple to do.
Sounds fine.
I'll look into doing this some time... :-)
This should be done together with fixing these host drivers that don't handle IORDY properly.
Erm, not necessarily...
Hmm, yes.
I'll just count on you with fixing all this IORDY stuff (as you have much more expertise in this field) and concentrate on other things.
I'm afraid this compliment is not well deserved. :-<
Oh yes, I keep forgetting about it - some nice FIXME comment in <linux/ata.h> would be of a great help. :-)
Well, some drivers (like pdc202xx_*) don't do the IORDY thing right for PIO modes < 3 as well...
Added to the existing IDE TODO at
http://kernel.org/pub/linux/kernel/people/bart/pata-2.6/TODO
I think this list already needs cleaning -- e.g. at least cmd64x.c and hpt366.c have been deatl with.
Patches adding/removing items are welcomed.
Patches fixing actual issues are welcomed even more.
Sigh, I'm trying to get some time (more like time slices :-) off to deal with my own issues...
By by "my own issues" I don't mean patches -- patches don't help me, it's rather injections and pills... +:-)
Which reminds me about some HPT IDE patches... 8)
Done now -- at last. However, I've had troubles with SMPT sending them. :-/
Bart
WBR, Sergei - 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
