> should they be attached to the scsi code rather than the ata host code? 
> Anyway, this fixes suspend/resume on nc6220, which is what I was aiming 
> for...
> 
> Signed-off-by: Matthew Garrett <[EMAIL PROTECTED]>

NAK - there is code to do the _gtm/_stm parts in the -mm tree already
used by pata_acpi, waiting to be merged and it's a lot cleaner. It does
want hooking to the suspend/resume paths as well as to pata_acpi but that
is all.

This code also has bugs

> +
> +     if (ap->cbl == ATA_CBL_SATA)
> +             return;

Bangs head against wall. As I keep saying you *CANNOT* do any PATA v SATA
decisions based upon cable type. It just doesn't work. The kernel
sometimes doesn't know. The hardware sometimes turns up in weird formats
with ATA bridges.

You *MUST* see if the device has GTF/GTM or SATA methods in its ACPI
space and decide based upon that alone what the BIOS thinks should be
used.

-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to