[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
<PH0PR19MB49887C3E0537551D2B43B07CE3722@PH0PR19MB4988.namprd19.prod.outlook.com>
Date: Fri, 4 Oct 2024 14:30:21 +0000
From: "Wang, Crag" <Crag.Wang@...l.com>
To: Hans de Goede <hdegoede@...hat.com>, Crag Wang <crag0715@...il.com>,
"Limonciello, Mario" <mario.limonciello@....com>,
"Ksr, Prasanth"
<Prasanth.Ksr@...l.com>,
Ilpo Järvinen
<ilpo.jarvinen@...ux.intel.com>
CC: 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: [PATCHv3 1/2] platform/x86: dell-sysman: remove match on
www.dell.com
> But those old systems presumably still exist somewhere out there, right ?
Products with this driver have Dell System in place.
> And if they still exist then we do still want to support them, so I'm not sure why
> you think it is a good idea to drop this test ?
We can keep the check as it's harmless though the match should rely on Dell System
more than the URL, which is mainly for super older cases without Dell System.
> Adding the alienware match seems fine, dropping the URL match seems like a
> bad idea unless you are 100% sure that there are no systems out there which
> rely in this match to load dell-wmi-sysman.
I'll re-send with just the addition.
Crag
Powered by blists - more mailing lists