From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752941AbcFUSFR (ORCPT ); Tue, 21 Jun 2016 14:05:17 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:17621 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752907AbcFUSFO (ORCPT ); Tue, 21 Jun 2016 14:05:14 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Tue, 21 Jun 2016 11:04:32 -0700 Subject: Re: [PATCH v2 00/11] Switch to using CRITICAL clks for Tegra To: Peter De Schrijver , Thierry Reding References: <1464381494-27096-1-git-send-email-rklein@nvidia.com> CC: Michael Turquette , Stephen Boyd , Alexandre Courbot , , , , Stephen Warren From: Rhyland Klein Message-ID: Date: Tue, 21 Jun 2016 14:05:03 -0400 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <1464381494-27096-1-git-send-email-rklein@nvidia.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/27/2016 4:38 PM, Rhyland Klein wrote: > Switch to defining clks that need to be on as CRITICAL rather than > using the init_tables defined to enable clks. Some of these may be > able to be converted to HAND_OFF clks, when that is supported. > > I added a patch to mark CRITICAL clks in the clk_summary table, figuring > it would be handy to be able to easily tell what clks are CRITICAL > through the table. > > I added a warning for using the init_table to enable clks. I considered > removing it entirely, but I thought phasing it out might be better. > > Rhyland Klein (11): > clk: tegra: Switch to using critical clks > clk: tegra20: Mark required clks as CRITICAL > clk: tegra20: clean up init_table > clk: tegra30: Mark certain clks as critical > clk: tegra30: clean up init_table > clk: tegra114: clean up init_table > clk: tegra124: clean up init_table > clk: tegra210: Mark required clks as CRITICAL > clk: tegra210: clean up init_table > clk: Show CRITICAL clks in clk_summary output > clk: tegra: WARN if clk in the init_table has enable > > drivers/clk/clk.c | 7 ++++-- > drivers/clk/tegra/clk-tegra-periph.c | 21 ++++++++++------ > drivers/clk/tegra/clk-tegra-super-gen4.c | 12 ++++++---- > drivers/clk/tegra/clk-tegra114.c | 11 ++++----- > drivers/clk/tegra/clk-tegra124.c | 19 ++++++--------- > drivers/clk/tegra/clk-tegra20.c | 41 +++++++++++++++----------------- > drivers/clk/tegra/clk-tegra210.c | 28 ++++++++-------------- > drivers/clk/tegra/clk-tegra30.c | 28 ++++++++-------------- > drivers/clk/tegra/clk.c | 5 ++++ > 9 files changed, 81 insertions(+), 91 deletions(-) > Has anyone had a change to take a look at these patches yet? -rhyland -- nvpublic