Bartlomiej Zolnierkiewicz wrote:
On Monday 02 July 2007, Jeff Garzik wrote:
Bartlomiej Zolnierkiewicz wrote:
This would brake setups which currently work OK, i.e. BIOS set things up
(reminds me about cmd64x vs broken MWDMA)...
The RightThing(tm) to do is to fix alim15x3 driver to program DMA timings
(especially given that pata_ali seems to already contain the needed code).
I am not ready to trust that pata_ali works as well as alim15x3 in all
cases. Someone should test e.g. Alpha AXP systems with IDE (use
alim15x3) to make sure all is well.
Sure but there is no problem with the new code in alim15x3 being X86 until
it is verified to work with Alpha AXP machines etc...
I'm not sure I follow. You now want to mark alim15x3 X86-only? Or add
the new code inside #ifdef X86?
alim15x3 covers buggy, ugly, quirky chipsets. I consider that coverage
in general highly fragile.
and in case of MWDMA modes also highly dependent on BIOS settings
which very likely results in broken suspend/resume...
Agreed. But I would rather verify that timing programming works on
supported platforms, before expanding use of said new code?
Jeff
-
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
