Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp528306pxb; Wed, 27 Jan 2021 14:00:41 -0800 (PST) X-Google-Smtp-Source: ABdhPJwYuv0TEY6sYe46VRU7babfy9/p8T/kr2oy81TpcqyM8EJ/4aaDoa1LQej8fQEhjAXy6xvb X-Received: by 2002:a05:6402:3589:: with SMTP id y9mr11108301edc.344.1611784841214; Wed, 27 Jan 2021 14:00:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1611784841; cv=none; d=google.com; s=arc-20160816; b=oUsbT86Ueb4bUxOlHdm5yY8ZQF7xFweEtUyLMl/Us1loaU0pDccoxSn7ZKsT/8eKiI y6s/yffMJguT7CIs8jlP4VkbIHG6b2/KhUJj0eQOw4wU/NIFhInjP8fSrLeyp8zYtsz8 ttj/AxHP/IudnRJ+EJSIL5U9BYxh19xlijq13cYXTCvh0xdWbH2SiORCZ0g3Xh1FllIh 7lTfEDr6fE7K8q0a1MRVJ5ZWBfZ1x64JptncsYVHSAxaLgqptJb6Ft4rwhheKbxY9SxV AfuCvp5q12VVg7zCEF0vzSBj0nb2+eobXcEqKBphdqFKZKcvu+xPBamZJEO3qyVgxA4d iKHg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=eyFAQbZ1vGq54izfXLSYq/bJfr57Ytsvd3Yk8EecG5g=; b=AyhP60I3bEutvJNSvZ4zBCcMQtbph60Cj3k51VydJ30fO5Jxjsoc+/JmOees6nb0m1 04PWOjfs5ZlGUu05dXK5N5ZYN9eCIcF7qDmWxi0CEWT4WGgTFck46uJdWsPqnC3kD1Am pHDTgZ2E2E8z433E5QIumWq1RNkJ5TNW1WBU7LhVBtQ2U0eDl1jliiolkBCviY7bvHTK yvo6S8Usee2FnWTq13H9QZRmKAgDd+JdFgOSAcTZOAB9buEPi6RU9WvAzuWeMxnt5ksZ UdyWcEMkmQX6aB1Fzjzc1apqupjNlO+W7tpyAjBdZvjax35lxcW7P5NqbUWJlOfOz37s 2aiQ== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q19si1423963eja.19.2021.01.27.14.00.16; Wed, 27 Jan 2021 14:00:41 -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; 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=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235683AbhA0KPY (ORCPT + 99 others); Wed, 27 Jan 2021 05:15:24 -0500 Received: from foss.arm.com ([217.140.110.172]:36810 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235688AbhA0KMv (ORCPT ); Wed, 27 Jan 2021 05:12:51 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 8657A31B; Wed, 27 Jan 2021 02:11:56 -0800 (PST) Received: from [10.57.4.29] (unknown [10.57.4.29]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id EA1D23F66B; Wed, 27 Jan 2021 02:11:52 -0800 (PST) Subject: Re: [RFC][PATCH 0/3] New thermal interface allowing IPA to get max power To: Viresh Kumar Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, vireshk@kernel.org, rafael@kernel.org, daniel.lezcano@linaro.org, Dietmar.Eggemann@arm.com, amitk@kernel.org, rui.zhang@intel.com, cw00.choi@samsung.com, myungjoo.ham@samsung.com, kyungmin.park@samsung.com References: <20210126104001.20361-1-lukasz.luba@arm.com> <20210127091540.xesvwoeavyaf37jn@vireshk-i7> From: Lukasz Luba Message-ID: <9aecd2cd-771e-58b8-6672-f133600b70b5@arm.com> Date: Wed, 27 Jan 2021 10:11:50 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <20210127091540.xesvwoeavyaf37jn@vireshk-i7> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/27/21 9:15 AM, Viresh Kumar wrote: > On 26-01-21, 10:39, Lukasz Luba wrote: >> As it's a RFC, it still misses the cpufreq sysfs implementation, but would >> be addressed if all agree. > > Not commenting on the whole stuff but if you ever need something for cpufreq, it > is already there. Look for these. > > store_one(scaling_min_freq, min); > store_one(scaling_max_freq, max); > > Hopefully they will work just fine. > So, can I assume you don't mind to plumb it into these two? Yes, I know them and the tricky macro. I just wanted to avoid one patch for this macro and one patch for cpufreq_cooling.c, which would use it. If you agree and Chanwoo agrees for the devfreq, and Daniel for the new callback in cooling device, then I would continue by adding missing patches for cpufreq-cooling part. Regards, Lukasz