[PATCH kernel] powerpc/powernv/ioda1: Remove dead code for a single device PE

[Alexey Kardashevskiy]

Ping?


On 16/10/2018 13:30, Alexey Kardashevskiy wrote:
> At the moment PNV_IODA_PE_DEV is only used for NPU PEs which are not
> present on IODA1 machines (i.e. POWER7) so let's remove a piece of
> dead code.
> 
> Signed-off-by: Alexey Kardashevskiy <aik at ozlabs.ru>
> ---
> 
> We might actually want to get rid of the entire IODA1 there.
> ---
>  arch/powerpc/platforms/powernv/pci-ioda.c | 10 +---------
>  1 file changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c
> index cde7102..78b61f0 100644
> --- a/arch/powerpc/platforms/powernv/pci-ioda.c
> +++ b/arch/powerpc/platforms/powernv/pci-ioda.c
> @@ -2367,15 +2367,7 @@ static void pnv_pci_ioda1_setup_dma_pe(struct pnv_phb *phb,
>  	pe->table_group.tce32_size = tbl->it_size << tbl->it_page_shift;
>  	iommu_init_table(tbl, phb->hose->node);
>  
> -	if (pe->flags & PNV_IODA_PE_DEV) {
> -		/*
> -		 * Setting table base here only for carrying iommu_group
> -		 * further down to let iommu_add_device() do the job.
> -		 * pnv_pci_ioda_dma_dev_setup will override it later anyway.
> -		 */
> -		set_iommu_table_base(&pe->pdev->dev, tbl);
> -		iommu_add_device(&pe->pdev->dev);
> -	} else if (pe->flags & (PNV_IODA_PE_BUS | PNV_IODA_PE_BUS_ALL))
> +	if (pe->flags & (PNV_IODA_PE_BUS | PNV_IODA_PE_BUS_ALL))
>  		pnv_ioda_setup_bus_dma(pe, pe->pbus, true);
>  
>  	return;
> 

-- 
Alexey