On Sat, 2005-07-02 at 17:23 -0700, Gaston, Jason D wrote: >>> Please do not apply this patch. The ICH6M SATA DID is all ready >>> being used in the SATA ata_piix.c and ahci.c drivers. Adding the >>> ICH6M SATA DID to the PATA piix.c driver will conflict. This patch >>> would add the ICH6M >SATA controller DID 0x2653 to the PATA >>> piix.c driver. > > That's why you either enable piix support from the standard ide driver > > xor from libsata. The comments in the configure script even mention this > > fact! > You need to have support for both at the same time if you want to run in > enhanced mode and have full access to your PATA and SATA controllers and > drives. Hmmm. I guess this would be in the (imho unlikely) situation where you have both PATA and SATA drives connected to the ICH6M. I do see the problem, though. It's a "works for me(r)" solution ;-) > I suggest you try to place your BIOS in Enhanced mode for the SATA/PATA > controllers so that you have both the PATA and SATA DID's and > controllers showing up. Then your PATA drive will show up as an IDE > device and use the piix driver. Bzzzt! Nice theory, but in practise, MANY bioses (including mine) don't offer the choice to put the controller in whatever mode. Add to that, that if you only have pata drives attached, it doesn't make much sense to drive it with libata. I'd be happy to drive my ICH6M with libata, but it just doesn't work well. No smart&hdparm (the corresponding patch freezes the kernel every now and then) support, drives show up as scsi drives..., no atapi (unless #defined, but also not yet bugfree). Well anyway, this is just my $0.02 I guess it won't make it into the kernel, but you never know how might be able to use it.
Attachment:
signature.asc
Description: This is a digitally signed message part
- Prev by Date: RE: [PATCH] ich6m-pciid-piix.patch
- Next by Date: Re: [RFC] Fix SMP brokenness for PF_FREEZE and make freezing usable for other purposes
- Previous by thread: RE: [PATCH] ich6m-pciid-piix.patch
- Next by thread: [-mm patch] CONFIGFS_FS: "If unsure, say N."
- Index(es):