From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751564AbdFHMEf (ORCPT ); Thu, 8 Jun 2017 08:04:35 -0400 Received: from mail-wr0-f177.google.com ([209.85.128.177]:34014 "EHLO mail-wr0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751452AbdFHMEd (ORCPT ); Thu, 8 Jun 2017 08:04:33 -0400 Date: Thu, 8 Jun 2017 14:04:28 +0200 From: Daniel Lezcano To: Viresh Kumar Cc: Tao Wang , rui.zhang@intel.com, edubezval@gmail.com, amit.kachhap@gmail.com, javi.merino@kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, sunzhaosheng@hisilicon.com, vincent.guittot@linaro.org, jean.wangtao@linaro.org Subject: Re: [PATCH RFC 1/2] thermal/cpu idle cooling: Introduce cpu idle cooling driver Message-ID: <20170608120428.GC2244@mai> References: <1496653645-20283-1-git-send-email-kevin.wangtao@hisilicon.com> <20170606034135.GA5103@vireshk-i7> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170606034135.GA5103@vireshk-i7> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 06, 2017 at 09:11:35AM +0530, viresh kumar wrote: > + Daniel > > On 05-06-17, 17:07, Tao Wang wrote: > > cpu idle cooling driver performs synchronized idle injection across > > all cpu in same cluster, offers a new method to cooling down cpu, > > that is similar to intel_power_clamp driver, but is basically > > designed for ARM platform. > > Each cluster has its own idle cooling device, each core has its own > > idle injection thread, idle injection thread use play_idle to enter > > idle. In order to reach deepest idle state, all cores are aligned by > > jiffies. the injected idle ratio can be controlled through cooling > > device interface. > > I don't see any ARM specific code in this driver as stated in the description. Was it considered to make the intel_powerclamp and this driver to converge to a single generic driver?