On 8/13/19 3:25 AM, Christoph Hellwig wrote:
> Now that hpsim support is gone, CONFIG_PCI is forced on for ia64, and
> we can remove a few ifdefs for it.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
Okay to remove hpsim support
Acked-by: Tom Vaden <tom.vaden@xxxxxxx>
> ---
> arch/ia64/Makefile | 2 +-
> arch/ia64/hp/common/sba_iommu.c | 10 +---------
> arch/ia64/include/asm/dma.h | 6 +-----
> arch/ia64/kernel/sys_ia64.c | 18 ------------------
> arch/ia64/mm/init.c | 2 --
> 5 files changed, 3 insertions(+), 35 deletions(-)
>
> diff --git a/arch/ia64/Makefile b/arch/ia64/Makefile
> index 8b866fc1f9cb..c06802799659 100644
> --- a/arch/ia64/Makefile
> +++ b/arch/ia64/Makefile
> @@ -57,7 +57,7 @@ core-$(CONFIG_IA64_HP_ZX1) += arch/ia64/dig/
> core-$(CONFIG_IA64_HP_ZX1_SWIOTLB) += arch/ia64/dig/
> core-$(CONFIG_IA64_SGI_UV) += arch/ia64/uv/
>
> -drivers-$(CONFIG_PCI) += arch/ia64/pci/
> +drivers-y += arch/ia64/pci/
> drivers-$(CONFIG_IA64_HP_ZX1) += arch/ia64/hp/common/ arch/ia64/hp/zx1/
> drivers-$(CONFIG_IA64_HP_ZX1_SWIOTLB) += arch/ia64/hp/common/ arch/ia64/hp/zx1/
> drivers-$(CONFIG_IA64_GENERIC) += arch/ia64/hp/common/ arch/ia64/hp/zx1/ arch/ia64/uv/
> diff --git a/arch/ia64/hp/common/sba_iommu.c b/arch/ia64/hp/common/sba_iommu.c
> index 3d24cc43385b..18321ce8bfa0 100644
> --- a/arch/ia64/hp/common/sba_iommu.c
> +++ b/arch/ia64/hp/common/sba_iommu.c
> @@ -251,12 +251,8 @@ static SBA_INLINE void sba_free_range(struct ioc *, dma_addr_t, size_t);
> static u64 prefetch_spill_page;
> #endif
>
> -#ifdef CONFIG_PCI
> -# define GET_IOC(dev) ((dev_is_pci(dev)) \
> +#define GET_IOC(dev) ((dev_is_pci(dev)) \
> ? ((struct ioc *) PCI_CONTROLLER(to_pci_dev(dev))->iommu) : NULL)
> -#else
> -# define GET_IOC(dev) NULL
> -#endif
>
> /*
> ** DMA_CHUNK_SIZE is used by the SCSI mid-layer to break up
> @@ -1741,9 +1737,7 @@ ioc_sac_init(struct ioc *ioc)
> controller->iommu = ioc;
> sac->sysdata = controller;
> sac->dma_mask = 0xFFFFFFFFUL;
> -#ifdef CONFIG_PCI
> sac->dev.bus = &pci_bus_type;
> -#endif
> ioc->sac_only_dev = sac;
> }
>
> @@ -2121,13 +2115,11 @@ sba_init(void)
> }
> #endif
>
> -#ifdef CONFIG_PCI
> {
> struct pci_bus *b = NULL;
> while ((b = pci_find_next_bus(b)) != NULL)
> sba_connect_bus(b);
> }
> -#endif
>
> #ifdef CONFIG_PROC_FS
> ioc_proc_init();
> diff --git a/arch/ia64/include/asm/dma.h b/arch/ia64/include/asm/dma.h
> index 23604d6a2cb2..59625e9c1f9c 100644
> --- a/arch/ia64/include/asm/dma.h
> +++ b/arch/ia64/include/asm/dma.h
> @@ -12,11 +12,7 @@
>
> extern unsigned long MAX_DMA_ADDRESS;
>
> -#ifdef CONFIG_PCI
> - extern int isa_dma_bridge_buggy;
> -#else
> -# define isa_dma_bridge_buggy (0)
> -#endif
> +extern int isa_dma_bridge_buggy;
>
> #define free_dma(x)
>
> diff --git a/arch/ia64/kernel/sys_ia64.c b/arch/ia64/kernel/sys_ia64.c
> index 9ebe1d633abc..e14db25146c2 100644
> --- a/arch/ia64/kernel/sys_ia64.c
> +++ b/arch/ia64/kernel/sys_ia64.c
> @@ -166,21 +166,3 @@ ia64_mremap (unsigned long addr, unsigned long old_len, unsigned long new_len, u
> force_successful_syscall_return();
> return addr;
> }
> -
> -#ifndef CONFIG_PCI
> -
> -asmlinkage long
> -sys_pciconfig_read (unsigned long bus, unsigned long dfn, unsigned long off, unsigned long len,
> - void *buf)
> -{
> - return -ENOSYS;
> -}
> -
> -asmlinkage long
> -sys_pciconfig_write (unsigned long bus, unsigned long dfn, unsigned long off, unsigned long len,
> - void *buf)
> -{
> - return -ENOSYS;
> -}
> -
> -#endif /* CONFIG_PCI */
> diff --git a/arch/ia64/mm/init.c b/arch/ia64/mm/init.c
> index aae75fd7b810..9a4a16439900 100644
> --- a/arch/ia64/mm/init.c
> +++ b/arch/ia64/mm/init.c
> @@ -632,14 +632,12 @@ mem_init (void)
> BUG_ON(PTRS_PER_PMD * sizeof(pmd_t) != PAGE_SIZE);
> BUG_ON(PTRS_PER_PTE * sizeof(pte_t) != PAGE_SIZE);
>
> -#ifdef CONFIG_PCI
> /*
> * This needs to be called _after_ the command line has been parsed but _before_
> * any drivers that may need the PCI DMA interface are initialized or bootmem has
> * been freed.
> */
> platform_dma_init();
> -#endif
>
> #ifdef CONFIG_FLATMEM
> BUG_ON(!mem_map);
>
