From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DDB24C4332F for ; Mon, 13 Nov 2023 20:18:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230175AbjKMUSX (ORCPT ); Mon, 13 Nov 2023 15:18:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53590 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229689AbjKMUSU (ORCPT ); Mon, 13 Nov 2023 15:18:20 -0500 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B724810E5 for ; Mon, 13 Nov 2023 12:17:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1699906672; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=WHG7y6RDisjgRBKsr9b5mVEL+3CBXjyGr1DFkZoWxFc=; b=BN802Sa1GITASuKmN6jtbrGa6XOTzygNpDSwMNtOSHMbyj4MpEiS3MbDsHF5LHXeJlQFuk IApnwVhD0bd8B5IvcxR1/9U76dwTn1x2UE9sth9JpWd4+LlsXDsJUwqt3fyvgpxlkeBerM 5PZ57mku8rsyIeIJ2NQ7u789i9im8+I= Received: from mail-ed1-f70.google.com (mail-ed1-f70.google.com [209.85.208.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-537-tL2zQOvcMrqFTPlEt64gJg-1; Mon, 13 Nov 2023 15:17:51 -0500 X-MC-Unique: tL2zQOvcMrqFTPlEt64gJg-1 Received: by mail-ed1-f70.google.com with SMTP id 4fb4d7f45d1cf-543714a9d86so3238096a12.2 for ; Mon, 13 Nov 2023 12:17:51 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699906670; x=1700511470; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=WHG7y6RDisjgRBKsr9b5mVEL+3CBXjyGr1DFkZoWxFc=; b=uC6kQVHXnbd6DDXo9PrEZ9It6baNa+nZUkNpEYhbg3A4F2RNHqIomUmQ74sY1l6rbw UKsxT0mwGPTZ1t/A6YRLBJakjellK/SAWshiEOM2N7iNpx2NF1gQzYWY3M2UKTREeYqr mdEmsDpKGma7U39qAI/HN7j/FNvqEwypnthmWJlrpBlL7FKHn4keiusduKx/jBu9tHOw c2vs3ZsQD0uL/7iOm+5ZwyqL54RGmgYemD3ZXG6GFOARU9n8V6eoGaI8JnclQ4JqD8GJ 49zLgMfPET32Y/3mqbAKF7J/epGykCa8Xt3NIb3CnoV0MFXbUgjYKwTMlnzwtpM/oZP9 BYfw== X-Gm-Message-State: AOJu0Yzg0oSlRoq6Rhxyjp9/jBbeW+546bOAnPAtFEK1IjXLRokW6sNP 1Rx+g4omxyifGwvUwB5PM/ya8Crfa4sFThk5yatPqiW+2jY0esstnaMPEmUjESOolCHZFkZocyx MCEDi1lsvlos06ial7sQaKufx X-Received: by 2002:aa7:d555:0:b0:546:d0b7:f4e4 with SMTP id u21-20020aa7d555000000b00546d0b7f4e4mr5720411edr.8.1699906670246; Mon, 13 Nov 2023 12:17:50 -0800 (PST) X-Google-Smtp-Source: AGHT+IHJ31+bJ3bnZRU8DicZJd+AsZ1d8QMmiId8Sfli7YEux00xVSQkU/oZU9dAgygaDj14tUz0kA== X-Received: by 2002:aa7:d555:0:b0:546:d0b7:f4e4 with SMTP id u21-20020aa7d555000000b00546d0b7f4e4mr5720398edr.8.1699906669927; Mon, 13 Nov 2023 12:17:49 -0800 (PST) Received: from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec? (2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl. [2001:1c00:c32:7800:5bfa:a036:83f0:f9ec]) by smtp.gmail.com with ESMTPSA id d22-20020a50cd56000000b0054719a2a0cdsm3696000edj.16.2023.11.13.12.17.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 13 Nov 2023 12:17:49 -0800 (PST) Message-ID: <8c25dfec-b0ca-4e2f-964a-5ae62452e141@redhat.com> Date: Mon, 13 Nov 2023 21:17:48 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 0/9] hwmon: (dell-smm) Add support for WMI SMM interface To: Armin Wolf , pali@kernel.org Cc: markgross@kernel.org, ilpo.jarvinen@linux.intel.com, jdelvare@suse.com, linux@roeck-us.net, platform-driver-x86@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org References: <20231106064351.42347-1-W_Armin@gmx.de> <3ec0496d-3b89-46f5-9faf-9fcce78f6b38@gmx.de> Content-Language: en-US, nl From: Hans de Goede In-Reply-To: <3ec0496d-3b89-46f5-9faf-9fcce78f6b38@gmx.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Armin, On 11/13/23 20:55, Armin Wolf wrote: > Am 06.11.23 um 07:43 schrieb Armin Wolf: > >> This patch series adds support for an alternative SMM calling >> backend to the dell-smm-hwmon driver. The reason for this is >> that on some modern machines, the legacy SMM calling interface >> does not work anymore and the SMM handler can be called over >> ACPI WMI instead. >> >> The first four patches prepare the driver by allowing to >> specify different SMM calling backends, and by moving most of >> the DMI handling into i8k_init() so that the DMI tables can >> keep their __initconst attributes (the WMI SMM backend driver >> does not probe at module init time). The fifth patch does some >> minor cleanup, while the next three patches implement the new >> WMI SMM calling backend. The last patch adds the machine of >> the user who requested and tested the changes to the fan control >> whitelist. >> >> If the driver does not detect the legacy SMM interface, either >> because the machine is not whitelisted or because the SMM handler >> does not react, it registers an WMI driver which will then bound >> to the WMI SMM interface and do the remaining initialization. >> >> The deprecated procfs interface is not supported when using the >> WMI SMM calling backend for the following reason: the WMI driver >> can potentially be instantiated multiple times while the deprectated >> procfs interface cannot. This should not cause any regressions >> because on machines supporting only the WMI SMM interface, the >> driver would, until now, not load anyway. >> >> All patches where tested on a Dell Inspiron 3505 and a Dell >> OptiPlex 7000. > > Any thoughts on this? I was waiting for the merge window to close before reviewing / merging patches for the next cycle. I plan to review and hopefully merge this and your other series sometime this week. Regards, Hans