From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next] mlxsw: core: Extend cooling device with cooling levels Date: Tue, 20 Nov 2018 10:31:28 -0800 (PST) Message-ID: <20181120.103128.1082546038469815846.davem@davemloft.net> References: <20181120065121.542-1-idosch@mellanox.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, jiri@mellanox.com, vadimp@mellanox.com, alexpe@mellanox.com, mlxsw@mellanox.com To: idosch@mellanox.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:44812 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725728AbeKUFCA (ORCPT ); Wed, 21 Nov 2018 00:02:00 -0500 In-Reply-To: <20181120065121.542-1-idosch@mellanox.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Ido Schimmel Date: Tue, 20 Nov 2018 06:52:03 +0000 > From: Vadim Pasternak > > Extend cooling device with cooling levels vector to allow more > flexibility of PWM setting. > > Thermal zone algorithm operates with the numerical states for PWM > setting. Each state is the index, defined in range from 0 to 10 and it's > mapped to the relevant duty cycle value, which is written to PWM > controller. With the current definition fan speed is set to 0% for state > 0, 10% for state 1, and so on up to 100% for the maximum state 10. > > Some systems have limitation for the PWM speed minimum. For such systems > PWM setting speed to 0% will just disable the ability to increase speed > anymore and such device will be stall on zero speed. Cooling levels > allow to configure state vector according to the particular system > requirements. For example, if PWM speed is not allowed to be below 30%, > cooling levels could be configured as 30%, 30%, 30%, 30%, 40%, 50% and > so on. > > Signed-off-by: Vadim Pasternak > Reviewed-by: Jiri Pirko > Signed-off-by: Ido Schimmel Applied.