[00/13] Remove unused microblaze PCIe bus architecture

Message ID	20221025065214.4663-1-thippeswamy.havalige@amd.com (mailing list archive)
Headers	show Return-Path: <linux-pci-owner@kernel.org> Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB03.amd.com; pr=C From: Thippeswamy Havalige <thippeswamy.havalige@amd.com> To: <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>, <bharat.kumar.gogada@amd.com>, "Thippeswamy Havalige" <thippeswamy.havalige@amd.com> Subject: [PATCH 00/13] Remove unused microblaze PCIe bus architecture Date: Tue, 25 Oct 2022 12:22:01 +0530 Message-ID: <20221025065214.4663-1-thippeswamy.havalige@amd.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk
Series	Remove unused microblaze PCIe bus architecture \| expand [00/13] Remove unused microblaze PCIe bus architecture [01/13] microblaze/PCI: Remove unused early_read_config_byte() et al declarations [02/13] microblaze/PCI: Remove Null PCI config access unused functions [03/13] microblaze/PCI: Remove unused PCI bus scan if configured as a host [04/13] microblaze/PCI: Remove unused PCI legacy IO's access on a bus [05/13] microblaze/PCI: Remove unused device tree parsing for a host bridge resources [06/13] microblaze/PCI: Remove unused allocation & free of PCI host bridge structure [07/13] microblaze/PCI: Remove unused PCI BIOS resource allocation [08/13] microblaze/PCI: Remove unused PCI Indirect ops [09/13] microblaze/PCI: Remove unused pci_address_to_pio() conversion of CPU address to I/O port [10/13] microblaze/PCI: Remove unused sys_pciconfig_iobase() and et al declaration [11/13] microblaze/PCI: Remove unused pci_iobar_pfn() and et al declarations [12/13] microblaze/PCI: Remove support for Xilinx PCI host bridge [13/13] microblaze/PCI: Moving PCI iounmap and dependent code

Thippeswamy Havalige Oct. 25, 2022, 6:52 a.m. UTC

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

Michal Simek Oct. 25, 2022, 7:31 a.m. UTC | #1

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

Thippeswamy Havalige Oct. 25, 2022, 8:26 a.m. UTC | #2

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

Michal Simek Oct. 25, 2022, 9:22 a.m. UTC | #3

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

Thippeswamy Havalige Oct. 25, 2022, 2:07 p.m. UTC | #4

> -----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

Lorenzo Pieralisi Oct. 27, 2022, 9:46 a.m. UTC | #5

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

Thippeswamy Havalige Nov. 4, 2022, 8:56 a.m. UTC | #6

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

Michal Simek Nov. 25, 2022, 10:39 a.m. UTC | #7

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

[00/13] Remove unused microblaze PCIe bus architecture

Message

Comments