Commit 9f4fd61f authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Linus Torvalds

[PATCH] ACPI: clean up memory attribute checking for map/read/write

ia64 ioremap is now smart enough to use the correct memory attributes, so
remove the EFI checks from osl.c.
Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Cc: Matt Domsch <Matt_Domsch@dell.com>
Cc: "Tolentino, Matthew E" <matthew.e.tolentino@intel.com>
Cc: "Brown, Len" <len.brown@intel.com>
Cc: Andi Kleen <ak@muc.de>
Acked-by: default avatar"Luck, Tony" <tony.luck@intel.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 23dd842c
...@@ -180,13 +180,6 @@ acpi_status ...@@ -180,13 +180,6 @@ acpi_status
acpi_os_map_memory(acpi_physical_address phys, acpi_size size, acpi_os_map_memory(acpi_physical_address phys, acpi_size size,
void __iomem ** virt) void __iomem ** virt)
{ {
if (efi_enabled) {
if (EFI_MEMORY_WB & efi_mem_attributes(phys)) {
*virt = (void __iomem *)phys_to_virt(phys);
} else {
*virt = ioremap(phys, size);
}
} else {
if (phys > ULONG_MAX) { if (phys > ULONG_MAX) {
printk(KERN_ERR PREFIX "Cannot map memory that high\n"); printk(KERN_ERR PREFIX "Cannot map memory that high\n");
return AE_BAD_PARAMETER; return AE_BAD_PARAMETER;
...@@ -195,7 +188,6 @@ acpi_os_map_memory(acpi_physical_address phys, acpi_size size, ...@@ -195,7 +188,6 @@ acpi_os_map_memory(acpi_physical_address phys, acpi_size size,
* ioremap checks to ensure this is in reserved space * ioremap checks to ensure this is in reserved space
*/ */
*virt = ioremap((unsigned long)phys, size); *virt = ioremap((unsigned long)phys, size);
}
if (!*virt) if (!*virt)
return AE_NO_MEMORY; return AE_NO_MEMORY;
...@@ -407,18 +399,8 @@ acpi_os_read_memory(acpi_physical_address phys_addr, u32 * value, u32 width) ...@@ -407,18 +399,8 @@ acpi_os_read_memory(acpi_physical_address phys_addr, u32 * value, u32 width)
{ {
u32 dummy; u32 dummy;
void __iomem *virt_addr; void __iomem *virt_addr;
int iomem = 0;
if (efi_enabled) {
if (EFI_MEMORY_WB & efi_mem_attributes(phys_addr)) {
/* HACK ALERT! We can use readb/w/l on real memory too.. */
virt_addr = (void __iomem *)phys_to_virt(phys_addr);
} else {
iomem = 1;
virt_addr = ioremap(phys_addr, width); virt_addr = ioremap(phys_addr, width);
}
} else
virt_addr = (void __iomem *)phys_to_virt(phys_addr);
if (!value) if (!value)
value = &dummy; value = &dummy;
...@@ -436,10 +418,7 @@ acpi_os_read_memory(acpi_physical_address phys_addr, u32 * value, u32 width) ...@@ -436,10 +418,7 @@ acpi_os_read_memory(acpi_physical_address phys_addr, u32 * value, u32 width)
BUG(); BUG();
} }
if (efi_enabled) {
if (iomem)
iounmap(virt_addr); iounmap(virt_addr);
}
return AE_OK; return AE_OK;
} }
...@@ -448,18 +427,8 @@ acpi_status ...@@ -448,18 +427,8 @@ acpi_status
acpi_os_write_memory(acpi_physical_address phys_addr, u32 value, u32 width) acpi_os_write_memory(acpi_physical_address phys_addr, u32 value, u32 width)
{ {
void __iomem *virt_addr; void __iomem *virt_addr;
int iomem = 0;
if (efi_enabled) {
if (EFI_MEMORY_WB & efi_mem_attributes(phys_addr)) {
/* HACK ALERT! We can use writeb/w/l on real memory too */
virt_addr = (void __iomem *)phys_to_virt(phys_addr);
} else {
iomem = 1;
virt_addr = ioremap(phys_addr, width); virt_addr = ioremap(phys_addr, width);
}
} else
virt_addr = (void __iomem *)phys_to_virt(phys_addr);
switch (width) { switch (width) {
case 8: case 8:
...@@ -475,7 +444,6 @@ acpi_os_write_memory(acpi_physical_address phys_addr, u32 value, u32 width) ...@@ -475,7 +444,6 @@ acpi_os_write_memory(acpi_physical_address phys_addr, u32 value, u32 width)
BUG(); BUG();
} }
if (iomem)
iounmap(virt_addr); iounmap(virt_addr);
return AE_OK; return AE_OK;
......
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