From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755746AbcEEOAT (ORCPT ); Thu, 5 May 2016 10:00:19 -0400 Received: from foss.arm.com ([217.140.101.70]:50151 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754017AbcEEOAR (ORCPT ); Thu, 5 May 2016 10:00:17 -0400 Subject: Re: [PATCH] PM / OPP: Move CONFIG_OF dependent code in a separate file To: Viresh Kumar , Rafael Wysocki References: <1f90c873e5801967acbb2e38fe0b268189247049.1462445391.git.viresh.kumar@linaro.org> Cc: Viresh Kumar , Nishanth Menon , Stephen Boyd , Sudeep Holla , linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, eric.engestrom@imgtec.com, rufus.hamade@imgtec.com, linux-kernel@vger.kernel.org From: Sudeep Holla Organization: ARM Message-ID: <572B51EB.1080505@arm.com> Date: Thu, 5 May 2016 15:00:11 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: <1f90c873e5801967acbb2e38fe0b268189247049.1462445391.git.viresh.kumar@linaro.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/05/16 11:50, Viresh Kumar wrote: > Recently few issues were noticed in the code where CONFIG_OF wasn't > consistently used for many routines. The core file is big enough now and > ifdef hackery makes it less readable. > > Move OF specific code to another file and compile that only if CONFIG_OF > is enabled. > > Compile-tested: > - For ARM (exynos) with CONFIG_OF enabled > - For X86 with CONFIG_OF disabled (have to enable CONFIG_PM_OPP separately) > Just gave it a spin on ARM64 Juno on top of linux-pm/bleeding-edge so you can have: Tested-by: Sudeep Holla -- Regards, Sudeep