From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [RFC] cpufreq: Add "dvfs-method" binding to probe cpufreq drivers Date: Thu, 27 Nov 2014 10:44:45 +0530 Message-ID: References: <596a6d49f2b3e2837aa9a54a3e1249161d3c9265.1416991009.git.viresh.kumar@linaro.org> <54760107.3010205@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Received: from mail-ob0-f176.google.com ([209.85.214.176]:44481 "EHLO mail-ob0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753701AbaK0FOq (ORCPT ); Thu, 27 Nov 2014 00:14:46 -0500 Received: by mail-ob0-f176.google.com with SMTP id vb8so3219728obc.7 for ; Wed, 26 Nov 2014 21:14:45 -0800 (PST) In-Reply-To: <54760107.3010205@oracle.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: santosh shilimkar Cc: Rafael Wysocki , Lists linaro-kernel , "linux-pm@vger.kernel.org" , Nishanth Menon , Sudeep Holla , Stephen Boyd , "devicetree@vger.kernel.org" , Lorenzo Pieralisi , Arnd Bergmann , Mike Turquette , Rob Herring , Grant Likely , Abhilash Kesavan , Catalin Marinas , Chander Kashyap , "olof@lixom.net" , Thomas Abraham On 26 November 2014 at 22:04, santosh shilimkar wrote: > Its really not 'dvfs-method' but really the actual driver which you > want to probe. Also we should just have one global way to parse > DT vs non-DT cpufreq drivers. In other words, instead of matching > multiple driver strings for different drivers, we should come up with > slightly generic binding. Probably 'cpufreq-dt' for all DT based probed > CPUFREQ drivers. > > What you say ? I got a bit confused. :( The whole purpose of this binding is to select which cpufreq driver to choose from the available ones. Are you saying that we should have only one cpufreq driver that works with DT ? I don't think that would be the right decision, but we should try to reuse cpufreq-dt wherever possible. Or are you saying that some common code should parse this DT information? This is what I said in the patch, we can write another file which will just create a platform_device for us, depending on the string that came in.