Browse Source

platform/x86: dell-smbios: Match on www.dell.com in OEM strings too

Sergey reported that some much older Dell systems don't support
the OEM string "Dell System" but instead supported www.dell.com
in OEM strings.

Match both of these to indicate that this driver is running on
a Dell system.

Reported-by: Sergey Kubushyn <ksi@koi8.net>
Tested-by: Sergey Kubushyn <ksi@koi8.net>
Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
[dvhart: Simplify DMI logic and eliminate unnecessary variables]
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
Mario Limonciello 7 years ago
parent
commit
b004b21cc6
1 changed files with 2 additions and 3 deletions
  1. 2 3
      drivers/platform/x86/dell-smbios-base.c

+ 2 - 3
drivers/platform/x86/dell-smbios-base.c

@@ -555,11 +555,10 @@ static void free_group(struct platform_device *pdev)
 
 
 static int __init dell_smbios_init(void)
 static int __init dell_smbios_init(void)
 {
 {
-	const struct dmi_device *valid;
 	int ret, wmi, smm;
 	int ret, wmi, smm;
 
 
-	valid = dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "Dell System", NULL);
-	if (!valid) {
+	if (!dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "Dell System", NULL) &&
+	    !dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "www.dell.com", NULL)) {
 		pr_err("Unable to run on non-Dell system\n");
 		pr_err("Unable to run on non-Dell system\n");
 		return -ENODEV;
 		return -ENODEV;
 	}
 	}