Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp1101959pxb; Fri, 13 Nov 2020 04:23:58 -0800 (PST) X-Google-Smtp-Source: ABdhPJxAUOUGFr9tJ7jH8CY6nAP1wptXg75MfhFIl3m/XT03VwR4fPBI1jQiyU/V2ZOfB3gIkoEZ X-Received: by 2002:a17:906:2ddb:: with SMTP id h27mr1625341eji.213.1605270237718; Fri, 13 Nov 2020 04:23:57 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1605270237; cv=none; d=google.com; s=arc-20160816; b=BOZzC5oUkX0ogQC6uVE9C5B9j8lWiA+NiNKfNp3WQ48ZMfkkKqCBW0DkTzE4Vyqnw9 QFIQoPP6+RhXpFXqdS5VQuQOCCrbZONgGOHYegp6QRCjsDXZP5g1cY/uRCxYqdPhk5E4 Mgfo4WVBQTlX57MC7/Ty676jduUSEo6Dse7b99OBiHs141tk9KVYjiAHGHPC1CcucXOp QIVv8Atp6384KKgcU2ETb+MuCLtGyOPExgsJdL7/cvTLnitWVArUDTMHcP0LtMVL6QLf OHUF9JbKFNROkyV2KdO0njkkgBm8nHSH6igfk43e+3Sv7dEhvAmOWv6sBMH947xPxvpx mfTA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-disposition:mime-version:message-id :subject:cc:to:from:date:dkim-signature; bh=idxWMHl88EOxCWNGY9AXKKLqzP9N71nc4LVhpxnVnGI=; b=jEfpJk7qObHKpI1Dhdd5sp/NqqCRKCJpXgF3avs5xjj0qADhUA/WlCDta/+cpqnWTL WJrISjNmcDa9kWt/qseW4nUPhBETwoTplOBQd5IiF2pZBkZg5M0f4BRa1jhdyrwg457i vNb4ZA42uQXa1+kUXJcVCB+LwrbtNNpfwK7SU3DwOj2lBoenkoxXZbTDyHPOACxw3kFB kJ8bpJKjWMWYIs3aBuLPuM9dhQd6bCDPFM9lbLvC2VKiPeFtxP819Ajycfo0lEoIu5TD 6Lv1UIO2noxltWLC40EU22gpOvhxmYO9vazu4COvKKBCvM8WCbnhQPAKDybz5WMy4EY1 oezQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@posteo.net header.s=2017 header.b="qKutQ/LJ"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=posteo.net Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id d22si5313474ejy.208.2020.11.13.04.23.34; Fri, 13 Nov 2020 04:23:57 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@posteo.net header.s=2017 header.b="qKutQ/LJ"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=posteo.net Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726437AbgKMMT6 (ORCPT + 99 others); Fri, 13 Nov 2020 07:19:58 -0500 Received: from mout02.posteo.de ([185.67.36.66]:42643 "EHLO mout02.posteo.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726376AbgKMMT6 (ORCPT ); Fri, 13 Nov 2020 07:19:58 -0500 Received: from submission (posteo.de [89.146.220.130]) by mout02.posteo.de (Postfix) with ESMTPS id 986872400FE for ; Fri, 13 Nov 2020 13:19:56 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017; t=1605269996; bh=ZEqgJv4B069sVgy/NRMeomEG6N6gQqY3dEgQs23fQk4=; h=Date:From:To:Cc:Subject:From; b=qKutQ/LJx/uU9etkGY9rKeaD03+SYwh2dbjJwAaW1m3rp9AdcsgxRynf4GwQmnogU hmaf0K32v69CD0SvGfTf9wl+9Z4KLL83cpxLVOMsoOQtGt6jpO4xSgPCvK6qIVbnOr sB5zhojE7gJ+kwLu5P1zgAcOroFyTFAgVaKL3O28I9zIKoac8qgMvB2qrR50UAm+R8 ftP/fshQiPXZrR9VDfhb/btngCbV7c0G6AOky6JI38BsTK2jtXA/hDkGZpFO5inJaq UFIc9RlNikU8OHLVs1qfMwtRvs1Je5ns7T4eXOMc4UJTnWUUTZW+WBuRWVPmIVh+O6 JRYap7hpTRDtw== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4CXcwq3RHNz9rxP; Fri, 13 Nov 2020 13:19:55 +0100 (CET) Date: Fri, 13 Nov 2020 13:19:54 +0100 From: Wilken Gottwalt To: linux-kernel@vger.kernel.org Cc: Jean Delvare , Guenter Roeck , Jonathan Corbet , linux-hwmon@vger.kernel.org Subject: [PATCH] hwmon: fix corsair-psu fan rpm calculation Message-ID: <20201113121954.GA8488@monster.powergraphx.local> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The correct fan rpm value is also a LINEAR11 value but without a factor. Verified by using the fan test button on the psu to let the fan spin up to maximum for some seconds. Fixes: 933222c98445 ("hwmon: (corsair-psu) fix unintentional sign extension issue") Signed-off-by: Wilken Gottwalt --- drivers/hwmon/corsair-psu.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/hwmon/corsair-psu.c b/drivers/hwmon/corsair-psu.c index 5d19a888231a..99494056f4bd 100644 --- a/drivers/hwmon/corsair-psu.c +++ b/drivers/hwmon/corsair-psu.c @@ -252,12 +252,7 @@ static int corsairpsu_get_value(struct corsairpsu_data *priv, u8 cmd, u8 rail, l *val = corsairpsu_linear11_to_int(tmp & 0xFFFF) * 1000; break; case PSU_CMD_FAN: - /* - * this value is best guess, so the calculated value could be wrong, it is hard - * to ge the fan to spin in these semi-passive power supplies, which need a - * quite high load to do so - */ - *val = ((tmp & 0xFF) << 8) + ((tmp >> 8) & 0xFF); + *val = corsairpsu_linear11_to_int(tmp & 0xFFFF); break; case PSU_CMD_RAIL_WATTS: case PSU_CMD_TOTAL_WATTS: -- 2.29.2