Revision 2bf427b25b79eb7cea27963a66c3d4684cae0e0c authored by Bartlomiej Zolnierkiewicz on 30 June 2009, 02:20:42 UTC, committed by David S. Miller on 30 June 2009, 02:20:42 UTC
commit 2f0d0fd2a605666d38e290c5c0d2907484352dc4 ("ide-acpi: cleanup
do_drive_get_GTF()") didn't account for the lack of hwif->acpidata
check in generic_ide_suspend() [ indirect user of do_drive_get_GTF()
through ide_acpi_exec_tfs() ] resulting in broken resume when ACPI
support is enabled but ACPI data is unavailable.

Fix it by adding ide_port_acpi() helper for checking if port needs
ACPI handling and cleaning generic_ide_{suspend,resume}() to use it
instead of hiding hwif->acpidata and ide_noacpi checks in IDE ACPI
helpers (this should help in preventing similar bugs in the future).

While at it:
- kill superfluous debugging printks in ide_acpi_{get,push}_timing()

Reported-and-tested-by: Etienne Basset <etienne.basset@numericable.fr>
Also-reported-and-tested-by: Jeff Chua <jeff.chua.linux@gmail.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 9c72ebe
History
File Mode Size
Makefile -rw-r--r-- 290 bytes
ps3-lpm.c -rw-r--r-- 31.2 KB
ps3-sys-manager.c -rw-r--r-- 20.4 KB
ps3-vuart.c -rw-r--r-- 30.6 KB
ps3av.c -rw-r--r-- 27.7 KB
ps3av_cmd.c -rw-r--r-- 23.8 KB
ps3stor_lib.c -rw-r--r-- 8.3 KB
sys-manager-core.c -rw-r--r-- 2.0 KB
vuart.h -rw-r--r-- 2.8 KB

back to top