linux-kernel - RE: [PATCHv2 1/1] platform/x86: dell-sysman: add support for alienware products

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <CY5PR19MB61479854D62CAD389C99BA15BA682@CY5PR19MB6147.namprd19.prod.outlook.com>
Date: Tue, 24 Sep 2024 14:34:17 +0000
From: "Tudor, Laurentiu" <Laurentiu.Tudor1@...l.com>
To: Mario Limonciello <mario.limonciello@....com>,
        Crag Wang
	<crag0715@...il.com>,
        "Ksr, Prasanth" <Prasanth.Ksr@...l.com>,
        Hans de Goede
	<hdegoede@...hat.com>,
        Ilpo Järvinen
	<ilpo.jarvinen@...ux.intel.com>
CC: "Wang, Crag" <Crag.Wang@...l.com>, "Wang, Crag" <Crag.Wang@...l.com>,
        Dell
 Client Kernel <Dell.Client.Kernel@...l.com>,
        "platform-driver-x86@...r.kernel.org" <platform-driver-x86@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCHv2 1/1] platform/x86: dell-sysman: add support for
 alienware products

Hi,


Internal Use - Confidential
-----Original Message-----
From: Mario Limonciello <mario.limonciello@....com>
Sent: Tuesday, September 24, 2024 4:24 PM

On 9/24/2024 00:02, Crag Wang wrote:
> Remove the URL check and add alienware string for broader support.

Couple of newbie questions: what's the reason for dropping the URL check? Would it make sense to include the reason in the commit msg?

---
Thanks & Best Regards, Laurentiu

> Signed-off-by: Crag Wang <crag_wang@...l.com>
> ---
Reviewed-by: Mario Limonciello <mario.limonciello@....com>
>   drivers/platform/x86/dell/dell-wmi-sysman/sysman.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c b/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c
> index 9def7983d7d6..68c63e1fbd27 100644
> --- a/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c
> +++ b/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c
> @@ -521,7 +521,7 @@ static int __init sysman_init(void)
>       int ret = 0;
>
>       if (!dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "Dell System", NULL) &&
> -         !dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "www.dell.com", NULL)) {
> +         !dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "Alienware", NULL)) {
>               pr_err("Unable to run on non-Dell system\n");
>               return -ENODEV;
>       }