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=-10.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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 CA346C433DF for ; Thu, 16 Jul 2020 22:09:09 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 93069207CB for ; Thu, 16 Jul 2020 22:09:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="FvZU4p5S"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="J8b6uiEa" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 93069207CB Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org 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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Message-ID:Subject:To:From: Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References:List-Owner; bh=IfIbltXGinLTBeMeqWhJIs2X94qDAlqz4w/Plq7SaJo=; b=FvZU4p5SKfYX8lqulz0s4bMNJ BB7jey59FaxIi0nik0r0aq0XkxSbL0RQ7Czh9DzwNq8sPydZ4Ay24Nh9NORfnvl33cv9AERJb9cR9 o7GG5zUIDVo/TzTxdGdGdZGa6XErnYdGBcygChXJjjxUxvxKZxLmqjAd6SywJ58jyGyeiEWXCAP2V /45Akbiy8iuowRVnhkv7OxroA6cZzsag7QrX2cSSqwbTk4vZXwg3gQxQu5G7k+6H5J6DfHT/rjx0r srs0X50U8hJ2DrDPqAMxWtYSYlkxhbSzlKxLx7UT9aPqAICS+62hZKfp2UVa9qr3SGmng6RxdZRLy TDzdj29ug==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jwC1T-0001vt-US; Thu, 16 Jul 2020 22:06:36 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jwC1R-0001vF-0y for linux-arm-kernel@lists.infradead.org; Thu, 16 Jul 2020 22:06:33 +0000 Received: from localhost (mobile-166-175-191-139.mycingular.net [166.175.191.139]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id A33CF207CB; Thu, 16 Jul 2020 22:06:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1594937192; bh=5HCKWjFC8y6BxQfhehJHW/n1R4IoPDaMe/czVqxsK1o=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=J8b6uiEa5Bnkr48S5nTo6/WA4aeskRqrZoig7lMkTV8a6qHT3hb1DuO6f+zv87XsT 5a00U0ZVi5SoGx5fa/mKZWlJRr4kXz/fMTj0r0fDyBbMWwRTS8N5yahHq2B1MeRswZ MHYnHSUrWAuSU3LaMzYIouJFjSud8+nComAEHi1A= Date: Thu, 16 Jul 2020 17:06:29 -0500 From: Bjorn Helgaas To: "Gustavo A. R. Silva" Subject: Re: [PATCH][next] PCI: imx6: Use fallthrough pseudo-keyword Message-ID: <20200716220629.GA663054@bjorn-Precision-5520> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200716211052.GA16893@embeddedor> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200716_180633_201067_09C5AB94 X-CRM114-Status: GOOD ( 18.69 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rob Herring , Lorenzo Pieralisi , Richard Zhu , Fabio Estevam , Sascha Hauer , linux-kernel@vger.kernel.org, NXP Linux Team , Pengutronix Kernel Team , linux-pci@vger.kernel.org, Bjorn Helgaas , Shawn Guo , linux-arm-kernel@lists.infradead.org, Lucas Stach 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 Thu, Jul 16, 2020 at 04:10:52PM -0500, Gustavo A. R. Silva wrote: > Replace the existing /* fall through */ comments and its variants with > the new pseudo-keyword macro fallthrough[1]. Also, remove unnecessary > fall-through markings when it is the case. > > [1] https://www.kernel.org/doc/html/latest/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Hi Gustavo, I'm certainly fine with these patches, and thanks for doing them! And thanks for providing a link to the rationale. But the URL contains "latest", so I think it may break if deprecated.rst or the section is ever renamed. I think I would prefer if we could reference the current text, e.g., via https://www.kernel.org/doc/html/v5.7-rc7/process/deprecated.html#implicit-switch-case-fall-through (The v5.7 doc would be better but doesn't seem to be generated yet; I pinged the helpdesk about that.) Or we could refer to b9918bdcac1f ("Documentation/process: Add fallthrough pseudo-keyword"), although it's not nearly as pretty as the HTML. > Signed-off-by: Gustavo A. R. Silva > --- > drivers/pci/controller/dwc/pci-imx6.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c > index 4e5c379ae418..1119ded593d0 100644 > --- a/drivers/pci/controller/dwc/pci-imx6.c > +++ b/drivers/pci/controller/dwc/pci-imx6.c > @@ -439,7 +439,7 @@ static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > break; > - case IMX6QP: /* FALLTHROUGH */ > + case IMX6QP: > case IMX6Q: > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > @@ -642,7 +642,7 @@ static void imx6_pcie_init_phy(struct imx6_pcie *imx6_pcie) > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6SX_GPR12_PCIE_RX_EQ_MASK, > IMX6SX_GPR12_PCIE_RX_EQ_2); > - /* FALLTHROUGH */ > + fallthrough; > default: > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > @@ -1107,7 +1107,7 @@ static int imx6_pcie_probe(struct platform_device *pdev) > dev_err(dev, "pcie_aux clock source missing or invalid\n"); > return PTR_ERR(imx6_pcie->pcie_aux); > } > - /* fall through */ > + fallthrough; > case IMX7D: > if (dbi_base->start == IMX8MQ_PCIE2_BASE_ADDR) > imx6_pcie->controller_id = 1; > -- > 2.27.0 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel