Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753842AbbERLGf (ORCPT ); Mon, 18 May 2015 07:06:35 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:13361 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752153AbbERLG1 (ORCPT ); Mon, 18 May 2015 07:06:27 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Mon, 18 May 2015 04:04:33 -0700 Message-ID: <5559C7BD.7040105@nvidia.com> Date: Mon, 18 May 2015 19:06:37 +0800 From: bilhuang User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Benson Leung CC: Peter De Schrijver , Mike Turquette , Stephen Warren , "Thierry Reding" , Paul Walmsley , , , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/1] clk: tegra: fix WARN_ON in PLL_RE registration References: <1431691653-17615-1-git-send-email-bilhuang@nvidia.com> In-Reply-To: X-Originating-IP: [10.19.108.124] X-ClientProxiedBy: HKMAIL104.nvidia.com (10.18.16.13) To HKMAIL103.nvidia.com (10.18.16.12) Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1808 Lines: 52 On 05/16/2015 01:12 AM, Benson Leung wrote: > On Fri, May 15, 2015 at 5:07 AM, Bill Huang wrote: >> This fixes two things. >> >> - Read the correct IDDQ register >> - Check the correct IDDQ bit position >> >> Signed-off-by: Bill Huang > > Reviewed-by: Benson Leung > > By the way, does it also make sense to do the same thing for > tegra_clk_register_pllss, which also reads the base register instead > of the specific iddq_reg from params? > Yes thanks for catching this, I've sent another fix in https://patchwork.ozlabs.org/patch/473329/ >> --- >> drivers/clk/tegra/clk-pll.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/clk/tegra/clk-pll.c b/drivers/clk/tegra/clk-pll.c >> index 05c6d08..734340e 100644 >> --- a/drivers/clk/tegra/clk-pll.c >> +++ b/drivers/clk/tegra/clk-pll.c >> @@ -1630,7 +1630,8 @@ struct clk *tegra_clk_register_pllre(const char *name, const char *parent_name, >> >> val = pll_readl_base(pll); >> if (val & PLL_BASE_ENABLE) >> - WARN_ON(val & pll_params->iddq_bit_idx); >> + WARN_ON(readl_relaxed(clk_base + pll_params->iddq_reg) & >> + BIT(pll_params->iddq_bit_idx)); >> else { >> int m; >> >> -- >> 1.9.1 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > > -- 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/