summaryrefslogtreecommitdiff
path: root/drivers/ide
diff options
context:
space:
mode:
authorAlan Jenkins <alan-jenkins@tuffmail.co.uk>2009-12-03 07:44:53 (GMT)
committerLen Brown <len.brown@intel.com>2009-12-09 20:54:30 (GMT)
commitfbe3d8942e8fd1e947e4d11a3e9e15675a1cac7e (patch)
tree59325f54e9f2d10d8bfea3a2a1ae5ca82d181c1d /drivers/ide
parent3c0eb510697dbbb53674c72544350624a04ab5b4 (diff)
downloadlinux-fsl-qoriq-fbe3d8942e8fd1e947e4d11a3e9e15675a1cac7e.tar.xz
eeepc-laptop: Remove redundant NULL checks
The acpi device callbacks add, start, remove, suspend and resume can never be called with a NULL acpi_device. Each callsite in acpi/scan.c has to dereference the device in order to get the ops structure, e.g. struct acpi_device *acpi_dev = to_acpi_device(dev); struct acpi_driver *acpi_drv = acpi_dev->driver; if (acpi_drv && acpi_drv->ops.suspend) return acpi_drv->ops.suspend(acpi_dev, state); Remove all checks for acpi_dev == NULL within these callbacks. Also remove the checks for acpi_driver_data(acpi_dev) == NULL. None of these checks could fail unless the driver does something strange (which none of them do), the acpi core did something terribly wrong, or we have a memory corruption issue. If this does happen then it's best to dereference the pointer and crash noisily. Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/ide')
0 files changed, 0 insertions, 0 deletions