From patchwork Thu Sep 28 17:50:38 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Abhishek Sahu X-Patchwork-Id: 9976577 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 6B3CF60365 for ; Thu, 28 Sep 2017 17:54:54 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5C65026E3A for ; Thu, 28 Sep 2017 17:54:54 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 5166128A10; Thu, 28 Sep 2017 17:54:54 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id ACFD526E3A for ; Thu, 28 Sep 2017 17:54:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752449AbdI1RvI (ORCPT ); Thu, 28 Sep 2017 13:51:08 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:50184 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751339AbdI1RvE (ORCPT ); Thu, 28 Sep 2017 13:51:04 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id B7D2360A0C; Thu, 28 Sep 2017 17:51:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1506621063; bh=wl3Dr2UbhLVyX7/DLmEhXrwzBAJBZEO0wr8lEjYr0Y4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=gLJie1EVonjUsbvX7d4/Sx9Wu3lKMR4HtWrIqcu5qNumBENa76LFrT9rvIXgluy4q Kzda2uosDYxmKcPVf7tyqISjHGHpHaaK/BDhGgogv6m76hTH+5qOByq9v0ov1HuUxF 36+PbwmeBlNnIRj9S/M1qUMia2oUnYXO+MgkMyoI= Received: from absahu-linux.qualcomm.com (blr-c-bdr-fw-01_globalnat_allzones-outside.qualcomm.com [103.229.19.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: absahu@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 6932A6071B; Thu, 28 Sep 2017 17:51:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1506621063; bh=wl3Dr2UbhLVyX7/DLmEhXrwzBAJBZEO0wr8lEjYr0Y4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=gLJie1EVonjUsbvX7d4/Sx9Wu3lKMR4HtWrIqcu5qNumBENa76LFrT9rvIXgluy4q Kzda2uosDYxmKcPVf7tyqISjHGHpHaaK/BDhGgogv6m76hTH+5qOByq9v0ov1HuUxF 36+PbwmeBlNnIRj9S/M1qUMia2oUnYXO+MgkMyoI= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 6932A6071B Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=absahu@codeaurora.org From: Abhishek Sahu To: Stephen Boyd , Michael Turquette Cc: Andy Gross , David Brown , Rajendra Nayak , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Abhishek Sahu Subject: [PATCH 01/13] clk: qcom: remove redundant PLL_MODE macro offset Date: Thu, 28 Sep 2017 23:20:38 +0530 Message-Id: <1506621050-10129-2-git-send-email-absahu@codeaurora.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1506621050-10129-1-git-send-email-absahu@codeaurora.org> References: <1506621050-10129-1-git-send-email-absahu@codeaurora.org> Sender: linux-clk-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP The PLL_MODE offset macro is redundant which is defined as zero. The offset in PLL structure is the address of PLL_MODE register itself so the PLL_MODE can be removed. It will help in subsequent patches to support different PLL offset registers to reduce the code diff. Signed-off-by: Abhishek Sahu --- drivers/clk/qcom/clk-alpha-pll.c | 33 ++++++++++++++++----------------- 1 file changed, 16 insertions(+), 17 deletions(-) diff --git a/drivers/clk/qcom/clk-alpha-pll.c b/drivers/clk/qcom/clk-alpha-pll.c index 47a1da3..b9be822 100644 --- a/drivers/clk/qcom/clk-alpha-pll.c +++ b/drivers/clk/qcom/clk-alpha-pll.c @@ -20,7 +20,6 @@ #include "clk-alpha-pll.h" #include "common.h" -#define PLL_MODE 0x00 # define PLL_OUTCTRL BIT(0) # define PLL_BYPASSNL BIT(1) # define PLL_RESET_N BIT(2) @@ -77,12 +76,12 @@ static int wait_for_pll(struct clk_alpha_pll *pll, u32 mask, bool inverse, const char *name = clk_hw_get_name(&pll->clkr.hw); off = pll->offset; - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return ret; for (count = 100; count > 0; count--) { - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return ret; if (inverse && !(val & mask)) @@ -139,7 +138,7 @@ void clk_alpha_pll_configure(struct clk_alpha_pll *pll, struct regmap *regmap, regmap_update_bits(regmap, off + PLL_USER_CTL, mask, val); if (pll->flags & SUPPORTS_FSM_MODE) - qcom_pll_set_fsm_mode(regmap, off + PLL_MODE, 6, 0); + qcom_pll_set_fsm_mode(regmap, off, 6, 0); } static int clk_alpha_pll_hwfsm_enable(struct clk_hw *hw) @@ -149,7 +148,7 @@ static int clk_alpha_pll_hwfsm_enable(struct clk_hw *hw) struct clk_alpha_pll *pll = to_clk_alpha_pll(hw); off = pll->offset; - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return ret; @@ -158,7 +157,7 @@ static int clk_alpha_pll_hwfsm_enable(struct clk_hw *hw) if (pll->flags & SUPPORTS_OFFLINE_REQ) val &= ~PLL_OFFLINE_REQ; - ret = regmap_write(pll->clkr.regmap, off + PLL_MODE, val); + ret = regmap_write(pll->clkr.regmap, off, val); if (ret) return ret; @@ -175,12 +174,12 @@ static void clk_alpha_pll_hwfsm_disable(struct clk_hw *hw) struct clk_alpha_pll *pll = to_clk_alpha_pll(hw); off = pll->offset; - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return; if (pll->flags & SUPPORTS_OFFLINE_REQ) { - ret = regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, + ret = regmap_update_bits(pll->clkr.regmap, off, PLL_OFFLINE_REQ, PLL_OFFLINE_REQ); if (ret) return; @@ -191,7 +190,7 @@ static void clk_alpha_pll_hwfsm_disable(struct clk_hw *hw) } /* Disable hwfsm */ - ret = regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, + ret = regmap_update_bits(pll->clkr.regmap, off, PLL_FSM_ENA, 0); if (ret) return; @@ -206,7 +205,7 @@ static int pll_is_enabled(struct clk_hw *hw, u32 mask) struct clk_alpha_pll *pll = to_clk_alpha_pll(hw); off = pll->offset; - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return ret; @@ -232,7 +231,7 @@ static int clk_alpha_pll_enable(struct clk_hw *hw) off = pll->offset; mask = PLL_OUTCTRL | PLL_RESET_N | PLL_BYPASSNL; - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return ret; @@ -248,7 +247,7 @@ static int clk_alpha_pll_enable(struct clk_hw *hw) if ((val & mask) == mask) return 0; - ret = regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, + ret = regmap_update_bits(pll->clkr.regmap, off, PLL_BYPASSNL, PLL_BYPASSNL); if (ret) return ret; @@ -260,7 +259,7 @@ static int clk_alpha_pll_enable(struct clk_hw *hw) mb(); udelay(5); - ret = regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, + ret = regmap_update_bits(pll->clkr.regmap, off, PLL_RESET_N, PLL_RESET_N); if (ret) return ret; @@ -269,7 +268,7 @@ static int clk_alpha_pll_enable(struct clk_hw *hw) if (ret) return ret; - ret = regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, + ret = regmap_update_bits(pll->clkr.regmap, off, PLL_OUTCTRL, PLL_OUTCTRL); /* Ensure that the write above goes through before returning. */ @@ -285,7 +284,7 @@ static void clk_alpha_pll_disable(struct clk_hw *hw) off = pll->offset; - ret = regmap_read(pll->clkr.regmap, off + PLL_MODE, &val); + ret = regmap_read(pll->clkr.regmap, off, &val); if (ret) return; @@ -296,14 +295,14 @@ static void clk_alpha_pll_disable(struct clk_hw *hw) } mask = PLL_OUTCTRL; - regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, mask, 0); + regmap_update_bits(pll->clkr.regmap, off, mask, 0); /* Delay of 2 output clock ticks required until output is disabled */ mb(); udelay(1); mask = PLL_RESET_N | PLL_BYPASSNL; - regmap_update_bits(pll->clkr.regmap, off + PLL_MODE, mask, 0); + regmap_update_bits(pll->clkr.regmap, off, mask, 0); } static unsigned long alpha_pll_calc_rate(u64 prate, u32 l, u32 a)