Matthew Wilcox wrote: > The usual way to do this is to make sure there's a dummy > ahci_gtf_filter_workaround so there doesn't have to be that ifdef in > the caller. This is probably easier to do like this: > > @@ -2884,6 +2884,7 @@ static bool ahci_broken_online(struct pc > return pdev->bus->number == (val >> 8) && pdev->devfn == (val & 0xff); > } > > static void ahci_gtf_filter_workaround(struct ata_host *host) > { > +#ifdef CONFIG_ATA_ACPI > static const struct dmi_system_id sysids[] = { > @@ -2927,6 +2928,7 @@ static void ahci_gtf_filter_workaround(s > dev->gtf_filter |= filter; > } > +#endif > } Fix is already included in Jeff's tree. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html