Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932419Ab3CLOYV (ORCPT ); Tue, 12 Mar 2013 10:24:21 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:34396 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754570Ab3CLOYT (ORCPT ); Tue, 12 Mar 2013 10:24:19 -0400 Message-ID: <513F3A88.5060903@ti.com> Date: Tue, 12 Mar 2013 15:24:08 +0100 From: Benoit Cousson Organization: Texas Instruments User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130221 Thunderbird/17.0.3 MIME-Version: 1.0 To: Santosh Shilimkar , Nishanth Menon CC: cpufreq , "Rafael J. Wysocki" , Shawn Guo , , , Subject: Re: [PATCH 1/2] cpufreq: cpufreq-cpu0: support for clock which are not in DT yet. References: <1363043130-30270-1-git-send-email-nm@ti.com> <1363043130-30270-2-git-send-email-nm@ti.com> <513EB704.8030907@ti.com> In-Reply-To: <513EB704.8030907@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1446 Lines: 50 Hi Guys, On 03/12/2013 06:03 AM, Santosh Shilimkar wrote: > On Tuesday 12 March 2013 04:35 AM, Nishanth Menon wrote: >> On certain SoCs like variants of OMAP, the clock conversion to DT >> is not complete. In short, the ability to: >> cpus { >> cpu@0 { >> clocks = <&cpuclk 0>; >> }; >> }; >> is not possible. However, the clock node is registered. >> Allow for clk names to be provided as string so as to be used when needed. >> Example (for OMAP3630): >> cpus { >> cpu@0 { >> clock-name = "cpufreq_ck"; >> }; >> }; >> >> Cc: "Rafael J. Wysocki" >> Cc: Santosh Shilimkar >> Cc: Shawn Guo >> Cc: linux-kernel@vger.kernel.org >> Cc: cpufreq@vger.kernel.org >> Cc: linux-pm@vger.kernel.org >> Cc: linux-omap@vger.kernel.org >> >> Signed-off-by: Nishanth Menon >> --- > Seems a reasonable to me. No, it is not... You cannot add a temp binding just because the OMAP support is not there, since the real binding already exist. You need to register properly a clock provider to be able to reference it. If you do need a hacky temp code you could do it in OMAP code but not in the binding. Regards, Benoit -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/