Re: [PATCHv3 1/2] platform/x86: dell-sysman: remove match on www.dell.com

From: Ilpo Järvinen
Date: Fri Oct 04 2024 - 09:45:15 EST


On Fri, 4 Oct 2024, Crag Wang wrote:

> The URL is dynamic and may change according to the OEM. It was mainly used
> for old systems that do not have "Dell System" in the OEM String.
>
> Signed-off-by: Crag Wang <crag_wang@xxxxxxxx>
> ---
> drivers/platform/x86/dell/dell-wmi-sysman/sysman.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c b/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c
> index 9def7983d7d6..c05474f1ed70 100644
> --- a/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c
> +++ b/drivers/platform/x86/dell/dell-wmi-sysman/sysman.c
> @@ -520,8 +520,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)) {
> + if (!dmi_find_device(DMI_DEV_TYPE_OEM_STRING, "Dell System", NULL)) {

I suggested making the changes in the opposite order, that was to
faciliate easy revert of the URL patch if needed which is not the case if
things are changed in this order.

--
i.