On Sunday 28 September 2014 16:51:31 Suravee Suthikulpanit wrote: > >> @@ -87,6 +90,13 @@ static const struct of_device_id ahci_of_match[] = { > >> }; > >> MODULE_DEVICE_TABLE(of, ahci_of_match); > >> > >> +#ifdef CONFIG_ATA_ACPI > >> +static const struct acpi_device_id ahci_acpi_match[] = { > >> + { "AMDI0600", 0 }, /* AMD Seattle AHCI */ > >> + { }, > >> +}; > >> +#endif > >> + I just noticed this patch and stumbled over this hack. Since SBSA requires any SATA support to be compatible with AHCI and not vendor specific, I think this needs to be a more generic compatibility string that can be used by all vendors, while the current one looks very AMD specific for hardware that is not. Arnc -- 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