From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH 2/2] cpufreq: dt-platdev: Drop few entries from whitelist Date: Wed, 16 Aug 2017 16:54:16 +0530 Message-ID: <20170816112416.GD24299@vireshk-i7> References: <7182e308a552d37064f61e724c709667a6fa098d.1502861444.git.viresh.kumar@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pf0-f177.google.com ([209.85.192.177]:33272 "EHLO mail-pf0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751522AbdHPLYT (ORCPT ); Wed, 16 Aug 2017 07:24:19 -0400 Received: by mail-pf0-f177.google.com with SMTP id h68so5115019pfk.0 for ; Wed, 16 Aug 2017 04:24:18 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Chen-Yu Tsai Cc: Rafael Wysocki , "open list:THERMAL" , Vincent Guittot , Geert Uytterhoeven , Maxime Ripard , Wei Xu , Kukjin Kim , Krzysztof Kozlowski , Masahiro Yamada , jun.nie@linaro.org, Baoyou Xie , linux-kernel On 16-08-17, 16:53, Chen-Yu Tsai wrote: > On Wed, Aug 16, 2017 at 1:37 PM, Viresh Kumar wrote: > > Drop few ARM (32 and 64 bit) platforms from the whitelist which always > > use "operating-points-v2" property from their DT. They should continue > > to work after this patch. > > > > Tested on Hikey platform (only the "hisilicon,hi6220" entry). > > > > Signed-off-by: Viresh Kumar > > --- > > drivers/cpufreq/cpufreq-dt-platdev.c | 11 ----------- > > 1 file changed, 11 deletions(-) > > > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > > index 061b468512a2..45f2ec3b7f7a 100644 > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > > @@ -27,7 +27,6 @@ static const struct of_device_id whitelist[] __initconst = { > > { .compatible = "allwinner,sun6i-a31s", }, > > { .compatible = "allwinner,sun7i-a20", }, > > { .compatible = "allwinner,sun8i-a23", }, > > - { .compatible = "allwinner,sun8i-a33", }, > > { .compatible = "allwinner,sun8i-a83t", }, > > { .compatible = "allwinner,sun8i-h3", }, > > > > Acked-by: Chen-Yu Tsai > > In fact, we can drop all the sun8i entries. Of them, only sun8i-a33 has > cpufreq implemented. All the other ones are missing thermal sensor support, > and no OPP tables have been added yet. Then, why did you guys add all those SoCs there ? :) -- viresh