From patchwork Sat Feb 5 10:36:07 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Jonathan_Neusch=C3=A4fer?= X-Patchwork-Id: 12736040 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 76570C433EF for ; Sat, 5 Feb 2022 10:38:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=k/0lrjQ1pRPJXZi0HljwVRgEcQJrlSKjajAcnHOu3OE=; b=FCZEJ5kesvk6wj S3IDSSrtF7JzOH5/vM5gXYfGtW9Q5+qqiGk0O2tAroVOQsdU+PiKVZSXxOUow2i3XA5z5O8/PK0Nl Z5iBfmi4PR3n/U2soKDiR0gayZe/EMxWEaMBxuSXjyKhnhUpezdKgFWiEudurLivLDlwseCOuNYDr YOQUVXg20sj5gVnAGvAeFtZWFHPJOBB9xGA0yF0LtNtgcEYdC13F2ZG4fM7NSCUSnyvdYdG6Nuf+k CSHZLDDflXYj5BfYHRsG6X8uos//8lP5Oh0Hfw6jHwOYtG8/ynKzwucq24qF2Qzb3vvNvr4lbgcm1 wqNulHS/DYFABHsKiGXA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nGIRK-006LgN-Po; Sat, 05 Feb 2022 10:37:11 +0000 Received: from mout.gmx.net ([212.227.15.18]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nGIQv-006LX1-Jy for linux-arm-kernel@lists.infradead.org; Sat, 05 Feb 2022 10:36:47 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1644057395; bh=3hZ576UTl3/lTq+uXDRzn9J611V/XEzqawADOC1QOxE=; h=X-UI-Sender-Class:From:To:Cc:Subject:Date:In-Reply-To:References; b=lfQ2S/lUVhYnB0SrSbEUooYGCzSV9SmLiP+0K3rlclLiDG3Y93oAU7r4TWQEKHBzc P6aDjtGT0ONnQDrYJOgmEo25SLYc7XcoHxq9jicDuS9bafYQgrTwJP3KMB3p+KDJ5p nBTPWNDRbWHMExVzHEd+wQMx9qStsYatnfZr4wTQ= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from longitude ([5.146.194.160]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1MIdiZ-1n2IBr3nAT-00Ebbo; Sat, 05 Feb 2022 11:36:35 +0100 From: =?utf-8?q?Jonathan_Neusch=C3=A4fer?= To: linux-clk@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, =?utf-8?q?Jonathan_Neusch=C3=A4fer?= , kernel test robot , Michael Turquette , Stephen Boyd , Vladimir Zapolskiy , Joachim Eastwood Subject: [PATCH v2 1/7] clk: nxp: Remove unused variable Date: Sat, 5 Feb 2022 11:36:07 +0100 Message-Id: <20220205103613.1216218-2-j.neuschaefer@gmx.net> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220205103613.1216218-1-j.neuschaefer@gmx.net> References: <20220205103613.1216218-1-j.neuschaefer@gmx.net> MIME-Version: 1.0 X-Provags-ID: V03:K1:q6a4fF67vwV9Q2gA4ypPC6NgQDlJQUlmeAr9vKgZCkZuqBEySG2 2ofC6Tkp4doIVDc6chHBxl1+2bcpp2Ut7kRfNOKj7AvLc2y/ntflmq58yay2npkRurLR4pc QjWyVukMyWreSL3A0hw18bSFrPvqkk13c5w+0is7GXgtLN/dIgMUPnoSTZTLpvPkqJgXf54 fly8BWQOHGGypjV8VxB4g== X-UI-Out-Filterresults: notjunk:1;V03:K0:4G9LrCCCF/w=:gWZbduS6zWFfd6ebgvcF/M 4qnuWJ2a/sg7nhNLMzxxPyRc6K6LLJ9dmEQzO5YytyPLYhfhOmXMFZdFjqAAPs1ZL9aCWNox3 ky1ie+KMmYt1MvlyVH3rc8yOdfWToU0b2PfPHgBPNzFd4hIt3YgMOVddpFIbxaeBcat5yNe7R iMpkf/3lNgnfu+u+cLhWEb1BGAS5a86wUOkqzKOivfb43BEQUm0ubyulAl6oOBXX62Mj9P5Et cSL5IFWDUeruzQNMTbWIsVPtBh9SeOcaYIz8HyQJZZT+bBoj2a11o+FRgk/JH9ABLDFFbf01l WbIg7OCc0LUdFrF2DihU6MdISaoJwjAucqmQn4hvvvEbWxDF520wN3WaJ47utLpdsCcVk29IY Qpivs0nXxobg61Sz2M6jM0lW4d/JlfOwQewLqaQIRmUzlC6E/86ijAZ1H22158YR2P11whBOy BkzXnHIRk/cAmEeAn98PaGjpPDlkvLPFKkmfsOsrC0DEo3RtrzWMfR2XZpeTl6Rtly5iLYcHt VhDR1SKG3FBVdjizJgvc4XMAX7OmiBlJV+Uk3ryi5rPsWLBiBBFnPF6LTSMrRCo+kD3R9YnUu L18Pmycj6BPe/RdZR7TXumYpwMUy+AeRwfAjTpSixrihgNiZ7jTm2qQo8fPIXTfSn4dXQNux/ at5He9OZEbovl3j7wUjVesOlrgBOT6DJUWOj6S+TVs6Ky9V8dRJjVwLjCDNSTo4Y5siXl3+WP 3BJOdYsDtbiELym1FRfS1TYxYcP0ulUmP7WqJLkjtQYzrHvbPvnAhX5Y+pNF906tm2Ik4U3bS sX9xdkyEE/ttloZKK9WLFzk+IcoZ8A29QUHNtRwILQrEievTdov8zvhIa4TmIxl8RUru51xTy GJLUhlwdhmz2kjDCqLCV0q02wYwK/Q0uyy/Se1RQzne5nncPPeydpiF2PcebqSic8HJebn+Io AHJBVyIskHzOBuWsOzb/xPttirz/LYr6AEyPLODFcL+nctHV5JmKIjUB3FwXHNFetfRpoqp56 aXFGKEcQ/r5Sc2raktEfDP6si4quW6B0KMyTM6lyZiOF+8cLi0hw+rlkJ87FgJBURYP0rbb4G jBdv6b/kOJAr0o= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220205_023645_975774_6F34AC79 X-CRM114-Status: GOOD ( 12.82 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org GCC warns: > ../drivers/clk/nxp/clk-lpc18xx-cgu.c: In function ‘lpc18xx_pll1_recalc_rate’: > ../drivers/clk/nxp/clk-lpc18xx-cgu.c:460:13: warning: variable ‘stat’ set but not used [-Wunused-but-set-variable] > 460 | u32 stat, ctrl; > | ^~~~ Get rid of the (apparently) useless read from the PLL1_STAT register and the declaration of stat. Reported-by: kernel test robot Fixes: b04e0b8fd5443 ("clk: add lpc18xx cgu clk driver") Signed-off-by: Jonathan Neuschäfer --- If the register read is actually useful because of a side-effect, please speak up. v2: - New patch to address this warning --- drivers/clk/nxp/clk-lpc18xx-cgu.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) -- 2.34.1 diff --git a/drivers/clk/nxp/clk-lpc18xx-cgu.c b/drivers/clk/nxp/clk-lpc18xx-cgu.c index 8b686da5577b3..349eebf69ed93 100644 --- a/drivers/clk/nxp/clk-lpc18xx-cgu.c +++ b/drivers/clk/nxp/clk-lpc18xx-cgu.c @@ -457,9 +457,8 @@ static unsigned long lpc18xx_pll1_recalc_rate(struct clk_hw *hw, struct lpc18xx_pll *pll = to_lpc_pll(hw); u16 msel, nsel, psel; bool direct, fbsel; - u32 stat, ctrl; + u32 ctrl; - stat = readl(pll->reg + LPC18XX_CGU_PLL1_STAT); ctrl = readl(pll->reg + LPC18XX_CGU_PLL1_CTRL); direct = (ctrl & LPC18XX_PLL1_CTRL_DIRECT) ? true : false;