Commit 31a9ecd0 authored by Dave Jones's avatar Dave Jones

[AGPGART] Fix return check on request_mem_region()

Do things the way every other user of this function does.
Spotted by Arjan with a suitably pedantic gcc.
parent c7b42d9a
...@@ -279,7 +279,7 @@ static int __init aperture_valid(u64 aper, u32 size) ...@@ -279,7 +279,7 @@ static int __init aperture_valid(u64 aper, u32 size)
Maybe better to use pci_assign_resource/pci_enable_device instead trusting Maybe better to use pci_assign_resource/pci_enable_device instead trusting
the bridges? */ the bridges? */
if (!not_first_call && request_mem_region(aper, size, "aperture") < 0) { if (!not_first_call && !request_mem_region(aper, size, "aperture")) {
printk(KERN_ERR PFX "Aperture conflicts with PCI mapping.\n"); printk(KERN_ERR PFX "Aperture conflicts with PCI mapping.\n");
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment