From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH Resend] driver/core: cpu: initialize opp table Date: Mon, 19 May 2014 23:16:20 +0200 Message-ID: <1458497.gJ29RmexAd@vostro.rjw.lan> References: <60d825e8bfe01f8a5ff98fffaf51ffbf04c7d175.1400480033.git.viresh.kumar@linaro.org> <4765191.0FhyvSGyvF@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <4765191.0FhyvSGyvF@vostro.rjw.lan> Sender: linux-kernel-owner@vger.kernel.org To: Viresh Kumar Cc: linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, arvind.chauhan@arm.com, inderpal.s@samsung.com, pavel@ucw.cz, nm@ti.com, chander.kashyap@linaro.org, Greg Kroah-Hartman , Amit Daniel Kachhap , Kukjin Kim , Shawn Guo , Sudeep Holla List-Id: linux-pm@vger.kernel.org On Monday, May 19, 2014 11:13:24 PM Rafael J. Wysocki wrote: > On Monday, May 19, 2014 11:59:11 AM Viresh Kumar wrote: > > All drivers expecting CPU's OPPs from device tree initialize OPP table using > > of_init_opp_table() and there is nothing driver specific in that. They all do it > > in the same way adding to code redundancy. > > > > It would be better if we can get rid of code redundancy by initializing CPU OPPs > > from core code for all CPUs that have a "operating-points" property defined in > > their node. > > > > This patch initializes OPPs as soon as CPU device is registered in > > register_cpu(). > > > > Cc: Greg Kroah-Hartman > > Cc: Amit Daniel Kachhap > > Cc: Kukjin Kim > > Cc: Shawn Guo > > Cc: Sudeep Holla > > Signed-off-by: Viresh Kumar Do patches [2-5/5] depend on this one BTW? -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.