diff options
author | Aaron Lu <aaron.lu@intel.com> | 2014-03-14 01:46:09 -0400 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-03-14 11:23:47 -0400 |
commit | c75da205e02dda3b79ca057e558f97f3d61c855d (patch) | |
tree | 31ea36dbff95ef8926991e637aca9894aa1d2475 /drivers/ata | |
parent | 20e64fa42965a6ec6f0ae4c5d4fda0ccf6c94ff6 (diff) |
libata: acpi: avoid passing NULL to ACPI evaluation method
If ACPI handle for an ATA device is NULL, we shouldn't call
ata_dev_get_GTF as that function will use handle to do some ACPI
evaluation.
Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers/ata')
-rw-r--r-- | drivers/ata/libata-acpi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c index 9e69a5308693..b4f7cc2522d9 100644 --- a/drivers/ata/libata-acpi.c +++ b/drivers/ata/libata-acpi.c | |||
@@ -835,6 +835,7 @@ void ata_acpi_on_resume(struct ata_port *ap) | |||
835 | ata_for_each_dev(dev, &ap->link, ALL) { | 835 | ata_for_each_dev(dev, &ap->link, ALL) { |
836 | ata_acpi_clear_gtf(dev); | 836 | ata_acpi_clear_gtf(dev); |
837 | if (ata_dev_enabled(dev) && | 837 | if (ata_dev_enabled(dev) && |
838 | ata_dev_acpi_handle(dev) && | ||
838 | ata_dev_get_GTF(dev, NULL) >= 0) | 839 | ata_dev_get_GTF(dev, NULL) >= 0) |
839 | dev->flags |= ATA_DFLAG_ACPI_PENDING; | 840 | dev->flags |= ATA_DFLAG_ACPI_PENDING; |
840 | } | 841 | } |