Received: by 2002:a05:6358:5282:b0:b5:90e7:25cb with SMTP id g2csp1617784rwa; Sun, 21 Aug 2022 12:35:33 -0700 (PDT) X-Google-Smtp-Source: AA6agR441+ByB4A9UQLe9vnZ6XT0JeIr06JBlF6a35sngva9lVVeMg7mIF6IqCea4TySF6dehBlP X-Received: by 2002:a05:6402:360d:b0:445:bd16:803b with SMTP id el13-20020a056402360d00b00445bd16803bmr13461328edb.318.1661110533683; Sun, 21 Aug 2022 12:35:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1661110533; cv=none; d=google.com; s=arc-20160816; b=OLVhermxC0S1ngQdBm8LGHu692xvCf0OhESGRlQAiOubr5dVTNWa8XMW9Md8uGcPeK 52Drg8QrfbS/J1gI1UQj2/TuHOGtqljYcK5o4iwg9e8QVhTOW392JZ4YrmI4+ZsUbU0y Agh3XU8dIo5FmCtybp5zXOXSGRVjDTSgaIdTxVeFGs2hnJpFuBHBUYCjzlNGl0/mIjAi 0dutjsrOkPV7rWMQZSAx6G7x/dE5TYLnObCVWOHk7ud55qtcO3+6R9HdmReWeSEQQnvJ LtJapUSppUTnOknknvOk7DpdqLewIff/1RyWFerGD13pEIsHtNio5ndSDYXr8qV5EhmM scGQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-language:content-transfer-encoding :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=REDu/up15TSflSAcTos4BRVOx8pEiVKVWIBwYhg9Vug=; b=MKSe+Xji+MhkYO5ikuXfb7ywiFw/VSf43N9OMC+EM6FljTdXiLUVjvPF2cFjb++oXX M4d65Gaafd6XnJAlnv+1Bi7YV9S1FsbdR/p8mTo0T0CsBYkbuzz1EN7mYGswHFI3lYEl SoRg3lYTu5hBiQZkdq3gr0pTvALEf7ybgDVpSyFQ5ep+b4QGLkZLPlQsPdO2uBq8Jd8f yliKA8CnXXkaM30tO1jLyvziA3oDVTllozMfTXXHwoXa8VYbMdey3ShLp823182x2DKt MtW0+oATmm8rtcM7as38DKhSs9yzx+jMAFfP4t0XpA9GF9JY5kzL9NRkO92yYXDblR/L /LAQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmx.net header.s=badeba3b8450 header.b=hHw03Hdc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=gmx.de Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id qb41-20020a1709077ea900b007304a1ee3e3si7183469ejc.517.2022.08.21.12.35.06; Sun, 21 Aug 2022 12:35:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmx.net header.s=badeba3b8450 header.b=hHw03Hdc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=gmx.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231328AbiHUTRh (ORCPT + 99 others); Sun, 21 Aug 2022 15:17:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36860 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229561AbiHUTRf (ORCPT ); Sun, 21 Aug 2022 15:17:35 -0400 Received: from mout.gmx.net (mout.gmx.net [212.227.15.19]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8587D1CFCF; Sun, 21 Aug 2022 12:17:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1661109428; bh=w9Oco7NMKS//9+T6Z5wgnNMKbJiFLo0nDjeIiWXYe8k=; h=X-UI-Sender-Class:Subject:To:Cc:References:From:Date:In-Reply-To; b=hHw03Hdca7yKDoQdXR0f/0XuDEibfi5dfz7VXMZD0DicamW1UxxedwF5oKBksEd3U o8DJXWKsgCM1q6hzoB1Sw7n02kCV32PCSR1WyhJYl5dh/HAtN2uPULVZ1ECtcxXdCK xeVaBL1f51m4O0MKdkqU2DIEnvtZscRAwo07WMdU= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from [192.168.0.21] ([141.30.226.119]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1Mz9Yv-1pKiYB2kHD-00wC8j; Sun, 21 Aug 2022 21:17:08 +0200 Subject: Re: [PATCH 3/3] hwmon: (dell-smm) Improve warning messages To: =?UTF-8?Q?Pali_Roh=c3=a1r?= Cc: jdelvare@suse.com, linux@roeck-us.net, linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220821151713.11198-1-W_Armin@gmx.de> <20220821151713.11198-4-W_Armin@gmx.de> <20220821154610.e2fokkqzytoqm2ba@pali> From: Armin Wolf Message-ID: <54f18641-a7a3-65f8-1542-369b6652614c@gmx.de> Date: Sun, 21 Aug 2022 21:17:08 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20220821154610.e2fokkqzytoqm2ba@pali> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: quoted-printable Content-Language: en-US X-Provags-ID: V03:K1:UaOYNGQTQ0ePyD0xqH79UZwT6xU5jPFHN7syj+IphxUtV2vK9go PALxfgSV7fWTvOIeQ+N1uEnZDVOnfVxhwNi9HCGrBdcYvTIqFqw6ah6LSHJIN5BVFDH/IEz WGMic+ZPZ1PPCffaq+dLri47wNhrYdDAyZzXMeFv8wsDSHXb3GaUOfmB6SGGo5rZ9GMOvrF Qd/ipcD2bKvqQ2SzjwiEA== X-UI-Out-Filterresults: notjunk:1;V03:K0:fjykjFFon7Y=:k4zwiHnq+hpWfF7QSuo43a 7311So0vkEjsOzMnQ8kstU9EzK619i2xkhs6mMe1BA9rrnNyOC8vMnmQ+hg6R4lg5YhziJ1Y6 clrnK2lowcoZgvOg43ZsqGELpjSJNki+u2tqImPDoSZqrHG4Y1kKUCjwI8mPZ4r1Jsl95BnMy 4y9Y71qQRg990LDDV19qHdtJfMgDFPLQXc+IS77nkCOEkgb/MZuQLhC6GiLIM/NSMy1gXAFfK 6JyhDgDN89gWWxV7F4kdEe9n6/Gn1MH83hASYrQZ46oiAacLZC2OaAtvorgmEK6KxPLEjLMmF 9RlftzCjqL133WJo44lJl7m6YcWeQboj8qOEifAA2Z6Nqp/tVLLdZkVEkypny7bV8qQHxzWdN 8Bxkx2J7oGizjdbP36rx2BxxbEf+6abjp1tFFnefnvWP5YuktmJXagZaYtNeOQ4sOBTKXcSCp vTsdXfU8fTd/Ea5d5mvANaUmzjmt/mow0Y/8tIZPju2pOZW7ZAPkLbHqA5tXTQBBsY73BcATp xNt3Ai0VAtc/yar+omw7W1XvR8/z5NRMPlK2nt9piPiSVBNaHcApd726XweYFU42O50uaz0PD 87NjYHmEPsTY0UtCQO4OMELJ56KNaXJA3c9NxG1AuPzRcpuw7QoehECIQY8Vl/E5tu+7YNIeX QpsweNvsp1Iojfk3E40xMGCoZy18hXuqilydEbqrgrHCs8nDIO/QYFpDtBfsfidvafuB+gmzP PwAG/1jiYm+cArM9ZhXK0L+kPFvtEu+t9PdSmMZYstXgIVV+NGr7L26OMU4BqpSPIIGei3L1H Qz1eIZL5XQKbPM6cxRQbKZ+Dvv0m+Wuw8XcwBwQCrRu16VjzDeI9hBCqWw+/w2HA+zHQBrGyC V5da/nlb92ZDY7Dr0fZAveAilErtsBsh1nstSdrbnfzN+h0klbB4uCfWL+ZHpIZiEaxGN0L+S dW5zQ6iRATtZW11M5uPhQL8cZfjus6JE2L+1v+bE20pDCmsGA75xlWoO4kfdxrd/Itd0vjJ52 VqST9Z73mzUWI+dwncGJblm5Brs9PXretiFVvkmoe6zQ3+HV8ZsYyGOn4VZijcy+kveVn3Jol 6XwVITWAzCse5QIs9McWlTKyf0K4CCVVsMua+KH6QWnHQ543+0UHzMrwg== X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,FREEMAIL_FROM,NICE_REPLY_A,RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am 21.08.22 um 17:46 schrieb Pali Roh=C3=A1r: > On Sunday 21 August 2022 17:17:13 Armin Wolf wrote: >> When dell-smm-hwmon is loaded on a machine with a buggy BIOS >> with the option "force" being enabled, it wrongly prints >> what the buggy features where disabled. This may cause >> users to wrongly assume that the driver still protects them >> from these BIOS bugs even with "force" being enabled. >> Change the warning message to avoid such a misunderstanding. > Should not there be also FW_BUG too? Since the driver itself is unable to detect if a buggy BIOS was fixed, i would not use FW_BUG here since it may cause confusion if such a message was printed on an already fixed BIOS. > I'm thinking more about message, would not it be better to print also > information if fan support and fan type call is allowed or disallowed > (based on force argument) when broken BIOS is detected? Something like "Disabling fan support due to BIOS bugs" and "Enabling fan = support despite BIOS bugs"? The first message could only be a notice message, while the second message= could be a warning message. Armin Wolf >> Tested on a Dell Inspiron 3505. >> >> Signed-off-by: Armin Wolf >> --- >> drivers/hwmon/dell-smm-hwmon.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hw= mon.c >> index f7bab1a91b93..bf13852afe48 100644 >> --- a/drivers/hwmon/dell-smm-hwmon.c >> +++ b/drivers/hwmon/dell-smm-hwmon.c >> @@ -1354,13 +1354,13 @@ static int __init dell_smm_probe(struct platfor= m_device *pdev) >> platform_set_drvdata(pdev, data); >> >> if (dmi_check_system(i8k_blacklist_fan_support_dmi_table)) { >> - dev_warn(&pdev->dev, "broken Dell BIOS detected, disallow fan suppor= t\n"); >> + dev_warn(&pdev->dev, "BIOS has broken fan support\n"); >> if (!force) >> data->disallow_fan_support =3D true; >> } >> >> if (dmi_check_system(i8k_blacklist_fan_type_dmi_table)) { >> - dev_warn(&pdev->dev, "broken Dell BIOS detected, disallow fan type c= all\n"); >> + dev_warn(&pdev->dev, "BIOS has broken fan type call\n"); >> if (!force) >> data->disallow_fan_type_call =3D true; >> } >> -- >> 2.30.2 >>