From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH 00/14] cpufreq: cpu0: Extend support beyond CPU0, V2 Date: Fri, 4 Jul 2014 09:51:39 +0530 Message-ID: References: <53B4B0B3.9080000@codeaurora.org> <20140703221609.11380.6884@quantum> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Received: from mail-oa0-f52.google.com ([209.85.219.52]:39734 "EHLO mail-oa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750839AbaGDEVk (ORCPT ); Fri, 4 Jul 2014 00:21:40 -0400 Received: by mail-oa0-f52.google.com with SMTP id j17so1227871oag.25 for ; Thu, 03 Jul 2014 21:21:39 -0700 (PDT) In-Reply-To: <20140703221609.11380.6884@quantum> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Mike Turquette Cc: Stephen Boyd , "Rafael J. Wysocki" , Lists linaro-kernel , "linux-pm@vger.kernel.org" , Arvind Chauhan , linux-arm-msm@vger.kernel.org, Sachin Kamat , Thomas P Abraham , Shawn Guo , Linux Kernel Mailing List , Nishanth Menon , Tomasz Figa , "devicetree@vger.kernel.org" , Kukjin Kim , Michal Simek , Rob Herring , Santosh Shilimkar , Simon Horman On 4 July 2014 03:46, Mike Turquette wrote: > Sorry for being dense, but I still do not get why trying to dynamically > discover a shared rate-changeable clock is a better approach than simply > describing the hardware in DT? > > Is adding a property to the CPU binding that describes how the CPUs in a > cluster expect to use a clock somehow a non-starter? It is certainly a > win for readability when staring at DT and trying to understand how DVFS > on that CPU is meant to work (as opposed to hiding that knowledge behind > a tree walk). Yeah, having something like what you suggested from DT is the perfect solution to get over this. The only reason why I am not touching that here is to not delay other patches just because of that. There are separate threads going on for that and probably somebody else was trying to push for that. That's it, nothing more. I would definitely like to use those bindings instead of the crazy routines we are trying here, once that is finalized :) -- viresh