diff options
| author | Aaron Lu <aaron.lu@intel.com> | 2013-01-28 06:08:02 +0100 |
|---|---|---|
| committer | Jeff Garzik <jgarzik@redhat.com> | 2013-02-20 23:14:03 +0100 |
| commit | 53637e0760d56274177be7e6d630490bc49766c1 (patch) | |
| tree | 1460da4181c17e2498aef9da44d6f3289ecfcddc /drivers/ata | |
| parent | [libata] Set proper SK when CK_COND is set. (diff) | |
| download | linux-53637e0760d56274177be7e6d630490bc49766c1.tar.xz linux-53637e0760d56274177be7e6d630490bc49766c1.zip | |
[libata] fix smatch warning for zpodd_wake_dev
Fix a smatch warning caused by an useless pointer check.
The context parameter (aka. ata_dev) will never be NULL until we remove
the acpi notification handler, so it is pointless to check it for NULL.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/ata')
| -rw-r--r-- | drivers/ata/libata-zpodd.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/ata/libata-zpodd.c b/drivers/ata/libata-zpodd.c index a7df60383532..90b159b740b3 100644 --- a/drivers/ata/libata-zpodd.c +++ b/drivers/ata/libata-zpodd.c @@ -246,8 +246,7 @@ static void zpodd_wake_dev(acpi_handle handle, u32 event, void *context) struct zpodd *zpodd = ata_dev->zpodd; struct device *dev = &ata_dev->sdev->sdev_gendev; - if (event == ACPI_NOTIFY_DEVICE_WAKE && ata_dev && - pm_runtime_suspended(dev)) { + if (event == ACPI_NOTIFY_DEVICE_WAKE && pm_runtime_suspended(dev)) { zpodd->from_notify = true; pm_runtime_resume(dev); } |