From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756592AbcCUPN6 (ORCPT ); Mon, 21 Mar 2016 11:13:58 -0400 Received: from gloria.sntech.de ([95.129.55.99]:46362 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756541AbcCUPN4 convert rfc822-to-8bit (ORCPT ); Mon, 21 Mar 2016 11:13:56 -0400 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Feng Xiao Cc: Viresh Kumar , 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 Date: Mon, 21 Mar 2016 16:13:40 +0100 Message-ID: <1971421.XfIKeB8O8v@diego> User-Agent: KMail/4.14.10 (Linux/4.4.0-1-amd64; KDE/4.14.14; x86_64; ; ) In-Reply-To: <56EFF610.6040402@rock-chips.com> References: <1458303004-26445-1-git-send-email-xf@rock-chips.com> <20160321095855.GE27778@vireshk-i7> <56EFF610.6040402@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Am Montag, 21. März 2016, 21:24:32 schrieb Feng Xiao: > 在 2016/3/21 17:58, Viresh Kumar 写道: > > 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)? > > Of_clk_init is called early, and platform_device_register_simple should > be called after devices_init, it will be failed to do it from clk-code. > So we need add a new file or add module_init to each clock controller > driver(like clk-rk3368.c, clk-rk3399.c) ? as Viresh said, it should be ok to do it like your approach creating a module in drivers/cpufreq. But the compatible check is necessary. Doing it this way also makes it easier to have > > 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. > Rockchip-cpufreq.c depends on ARM_ROCKCHIP_CPUFREQ, it will not be > compiled on non-Rockchip platforms. > The driver can support all Rockchip SoCs up to now, add > of_machine_is_compatible may be redundant ? Please always keep multiplatform in mind. These days the kernel can be compiled for multiple architectures at the same time, so you can have support for Rockchip, Exynos, Qualcom and whatever in the same kernel image. Therefore a compile-time check is not enough and you need to check the actually running machine as well. Heiko From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko =?ISO-8859-1?Q?St=FCbner?=) Date: Mon, 21 Mar 2016 16:13:40 +0100 Subject: [PATCH] cpufreq: rockchip: add driver In-Reply-To: <56EFF610.6040402@rock-chips.com> References: <1458303004-26445-1-git-send-email-xf@rock-chips.com> <20160321095855.GE27778@vireshk-i7> <56EFF610.6040402@rock-chips.com> Message-ID: <1971421.XfIKeB8O8v@diego> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Am Montag, 21. M?rz 2016, 21:24:32 schrieb Feng Xiao: > ? 2016/3/21 17:58, Viresh Kumar ??: > > 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)? > > Of_clk_init is called early, and platform_device_register_simple should > be called after devices_init, it will be failed to do it from clk-code. > So we need add a new file or add module_init to each clock controller > driver(like clk-rk3368.c, clk-rk3399.c) ? as Viresh said, it should be ok to do it like your approach creating a module in drivers/cpufreq. But the compatible check is necessary. Doing it this way also makes it easier to have > > 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. > Rockchip-cpufreq.c depends on ARM_ROCKCHIP_CPUFREQ, it will not be > compiled on non-Rockchip platforms. > The driver can support all Rockchip SoCs up to now, add > of_machine_is_compatible may be redundant ? Please always keep multiplatform in mind. These days the kernel can be compiled for multiple architectures at the same time, so you can have support for Rockchip, Exynos, Qualcom and whatever in the same kernel image. Therefore a compile-time check is not enough and you need to check the actually running machine as well. Heiko