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 X-Spam-Level: X-Spam-Status: No, score=-15.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1180EC48BE5 for ; Wed, 16 Jun 2021 10:22:21 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id D31B861185 for ; Wed, 16 Jun 2021 10:22:20 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D31B861185 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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=ePPBexmrnEuUKFsmDhld6tpjP2yBBoVjf9tL7daFhv0=; b=kZGEOhIVUSe56M rTX/gaKOXWQxX3MmlFIEmkZnYtUVnX1ofGDGbQ+kScm7wQhNfkPYUPiUTUgaC82AlqjyyHGy0l1Pk yxO2znH0nXTPwnS4+gs2U8m2ycRUif4PiMgqNNMPH7XC1bO3UILw8rnRX0eNDXODpKH9+hoV3ur4s 06qxuMgML1DG3ek1c206jl6s1EvtIH8KArUT77Qb8LWYo+hlxc1soW6lvxfyPDOt58azxatT8/WgV Bew1Cc08rdm9iyT4wFZNPR7YOXSiuzO97E1I71i3zKKxfqq+igPLg1vfMnqdJ8KuHeEecYf7fSYu0 iSstsnyX1JMiEPVpwYMA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ltSfI-005lI1-7p; Wed, 16 Jun 2021 10:20:56 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ltSfD-005lGE-IX for linux-arm-kernel@lists.infradead.org; Wed, 16 Jun 2021 10:20:53 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=3S/UaOiglV8sU2MJUjiJaJvpv8j/sHW/Cezjvx2y14Y=; b=wCqpiOo4dyL3V2cCWohqHRtIM iUybzBSVZvMe3eAPpHMfGBpVOs1LcAROZ8pt1aFybpM3jg1F1juGz8riiD4VUCCYWiDSC3HYz2ZFY UOjaNx43AB0zY/5LEbXBTA2nBijob7xipuUHmH9eOsMekCwNc5z3katj8Bwo3Ydi8frBP3RP3J3UC Y+6DDM8UxqA3K2mPIaekt3aEtsX8o1b9JcoE8aUOFjfb0u7gjNchBICJzxwq/tro3bsd91va6bgR2 HrWgghzrC8o6d9EYSP3STKmMJPnDraHNe8S59PqmCJyKHD8TztCkIbOuvNuIDgRgOnitVOLEIujqC VfTcm2m1A==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:45058) by pandora.armlinux.org.uk with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1ltSf3-00072V-Vr; Wed, 16 Jun 2021 11:20:42 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.92) (envelope-from ) id 1ltSf0-0005wF-Ns; Wed, 16 Jun 2021 11:20:38 +0100 Date: Wed, 16 Jun 2021 11:20:38 +0100 From: "Russell King (Oracle)" To: Joakim Zhang Cc: davem@davemloft.net, kuba@kernel.org, peppe.cavallaro@st.com, alexandre.torgue@foss.st.com, joabreu@synopsys.com, mcoquelin.stm32@gmail.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com Subject: Re: [PATCH net 1/2] net: fec_ptp: add clock rate zero check Message-ID: <20210616102038.GB22278@shell.armlinux.org.uk> References: <20210616091426.13694-1-qiangqing.zhang@nxp.com> <20210616091426.13694-2-qiangqing.zhang@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210616091426.13694-2-qiangqing.zhang@nxp.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210616_032051_681965_40386A4A X-CRM114-Status: GOOD ( 18.68 ) 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 On Wed, Jun 16, 2021 at 05:14:25PM +0800, Joakim Zhang wrote: > From: Fugang Duan > > Add clock rate zero check to fix coverity issue of "divide by 0". > > Fixes: commit 85bd1798b24a ("net: fec: fix spin_lock dead lock") > Signed-off-by: Fugang Duan > Signed-off-by: Joakim Zhang > --- > drivers/net/ethernet/freescale/fec_ptp.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/net/ethernet/freescale/fec_ptp.c b/drivers/net/ethernet/freescale/fec_ptp.c > index 1753807cbf97..7326a0612823 100644 > --- a/drivers/net/ethernet/freescale/fec_ptp.c > +++ b/drivers/net/ethernet/freescale/fec_ptp.c > @@ -604,6 +604,10 @@ void fec_ptp_init(struct platform_device *pdev, int irq_idx) > fep->ptp_caps.enable = fec_ptp_enable; > > fep->cycle_speed = clk_get_rate(fep->clk_ptp); > + if (!fep->cycle_speed) { > + fep->cycle_speed = NSEC_PER_SEC; > + dev_err(&fep->pdev->dev, "clk_ptp clock rate is zero\n"); If this is supposed to be an error message, it doesn't convey that something is really wrong to the user. Maybe something like this would be more meaningful to the user: "PTP clock rate should not be zero, using 1GHz instead. PTP clock may be unreliable.\n" It may be appropriate not to publish PTP support for the interface if we don't have a valid clock rate, which is probably the safer approach and would probably make the problem more noticable to the end user so it gets fixed. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel