Message ID | 20221025065214.4663-1-thippeswamy.havalige@amd.com (mailing list archive) |
---|---|
Headers | show |
Series | Remove unused microblaze PCIe bus architecture | expand |
Hi, On 10/25/22 08:52, Thippeswamy Havalige wrote: > The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe > subsystem framework. This driver works on both Microblaze and Zynq > architecture based platforms. > > The microblaze architecture specific code has unused PCIe host bridge > supported API's which are no longer needed. > > This series of patch removes unused architecture specific > microblaze PCIe code. > > Thippeswamy Havalige (13): > microblaze/PCI: Remove unused early_read_config_byte() et al > declarations > microblaze/PCI: Remove Null PCI config access unused functions > microblaze/PCI: Remove unused PCI bus scan if configured as a host > microblaze/PCI: Remove unused PCI legacy IO's access on a bus > microblaze/PCI: Remove unused device tree parsing for a host bridge > resources > microblaze/PCI: Remove unused allocation & free of PCI host bridge > structure > microblaze/PCI: Remove unused PCI BIOS resource allocation > microblaze/PCI: Remove unused PCI Indirect ops > microblaze/PCI: Remove unused pci_address_to_pio() conversion of CPU > address to I/O port > microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al > declaration > microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations > microblaze/PCI: Remove support for Xilinx PCI host bridge > microblaze/PCI: Moving PCI iounmap and dependent code > > arch/microblaze/Kconfig | 8 - > arch/microblaze/include/asm/pci-bridge.h | 92 --- > arch/microblaze/include/asm/pci.h | 29 - > arch/microblaze/pci/Makefile | 3 +- > arch/microblaze/pci/indirect_pci.c | 158 ----- > arch/microblaze/pci/iomap.c | 36 + > arch/microblaze/pci/pci-common.c | 1067 ------------------------------ > arch/microblaze/pci/xilinx_pci.c | 170 ----- > 8 files changed, 37 insertions(+), 1526 deletions(-) > delete mode 100644 arch/microblaze/pci/indirect_pci.c > delete mode 100644 arch/microblaze/pci/pci-common.c > delete mode 100644 arch/microblaze/pci/xilinx_pci.c > Why are you sending it again? M
Hi, > -----Original Message----- > From: Simek, Michal <michal.simek@amd.com> > Sent: Tuesday, October 25, 2022 1:02 PM > To: Havalige, Thippeswamy <thippeswamy.havalige@amd.com>; linux- > pci@vger.kernel.org; linux-kernel@vger.kernel.org; > devicetree@vger.kernel.org; krzysztof.kozlowski@linaro.org > Cc: bhelgaas@google.com; michals@xilinx.com; robh+dt@kernel.org; > lorenzo.pieralisi@arm.com; Gogada, Bharat Kumar > <bharat.kumar.gogada@amd.com> > Subject: Re: [PATCH 00/13] Remove unused microblaze PCIe bus architecture > > Hi, > > On 10/25/22 08:52, Thippeswamy Havalige wrote: > > The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe > > subsystem framework. This driver works on both Microblaze and Zynq > > architecture based platforms. > > > > The microblaze architecture specific code has unused PCIe host bridge > > supported API's which are no longer needed. > > > > This series of patch removes unused architecture specific microblaze > > PCIe code. > > > > Thippeswamy Havalige (13): > > microblaze/PCI: Remove unused early_read_config_byte() et al > > declarations > > microblaze/PCI: Remove Null PCI config access unused functions > > microblaze/PCI: Remove unused PCI bus scan if configured as a host > > microblaze/PCI: Remove unused PCI legacy IO's access on a bus > > microblaze/PCI: Remove unused device tree parsing for a host bridge > > resources > > microblaze/PCI: Remove unused allocation & free of PCI host bridge > > structure > > microblaze/PCI: Remove unused PCI BIOS resource allocation > > microblaze/PCI: Remove unused PCI Indirect ops > > microblaze/PCI: Remove unused pci_address_to_pio() conversion of CPU > > address to I/O port > > microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al > > declaration > > microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations > > microblaze/PCI: Remove support for Xilinx PCI host bridge > > microblaze/PCI: Moving PCI iounmap and dependent code > > > > arch/microblaze/Kconfig | 8 - > > arch/microblaze/include/asm/pci-bridge.h | 92 --- > > arch/microblaze/include/asm/pci.h | 29 - > > arch/microblaze/pci/Makefile | 3 +- > > arch/microblaze/pci/indirect_pci.c | 158 ----- > > arch/microblaze/pci/iomap.c | 36 + > > arch/microblaze/pci/pci-common.c | 1067 ------------------------------ > > arch/microblaze/pci/xilinx_pci.c | 170 ----- > > 8 files changed, 37 insertions(+), 1526 deletions(-) > > delete mode 100644 arch/microblaze/pci/indirect_pci.c > > delete mode 100644 arch/microblaze/pci/pci-common.c > > delete mode 100644 arch/microblaze/pci/xilinx_pci.c > > > > Why are you sending it again? > > M Last time mails were not delivered to opensource maintainers due to some access permissions. Regards, Thippeswamy H
On 10/25/22 10:26, Havalige, Thippeswamy wrote: > Hi, >> -----Original Message----- >> From: Simek, Michal <michal.simek@amd.com> >> Sent: Tuesday, October 25, 2022 1:02 PM >> To: Havalige, Thippeswamy <thippeswamy.havalige@amd.com>; linux- >> pci@vger.kernel.org; linux-kernel@vger.kernel.org; >> devicetree@vger.kernel.org; krzysztof.kozlowski@linaro.org >> Cc: bhelgaas@google.com; michals@xilinx.com; robh+dt@kernel.org; >> lorenzo.pieralisi@arm.com; Gogada, Bharat Kumar >> <bharat.kumar.gogada@amd.com> >> Subject: Re: [PATCH 00/13] Remove unused microblaze PCIe bus architecture >> >> Hi, >> >> On 10/25/22 08:52, Thippeswamy Havalige wrote: >>> The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe >>> subsystem framework. This driver works on both Microblaze and Zynq >>> architecture based platforms. >>> >>> The microblaze architecture specific code has unused PCIe host bridge >>> supported API's which are no longer needed. >>> >>> This series of patch removes unused architecture specific microblaze >>> PCIe code. >>> >>> Thippeswamy Havalige (13): >>> microblaze/PCI: Remove unused early_read_config_byte() et al >>> declarations >>> microblaze/PCI: Remove Null PCI config access unused functions >>> microblaze/PCI: Remove unused PCI bus scan if configured as a host >>> microblaze/PCI: Remove unused PCI legacy IO's access on a bus >>> microblaze/PCI: Remove unused device tree parsing for a host bridge >>> resources >>> microblaze/PCI: Remove unused allocation & free of PCI host bridge >>> structure >>> microblaze/PCI: Remove unused PCI BIOS resource allocation >>> microblaze/PCI: Remove unused PCI Indirect ops >>> microblaze/PCI: Remove unused pci_address_to_pio() conversion of CPU >>> address to I/O port >>> microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al >>> declaration >>> microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations >>> microblaze/PCI: Remove support for Xilinx PCI host bridge >>> microblaze/PCI: Moving PCI iounmap and dependent code >>> >>> arch/microblaze/Kconfig | 8 - >>> arch/microblaze/include/asm/pci-bridge.h | 92 --- >>> arch/microblaze/include/asm/pci.h | 29 - >>> arch/microblaze/pci/Makefile | 3 +- >>> arch/microblaze/pci/indirect_pci.c | 158 ----- >>> arch/microblaze/pci/iomap.c | 36 + >>> arch/microblaze/pci/pci-common.c | 1067 ------------------------------ >>> arch/microblaze/pci/xilinx_pci.c | 170 ----- >>> 8 files changed, 37 insertions(+), 1526 deletions(-) >>> delete mode 100644 arch/microblaze/pci/indirect_pci.c >>> delete mode 100644 arch/microblaze/pci/pci-common.c >>> delete mode 100644 arch/microblaze/pci/xilinx_pci.c >>> >> >> Why are you sending it again? >> >> M > > > Last time mails were not delivered to opensource maintainers due to some access permissions. But people in TO/CC got it. It means you should send it as RESEND or v2 to avoid confusion. Thanks, Michal
> -----Original Message----- > From: Simek, Michal <michal.simek@amd.com> > Sent: Tuesday, October 25, 2022 2:53 PM > To: Havalige, Thippeswamy <thippeswamy.havalige@amd.com>; linux- > pci@vger.kernel.org; linux-kernel@vger.kernel.org; > devicetree@vger.kernel.org; krzysztof.kozlowski@linaro.org > Cc: bhelgaas@google.com; michals@xilinx.com; robh+dt@kernel.org; > lorenzo.pieralisi@arm.com; Gogada, Bharat Kumar > <bharat.kumar.gogada@amd.com> > Subject: Re: [PATCH 00/13] Remove unused microblaze PCIe bus architecture > > > > On 10/25/22 10:26, Havalige, Thippeswamy wrote: > > Hi, > >> -----Original Message----- > >> From: Simek, Michal <michal.simek@amd.com> > >> Sent: Tuesday, October 25, 2022 1:02 PM > >> To: Havalige, Thippeswamy <thippeswamy.havalige@amd.com>; linux- > >> pci@vger.kernel.org; linux-kernel@vger.kernel.org; > >> devicetree@vger.kernel.org; krzysztof.kozlowski@linaro.org > >> Cc: bhelgaas@google.com; michals@xilinx.com; robh+dt@kernel.org; > >> lorenzo.pieralisi@arm.com; Gogada, Bharat Kumar > >> <bharat.kumar.gogada@amd.com> > >> Subject: Re: [PATCH 00/13] Remove unused microblaze PCIe bus > >> architecture > >> > >> Hi, > >> > >> On 10/25/22 08:52, Thippeswamy Havalige wrote: > >>> The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe > >>> subsystem framework. This driver works on both Microblaze and Zynq > >>> architecture based platforms. > >>> > >>> The microblaze architecture specific code has unused PCIe host > >>> bridge supported API's which are no longer needed. > >>> > >>> This series of patch removes unused architecture specific microblaze > >>> PCIe code. > >>> > >>> Thippeswamy Havalige (13): > >>> microblaze/PCI: Remove unused early_read_config_byte() et al > >>> declarations > >>> microblaze/PCI: Remove Null PCI config access unused functions > >>> microblaze/PCI: Remove unused PCI bus scan if configured as a host > >>> microblaze/PCI: Remove unused PCI legacy IO's access on a bus > >>> microblaze/PCI: Remove unused device tree parsing for a host bridge > >>> resources > >>> microblaze/PCI: Remove unused allocation & free of PCI host bridge > >>> structure > >>> microblaze/PCI: Remove unused PCI BIOS resource allocation > >>> microblaze/PCI: Remove unused PCI Indirect ops > >>> microblaze/PCI: Remove unused pci_address_to_pio() conversion of > CPU > >>> address to I/O port > >>> microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al > >>> declaration > >>> microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations > >>> microblaze/PCI: Remove support for Xilinx PCI host bridge > >>> microblaze/PCI: Moving PCI iounmap and dependent code > >>> > >>> arch/microblaze/Kconfig | 8 - > >>> arch/microblaze/include/asm/pci-bridge.h | 92 --- > >>> arch/microblaze/include/asm/pci.h | 29 - > >>> arch/microblaze/pci/Makefile | 3 +- > >>> arch/microblaze/pci/indirect_pci.c | 158 ----- > >>> arch/microblaze/pci/iomap.c | 36 + > >>> arch/microblaze/pci/pci-common.c | 1067 ------------------------------ > >>> arch/microblaze/pci/xilinx_pci.c | 170 ----- > >>> 8 files changed, 37 insertions(+), 1526 deletions(-) > >>> delete mode 100644 arch/microblaze/pci/indirect_pci.c > >>> delete mode 100644 arch/microblaze/pci/pci-common.c > >>> delete mode 100644 arch/microblaze/pci/xilinx_pci.c > >>> > >> > >> Why are you sending it again? > >> > >> M > > > > > > Last time mails were not delivered to opensource maintainers due to some > access permissions. > > But people in TO/CC got it. It means you should send it as RESEND or v2 to > avoid confusion. > > Thanks, > Michal Hi Michal, Check this link, I could only see the patches were listed that are sent today, initially I had some issue or permission issues to send mails outside AMD. https://patchwork.ozlabs.org/project/linux-pci/list/ Regards, Thippeswamy H
On Tue, Oct 25, 2022 at 09:31:37AM +0200, Michal Simek wrote: > Hi, > > On 10/25/22 08:52, Thippeswamy Havalige wrote: > > The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe > > subsystem framework. This driver works on both Microblaze and Zynq > > architecture based platforms. > > > > The microblaze architecture specific code has unused PCIe host bridge > > supported API's which are no longer needed. > > > > This series of patch removes unused architecture specific > > microblaze PCIe code. > > > > Thippeswamy Havalige (13): > > microblaze/PCI: Remove unused early_read_config_byte() et al > > declarations > > microblaze/PCI: Remove Null PCI config access unused functions > > microblaze/PCI: Remove unused PCI bus scan if configured as a host > > microblaze/PCI: Remove unused PCI legacy IO's access on a bus > > microblaze/PCI: Remove unused device tree parsing for a host bridge > > resources > > microblaze/PCI: Remove unused allocation & free of PCI host bridge > > structure > > microblaze/PCI: Remove unused PCI BIOS resource allocation > > microblaze/PCI: Remove unused PCI Indirect ops > > microblaze/PCI: Remove unused pci_address_to_pio() conversion of CPU > > address to I/O port > > microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al > > declaration > > microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations > > microblaze/PCI: Remove support for Xilinx PCI host bridge > > microblaze/PCI: Moving PCI iounmap and dependent code > > > > arch/microblaze/Kconfig | 8 - > > arch/microblaze/include/asm/pci-bridge.h | 92 --- > > arch/microblaze/include/asm/pci.h | 29 - > > arch/microblaze/pci/Makefile | 3 +- > > arch/microblaze/pci/indirect_pci.c | 158 ----- > > arch/microblaze/pci/iomap.c | 36 + > > arch/microblaze/pci/pci-common.c | 1067 ------------------------------ > > arch/microblaze/pci/xilinx_pci.c | 170 ----- > > 8 files changed, 37 insertions(+), 1526 deletions(-) > > delete mode 100644 arch/microblaze/pci/indirect_pci.c > > delete mode 100644 arch/microblaze/pci/pci-common.c > > delete mode 100644 arch/microblaze/pci/xilinx_pci.c > > > > Why are you sending it again? Michal, it looks like you don't need anything from me or Bjorn on this series so I shall drop it from the PCI queue and let you handle it. If you need any help please let me know. Thanks, Lorenzo
Hi Michal, Can you review/check on this series of patches. Regards, Thippeswamy H > -----Original Message----- > From: Lorenzo Pieralisi <lpieralisi@kernel.org> > Sent: Thursday, October 27, 2022 3:17 PM > To: Simek, Michal <michal.simek@amd.com> > Cc: Havalige, Thippeswamy <thippeswamy.havalige@amd.com>; linux- > pci@vger.kernel.org; linux-kernel@vger.kernel.org; > devicetree@vger.kernel.org; krzysztof.kozlowski@linaro.org; > bhelgaas@google.com; michals@xilinx.com; robh+dt@kernel.org; > lorenzo.pieralisi@arm.com; Gogada, Bharat Kumar > <bharat.kumar.gogada@amd.com> > Subject: Re: [PATCH 00/13] Remove unused microblaze PCIe bus architecture > > On Tue, Oct 25, 2022 at 09:31:37AM +0200, Michal Simek wrote: > > Hi, > > > > On 10/25/22 08:52, Thippeswamy Havalige wrote: > > > The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe > > > subsystem framework. This driver works on both Microblaze and Zynq > > > architecture based platforms. > > > > > > The microblaze architecture specific code has unused PCIe host > > > bridge supported API's which are no longer needed. > > > > > > This series of patch removes unused architecture specific microblaze > > > PCIe code. > > > > > > Thippeswamy Havalige (13): > > > microblaze/PCI: Remove unused early_read_config_byte() et al > > > declarations > > > microblaze/PCI: Remove Null PCI config access unused functions > > > microblaze/PCI: Remove unused PCI bus scan if configured as a host > > > microblaze/PCI: Remove unused PCI legacy IO's access on a bus > > > microblaze/PCI: Remove unused device tree parsing for a host bridge > > > resources > > > microblaze/PCI: Remove unused allocation & free of PCI host bridge > > > structure > > > microblaze/PCI: Remove unused PCI BIOS resource allocation > > > microblaze/PCI: Remove unused PCI Indirect ops > > > microblaze/PCI: Remove unused pci_address_to_pio() conversion of > CPU > > > address to I/O port > > > microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al > > > declaration > > > microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations > > > microblaze/PCI: Remove support for Xilinx PCI host bridge > > > microblaze/PCI: Moving PCI iounmap and dependent code > > > > > > arch/microblaze/Kconfig | 8 - > > > arch/microblaze/include/asm/pci-bridge.h | 92 --- > > > arch/microblaze/include/asm/pci.h | 29 - > > > arch/microblaze/pci/Makefile | 3 +- > > > arch/microblaze/pci/indirect_pci.c | 158 ----- > > > arch/microblaze/pci/iomap.c | 36 + > > > arch/microblaze/pci/pci-common.c | 1067 ------------------------------ > > > arch/microblaze/pci/xilinx_pci.c | 170 ----- > > > 8 files changed, 37 insertions(+), 1526 deletions(-) > > > delete mode 100644 arch/microblaze/pci/indirect_pci.c > > > delete mode 100644 arch/microblaze/pci/pci-common.c > > > delete mode 100644 arch/microblaze/pci/xilinx_pci.c > > > > > > > Why are you sending it again? > > Michal, > > it looks like you don't need anything from me or Bjorn on this series so I shall > drop it from the PCI queue and let you handle it. > > If you need any help please let me know. > > Thanks, > Lorenzo
On 10/25/22 08:52, Thippeswamy Havalige wrote: > The current Xilinx AXI PCIe Host Bridge driver uses generic PCIe > subsystem framework. This driver works on both Microblaze and Zynq > architecture based platforms. > > The microblaze architecture specific code has unused PCIe host bridge > supported API's which are no longer needed. > > This series of patch removes unused architecture specific > microblaze PCIe code. > > Thippeswamy Havalige (13): > microblaze/PCI: Remove unused early_read_config_byte() et al > declarations > microblaze/PCI: Remove Null PCI config access unused functions > microblaze/PCI: Remove unused PCI bus scan if configured as a host > microblaze/PCI: Remove unused PCI legacy IO's access on a bus > microblaze/PCI: Remove unused device tree parsing for a host bridge > resources > microblaze/PCI: Remove unused allocation & free of PCI host bridge > structure > microblaze/PCI: Remove unused PCI BIOS resource allocation > microblaze/PCI: Remove unused PCI Indirect ops > microblaze/PCI: Remove unused pci_address_to_pio() conversion of CPU > address to I/O port > microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al > declaration > microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations > microblaze/PCI: Remove support for Xilinx PCI host bridge > microblaze/PCI: Moving PCI iounmap and dependent code > > arch/microblaze/Kconfig | 8 - > arch/microblaze/include/asm/pci-bridge.h | 92 --- > arch/microblaze/include/asm/pci.h | 29 - > arch/microblaze/pci/Makefile | 3 +- > arch/microblaze/pci/indirect_pci.c | 158 ----- > arch/microblaze/pci/iomap.c | 36 + > arch/microblaze/pci/pci-common.c | 1067 ------------------------------ > arch/microblaze/pci/xilinx_pci.c | 170 ----- > 8 files changed, 37 insertions(+), 1526 deletions(-) > delete mode 100644 arch/microblaze/pci/indirect_pci.c > delete mode 100644 arch/microblaze/pci/pci-common.c > delete mode 100644 arch/microblaze/pci/xilinx_pci.c > Applied with also removing PCI_XILINX from defconfig. Thanks, Michal