From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nishanth Menon Subject: Re: [PATCH V3 1/2] ARM: OMAP3+: use cpu0-cpufreq driver in device tree supported boot Date: Fri, 5 Apr 2013 17:23:24 -0500 Message-ID: <20130405222324.GA8732@kahuna> References: <20130405163254.GA7259@kahuna> <20130405170527.GC10155@atomide.com> <20130405171750.GA7398@kahuna> <20130405192829.GE10155@atomide.com> <20130405200201.GA7804@kahuna> <20130405211003.GK6182@atomide.com> <20130405213250.GA8534@kahuna> <20130405214051.GG10155@atomide.com> <20130405221041.GA8648@kahuna> <20130405221755.GI10155@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <20130405221755.GI10155@atomide.com> Sender: linux-omap-owner@vger.kernel.org To: Tony Lindgren Cc: Rajendra Nayak , Kevin Hilman , linux-omap , Rob Herring , cpufreq@vger.kernel.org, linux-pm@vger.kernel.org, Paul Walmsley , =?iso-8859-1?Q?Beno=EEt?= Cousson , Jon Hunter , Keerthy , Santosh Shilimkar , Shawn Guo List-Id: linux-pm@vger.kernel.org On 15:17-20130405, Tony Lindgren wrote: > * Nishanth Menon [130405 15:15]: > > Option b) leave a dummy node registered > > static inline void omap_init_cpufreq(void) > > { > > struct platform_device_info devinfo = { }; > > > > if (!of_have_populated_dt()) > > devinfo.name = "omap-cpufreq"; > > else > > devinfo.name = "cpufreq-cpu0" > > > > platform_device_register_full(&devinfo); > > } > > > > If there are no objections to (b), I dont mind it either. > > To me option (b) looks better as it's if else :) Alrity.. Going once...twice...sold! All Nay-sayers will now be branded with if-else braces ;) I will now wait for conclusion of https://lkml.org/lkml/2013/4/2/84 -- Regards, Nishanth Menon