lkml.org 
[lkml]   [2018]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.17 263/336] platform/x86: dell-smbios: Match on www.dell.com in OEM strings too
    Date
    4.17-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Mario Limonciello <mario.limonciello@dell.com>

    [ Upstream commit b004b21cc664ca00782508514dade43e29eebf94 ]

    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>
    Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/platform/x86/dell-smbios-base.c | 5 ++---
    1 file changed, 2 insertions(+), 3 deletions(-)

    --- a/drivers/platform/x86/dell-smbios-base.c
    +++ b/drivers/platform/x86/dell-smbios-base.c
    @@ -555,11 +555,10 @@ static void free_group(struct platform_d

    static int __init dell_smbios_init(void)
    {
    - const struct dmi_device *valid;
    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");
    return -ENODEV;
    }

    \
     
     \ /
      Last update: 2018-08-01 20:23    [W:2.212 / U:1.288 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site