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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (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 80FF6C43217 for ; Thu, 10 Nov 2022 21:03:43 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4N7Z7d5vy3z3f3w for ; Fri, 11 Nov 2022 08:03:41 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20201202 header.b=iJwMckJy; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=kernel.org (client-ip=139.178.84.217; helo=dfw.source.kernel.org; envelope-from=helgaas@kernel.org; receiver=) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20201202 header.b=iJwMckJy; dkim-atps=neutral Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4N7Z6S6LSNz2xG6 for ; Fri, 11 Nov 2022 08:02:40 +1100 (AEDT) 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 dfw.source.kernel.org (Postfix) with ESMTPS id 14F1361E56; Thu, 10 Nov 2022 21:02:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2AFFAC433C1; Thu, 10 Nov 2022 21:02:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1668114157; bh=dFjwsB2x4Vktj11UP1nFbKy1JuZUspP3pjwcHIsSSnc=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=iJwMckJy0uCcByLR5dTqb/sLhh/p4neWekIYZaZ3yeonbwH09ch8EqoG8ecGOC/gh LxSdAJBLbKmEE3CXGDI3A7Vc6BCxXLJ/bsFq5hQJgqS/nai15IhiKGetEeKH4+vpT2 j9Ld+7uAe0Uz9IipZQ/88pm8pLqURo9XzvxQWuunebeReJqpcbJfuCN8BWniAwIWF9 a+ZwQoYjb1mS5vRmT3xSXjeYdGWxfBo/pZQHhYRV38fM2nHaPFfGhzmt8f50haLCEQ 7sSMDQr3yt4Z74/HTNY4ceRxsq+32kGBXwaNszYjSX1Y2pvyiMxqjjRM7Gykh+Y6zM XSZjD4/Y6CHGA== Date: Thu, 10 Nov 2022 15:02:35 -0600 From: Bjorn Helgaas To: Lorenzo Pieralisi Subject: Re: [PATCH v3 0/5] PCI: Remove unnecessary includes Message-ID: <20221110210235.GA671572@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221031153954.1163623-1-helgaas@kernel.org> X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Heiko Stuebner , linux-pci@vger.kernel.org, Linus Walleij , Minghuan Lian , Conor Dooley , Thierry Reding , Thomas Petazzoni , Bharat Kumar Gogada , Toan Le , linux-riscv@lists.infradead.org, Kishon Vijay Abraham I , Rob Herring , Joyce Ooi , Jonathan Hunter , linux-rockchip@lists.infradead.org, bcm-kernel-feedback-list@broadcom.com, linux-arm-kernel@lists.infradead.org, Shawn Lin , Ray Jui , linux-tegra@vger.kernel.org, linux-omap@vger.kernel.org, Michal Simek , Mingkai Hu , Roy Zang , Bjorn Helgaas , Scott Branden , Daire McNamara , linux-kernel@vger.kernel.org, Tom Joseph , linuxppc-dev@lists.ozlabs.org Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Mon, Oct 31, 2022 at 10:39:49AM -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > Many host controller drivers #include even though they > don't need it. Remove the unnecessary #includes. > > v2: https://lore.kernel.org/r/20221025185147.665365-1-helgaas@kernel.org/ > v1: https://lore.kernel.org/r/20221019195452.37606-1-helgaas@kernel.org/ > > Changes from v2 to v3: > - Include explicitly in xgene-msi, which doesn't need > itself, but relied on it to include > . On x86, this was covered up by the fact that > includes , which includes , > which includes . But on parisc, is > actually asm-generic/msi.h, which does *not* include > > - Pick up tags from Conor Dooley and Thomas Petazzoni > > Changes from v1 to v2: > - Include explicitly in altera-msi and microchip, > which don't need itself, but relied on it to include > > - Include explicitly in mvebu, which needs both it > and > > Bjorn Helgaas (5): > PCI: altera-msi: Include explicitly > PCI: microchip: Include explicitly > PCI: mvebu: Include explicitly > PCI: xgene-msi: Include explicitly > PCI: Remove unnecessary includes > > drivers/pci/controller/cadence/pci-j721e.c | 1 - > drivers/pci/controller/dwc/pci-layerscape.c | 1 - > drivers/pci/controller/dwc/pcie-armada8k.c | 1 - > drivers/pci/controller/dwc/pcie-tegra194.c | 1 - > drivers/pci/controller/pci-mvebu.c | 1 + > drivers/pci/controller/pci-v3-semi.c | 1 - > drivers/pci/controller/pci-xgene-msi.c | 2 +- > drivers/pci/controller/pci-xgene.c | 1 - > drivers/pci/controller/pcie-altera-msi.c | 2 +- > drivers/pci/controller/pcie-iproc-platform.c | 1 - > drivers/pci/controller/pcie-iproc.c | 1 - > drivers/pci/controller/pcie-microchip-host.c | 2 +- > drivers/pci/controller/pcie-rockchip-host.c | 1 - > drivers/pci/controller/pcie-xilinx-cpm.c | 1 - > drivers/pci/controller/pcie-xilinx-nwl.c | 1 - > 15 files changed, 4 insertions(+), 14 deletions(-) I put these on a pci/kbuild branch that I propose to merge in after everything else. Lorenzo, let me know if you prefer another approach. Bjorn