From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934036AbeAJOqh (ORCPT + 1 other); Wed, 10 Jan 2018 09:46:37 -0500 Received: from mail-pg0-f68.google.com ([74.125.83.68]:34467 "EHLO mail-pg0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751352AbeAJOqf (ORCPT ); Wed, 10 Jan 2018 09:46:35 -0500 X-Google-Smtp-Source: ACJfBos49lvpsn7Wm72QMVoKdvQztJNoR2lFhygPuSC0GJITc9XOsF7HyPT78vs/yioSaOAcTUjYNg== Date: Wed, 10 Jan 2018 20:16:31 +0530 From: Viresh Kumar To: Sudeep Holla Cc: linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Jeremy Linton , Lorenzo Pieralisi , "Rafael J. Wysocki" Subject: Re: [PATCH 2/2] cpufreq: scpi: remove arm_big_little dependency Message-ID: <20180110144631.GE3626@vireshk-i7> References: <1515516568-31359-1-git-send-email-sudeep.holla@arm.com> <1515516568-31359-3-git-send-email-sudeep.holla@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1515516568-31359-3-git-send-email-sudeep.holla@arm.com> 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 Return-Path: On 09-01-18, 16:49, Sudeep Holla wrote: > The dependency on physical_package_id from the topology to get the > cluster identifier is wrong. The concept of cluster used in ARM topology > is unfortunately not well defined in the architecture, we should avoid > using it. Further the frequency domain need not be mapped to so called > "clusters" one to one. > > SCPI already provides means to obtain the frequency domain id from the > device tree. In order to support some new topologies(e.g. DSU which > contains 2 frequency domains within the physical cluster), pseudo > clusters are created to make this driver work which is wrong again. > > In order to solve those issues and also remove dependency of topological > physical id for frequency domain, this patch removes the arm_big_little > dependency from scpi driver. > > Cc: "Rafael J. Wysocki" > Cc: Viresh Kumar > Signed-off-by: Sudeep Holla > --- > drivers/cpufreq/scpi-cpufreq.c | 193 +++++++++++++++++++++++++++++++++++++---- > 1 file changed, 178 insertions(+), 15 deletions(-) Acked-by: Viresh Kumar -- viresh