From mboxrd@z Thu Jan 1 00:00:00 1970 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 9E20BC6FD20 for ; Tue, 21 Mar 2023 19:31:36 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8c0ZJu9swfibaDM45ZA4zZfiXUcD2wpJi3u6YiNSfH0=; b=YUkwTD/h/wm82k qzriuintvCYaAUN5aufcBbYzw86gzX29XeeCaRWzOQXjL4xZ5wOKXkSwxMEtNR68mPfSVaY+vFo/Y HiAleVizbavq3WXK0ZqDSn2XxokdtMd9xaqOe85ToDSWLLo5GM3ztS4nLLp0QZ9bXrhaD0xyZ+Krz qaBAZ3/slyZwuJZXgLqzURgrdyybjm0Q8PbaVn0VZTWcSUAACnxpAoLv/FVhPwNCT64PQT4Z3wOs6 +1Rr+WVxHVMc3tq4SGuEG2wBGBdwiWhCFVV+cOxGJ5Fn63uiepmeDPV4yg81MIN8hll9F2v+ZkvVr kkwPV8XBoYsR1cMVqlKQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pehgJ-00DTif-1j; Tue, 21 Mar 2023 19:30:03 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pehgE-00DThE-2H for linux-arm-kernel@lists.infradead.org; Tue, 21 Mar 2023 19:30:02 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 31EE3B81988; Tue, 21 Mar 2023 19:29:54 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 461AAC433D2; Tue, 21 Mar 2023 19:29:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1679426992; bh=OKN8kbSWPpvrhUxtqKypraEhkhHF4MVPJBKr6a5Rd8M=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nZsG76/AKO+1mjoywTNx+ON7pPgx4fKOyDHJ1IHyP09vljcFCuVPkqr0mB5SKRAj4 GggNjVDUkd/BY5WV9ynO5fRx6d1/F9iGU9A3gjEX5dzLOvGUItc5d+mFQnp3E8U7S7 f11PN0v5bvMQuhY/rBLjpbuxwKaH9cXPU2lVmODR/60cVVeNR1tyXtsTWFK/eD/bAk BbscdXNkpzbb7ZN+j5miO+0EfQEdRst+tOybJU1/qo8mw/YBaEq2hnApnku+891vPJ tVccg84ybCoWmF20YG5Vp3GBqN3txAH/U4rkBTaICc0j8vKAfuHDIm8eUnhLr6qdoR b8dnOVavu9e1g== Date: Tue, 21 Mar 2023 20:29:49 +0100 From: Andi Shyti To: Tom Rix Cc: krzysztof.kozlowski@linaro.org, alim.akhtar@samsung.com, wim@linux-watchdog.org, linux@roeck-us.net, nathan@kernel.org, ndesaulniers@google.com, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev Subject: Re: [PATCH] watchdog: s3c2410: remove unused freq_to_wdt and s3c2410wdt_is_running functions Message-ID: <20230321192949.rlegmetszb7rnury@intel.intel> References: <20230321183439.1826823-1-trix@redhat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230321183439.1826823-1-trix@redhat.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230321_122959_442639_2CB9BE2A X-CRM114-Status: GOOD ( 17.89 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Tom, On Tue, Mar 21, 2023 at 02:34:39PM -0400, Tom Rix wrote: > clang with W=1 reports > drivers/watchdog/s3c2410_wdt.c:311:35: error: unused function > 'freq_to_wdt' [-Werror,-Wunused-function] > static inline struct s3c2410_wdt *freq_to_wdt(struct notifier_block *nb) > ^ > drivers/watchdog/s3c2410_wdt.c:446:19: error: unused function > 's3c2410wdt_is_running' [-Werror,-Wunused-function] > static inline int s3c2410wdt_is_running(struct s3c2410_wdt *wdt) > ^ > These functions are not used, so remove them. > > Signed-off-by: Tom Rix > --- > drivers/watchdog/s3c2410_wdt.c | 10 ---------- > 1 file changed, 10 deletions(-) > > diff --git a/drivers/watchdog/s3c2410_wdt.c b/drivers/watchdog/s3c2410_wdt.c > index 200ba236a72e..6394dda7a5dc 100644 > --- a/drivers/watchdog/s3c2410_wdt.c > +++ b/drivers/watchdog/s3c2410_wdt.c > @@ -308,11 +308,6 @@ static inline unsigned int s3c2410wdt_max_timeout(struct s3c2410_wdt *wdt) > / S3C2410_WTCON_MAXDIV); > } > > -static inline struct s3c2410_wdt *freq_to_wdt(struct notifier_block *nb) > -{ > - return container_of(nb, struct s3c2410_wdt, freq_transition); > -} > - > static int s3c2410wdt_disable_wdt_reset(struct s3c2410_wdt *wdt, bool mask) > { > const u32 mask_val = BIT(wdt->drv_data->mask_bit); > @@ -443,11 +438,6 @@ static int s3c2410wdt_start(struct watchdog_device *wdd) > return 0; > } > > -static inline int s3c2410wdt_is_running(struct s3c2410_wdt *wdt) > -{ > - return readl(wdt->reg_base + S3C2410_WTCON) & S3C2410_WTCON_ENABLE; > -} > - there is only one instance where this can be actually used in the driver, but it's inside a dev_info, so no point having it around. Reviewed-by: Andi Shyti Andi > static int s3c2410wdt_set_heartbeat(struct watchdog_device *wdd, > unsigned int timeout) > { > -- > 2.27.0 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel