From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754143AbcCUJ7G (ORCPT ); Mon, 21 Mar 2016 05:59:06 -0400 Received: from mail-pf0-f181.google.com ([209.85.192.181]:33714 "EHLO mail-pf0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752925AbcCUJ66 (ORCPT ); Mon, 21 Mar 2016 05:58:58 -0400 Date: Mon, 21 Mar 2016 15:28:55 +0530 From: Viresh Kumar To: Heiko =?iso-8859-1?Q?St=FCbner?= Cc: Feng Xiao , linux@arm.linux.org.uk, rjw@rjwysocki.net, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, wxt@rock-chips.com, zyw@rock-chips.com, jay.xu@rock-chips.com, tim.chen@rock-chips.com, xxx@rock-chips.com, huangtao@rock-chips.com, Stephen Boyd , Michael Turquette Subject: Re: [PATCH] cpufreq: rockchip: add driver Message-ID: <20160321095855.GE27778@vireshk-i7> References: <1458303004-26445-1-git-send-email-xf@rock-chips.com> <1787319.AFAWVqkdk0@diego> <20160321095049.GD27778@vireshk-i7> <1505152.fhSnu3pvDr@diego> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1505152.fhSnu3pvDr@diego> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 21-03-16, 10:54, Heiko Stübner wrote: > I hadn't seen that yet ... nice that cpufreq-dt now also supports clusters :-) > > The other part still stands though, as we probably should register the > platform-device somewhere else and not in some new special module. > > When everything is using cpufreq-dt now, I guess we could just add it to the > core rockchip clk-code. Or was there some agreement where this should be done > (obviously not the devicetree itself)? Yeah, there was a discussion around creating a white or black list of platforms that want to create a platform device for cpufreq-dt. That can be done in cpufreq-dt.c or a new file, but I haven't worked out on that yet. You can do it from clk-code or from the driver that was added in this thread. Just that you need to match your platform's compatible string before doing that. -- viresh