Commit cf074402 authored by Ard Biesheuvel's avatar Ard Biesheuvel Committed by Linus Torvalds

firmware/dmi_scan: generalize for use by other archs

This patch makes a couple of changes to the SMBIOS/DMI scanning
code so it can be used on other archs (such as ARM and arm64):
(a) wrap the calls to ioremap()/iounmap(), this allows the use of a
    flavor of ioremap() more suitable for random unaligned access;
(b) allow the non-EFI fallback probe into hardcoded physical address
    0xF0000 to be disabled.
Signed-off-by: default avatarArd Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: default avatarGrant Likely <grant.likely@linaro.org>
Cc: Ingo Molnar <mingo@elte.hu>
Cc "Luck, Tony" <tony.luck@intel.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 3e2a4c18
...@@ -104,6 +104,7 @@ config HAVE_SETUP_PER_CPU_AREA ...@@ -104,6 +104,7 @@ config HAVE_SETUP_PER_CPU_AREA
config DMI config DMI
bool bool
default y default y
select DMI_SCAN_MACHINE_NON_EFI_FALLBACK
config EFI config EFI
bool bool
......
...@@ -5,8 +5,10 @@ ...@@ -5,8 +5,10 @@
#include <asm/io.h> #include <asm/io.h>
/* Use normal IO mappings for DMI */ /* Use normal IO mappings for DMI */
#define dmi_ioremap ioremap #define dmi_early_remap ioremap
#define dmi_iounmap(x,l) iounmap(x) #define dmi_early_unmap(x, l) iounmap(x)
#define dmi_remap ioremap
#define dmi_unmap iounmap
#define dmi_alloc(l) kzalloc(l, GFP_ATOMIC) #define dmi_alloc(l) kzalloc(l, GFP_ATOMIC)
#endif #endif
...@@ -731,6 +731,7 @@ config APB_TIMER ...@@ -731,6 +731,7 @@ config APB_TIMER
# The code disables itself when not needed. # The code disables itself when not needed.
config DMI config DMI
default y default y
select DMI_SCAN_MACHINE_NON_EFI_FALLBACK
bool "Enable DMI scanning" if EXPERT bool "Enable DMI scanning" if EXPERT
---help--- ---help---
Enabled scanning of DMI to identify machine quirks. Say Y Enabled scanning of DMI to identify machine quirks. Say Y
......
...@@ -13,7 +13,9 @@ static __always_inline __init void *dmi_alloc(unsigned len) ...@@ -13,7 +13,9 @@ static __always_inline __init void *dmi_alloc(unsigned len)
} }
/* Use early IO mappings for DMI because it's initialized early */ /* Use early IO mappings for DMI because it's initialized early */
#define dmi_ioremap early_ioremap #define dmi_early_remap early_ioremap
#define dmi_iounmap early_iounmap #define dmi_early_unmap early_iounmap
#define dmi_remap ioremap
#define dmi_unmap iounmap
#endif /* _ASM_X86_DMI_H */ #endif /* _ASM_X86_DMI_H */
...@@ -108,6 +108,9 @@ config DMI_SYSFS ...@@ -108,6 +108,9 @@ config DMI_SYSFS
under /sys/firmware/dmi when this option is enabled and under /sys/firmware/dmi when this option is enabled and
loaded. loaded.
config DMI_SCAN_MACHINE_NON_EFI_FALLBACK
bool
config ISCSI_IBFT_FIND config ISCSI_IBFT_FIND
bool "iSCSI Boot Firmware Table Attributes" bool "iSCSI Boot Firmware Table Attributes"
depends on X86 depends on X86
......
...@@ -116,7 +116,7 @@ static int __init dmi_walk_early(void (*decode)(const struct dmi_header *, ...@@ -116,7 +116,7 @@ static int __init dmi_walk_early(void (*decode)(const struct dmi_header *,
{ {
u8 *buf; u8 *buf;
buf = dmi_ioremap(dmi_base, dmi_len); buf = dmi_early_remap(dmi_base, dmi_len);
if (buf == NULL) if (buf == NULL)
return -1; return -1;
...@@ -124,7 +124,7 @@ static int __init dmi_walk_early(void (*decode)(const struct dmi_header *, ...@@ -124,7 +124,7 @@ static int __init dmi_walk_early(void (*decode)(const struct dmi_header *,
add_device_randomness(buf, dmi_len); add_device_randomness(buf, dmi_len);
dmi_iounmap(buf, dmi_len); dmi_early_unmap(buf, dmi_len);
return 0; return 0;
} }
...@@ -527,18 +527,18 @@ void __init dmi_scan_machine(void) ...@@ -527,18 +527,18 @@ void __init dmi_scan_machine(void)
* needed during early boot. This also means we can * needed during early boot. This also means we can
* iounmap the space when we're done with it. * iounmap the space when we're done with it.
*/ */
p = dmi_ioremap(efi.smbios, 32); p = dmi_early_remap(efi.smbios, 32);
if (p == NULL) if (p == NULL)
goto error; goto error;
memcpy_fromio(buf, p, 32); memcpy_fromio(buf, p, 32);
dmi_iounmap(p, 32); dmi_early_unmap(p, 32);
if (!dmi_present(buf)) { if (!dmi_present(buf)) {
dmi_available = 1; dmi_available = 1;
goto out; goto out;
} }
} else { } else if (IS_ENABLED(CONFIG_DMI_SCAN_MACHINE_NON_EFI_FALLBACK)) {
p = dmi_ioremap(0xF0000, 0x10000); p = dmi_early_remap(0xF0000, 0x10000);
if (p == NULL) if (p == NULL)
goto error; goto error;
...@@ -554,12 +554,12 @@ void __init dmi_scan_machine(void) ...@@ -554,12 +554,12 @@ void __init dmi_scan_machine(void)
memcpy_fromio(buf + 16, q, 16); memcpy_fromio(buf + 16, q, 16);
if (!dmi_present(buf)) { if (!dmi_present(buf)) {
dmi_available = 1; dmi_available = 1;
dmi_iounmap(p, 0x10000); dmi_early_unmap(p, 0x10000);
goto out; goto out;
} }
memcpy(buf, buf + 16, 16); memcpy(buf, buf + 16, 16);
} }
dmi_iounmap(p, 0x10000); dmi_early_unmap(p, 0x10000);
} }
error: error:
pr_info("DMI not present or invalid.\n"); pr_info("DMI not present or invalid.\n");
...@@ -831,13 +831,13 @@ int dmi_walk(void (*decode)(const struct dmi_header *, void *), ...@@ -831,13 +831,13 @@ int dmi_walk(void (*decode)(const struct dmi_header *, void *),
if (!dmi_available) if (!dmi_available)
return -1; return -1;
buf = ioremap(dmi_base, dmi_len); buf = dmi_remap(dmi_base, dmi_len);
if (buf == NULL) if (buf == NULL)
return -1; return -1;
dmi_table(buf, dmi_len, dmi_num, decode, private_data); dmi_table(buf, dmi_len, dmi_num, decode, private_data);
iounmap(buf); dmi_unmap(buf);
return 0; return 0;
} }
EXPORT_SYMBOL_GPL(dmi_walk); EXPORT_SYMBOL_GPL(dmi_walk);
......
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