Re: [PATCH] Add _GTM and _STM support to libata

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 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 <mjg59@xxxxxxxxxxxxx>

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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux