mirror of https://gitee.com/openkylin/linux.git
powerpc/powernv: Missed IOMMU table type
In function pnv_pci_ioda2_setup_dma_pe(), the IOMMU table type is set to (TCE_PCI_SWINV_CREATE | TCE_PCI_SWINV_FREE) unconditionally. It was just set to TCE_PCI by pnv_pci_setup_iommu_table(). So the primary IOMMU table type (TCE_PCI) is lost. The patch fixes it. Also, pnv_pci_setup_iommu_table() already set "tbl->it_busno" to zero and we needn't do it again. The patch removes the redundant assignment. The patch also fixes similar issues in pnv_pci_ioda_setup_dma_pe(). Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
parent
b2b5efcf20
commit
65fd766b99
|
@ -664,12 +664,12 @@ static void pnv_pci_ioda_setup_dma_pe(struct pnv_phb *phb,
|
|||
* errors, and on the first pass the data will be a relative
|
||||
* bus number, print that out instead.
|
||||
*/
|
||||
tbl->it_busno = 0;
|
||||
pe->tce_inval_reg_phys = be64_to_cpup(swinvp);
|
||||
tbl->it_index = (unsigned long)ioremap(pe->tce_inval_reg_phys,
|
||||
8);
|
||||
tbl->it_type = TCE_PCI_SWINV_CREATE | TCE_PCI_SWINV_FREE |
|
||||
TCE_PCI_SWINV_PAIR;
|
||||
tbl->it_type |= (TCE_PCI_SWINV_CREATE |
|
||||
TCE_PCI_SWINV_FREE |
|
||||
TCE_PCI_SWINV_PAIR);
|
||||
}
|
||||
iommu_init_table(tbl, phb->hose->node);
|
||||
iommu_register_group(tbl, pci_domain_nr(pe->pbus), pe->pe_number);
|
||||
|
@ -794,11 +794,10 @@ static void pnv_pci_ioda2_setup_dma_pe(struct pnv_phb *phb,
|
|||
* errors, and on the first pass the data will be a relative
|
||||
* bus number, print that out instead.
|
||||
*/
|
||||
tbl->it_busno = 0;
|
||||
pe->tce_inval_reg_phys = be64_to_cpup(swinvp);
|
||||
tbl->it_index = (unsigned long)ioremap(pe->tce_inval_reg_phys,
|
||||
8);
|
||||
tbl->it_type = TCE_PCI_SWINV_CREATE | TCE_PCI_SWINV_FREE;
|
||||
tbl->it_type |= (TCE_PCI_SWINV_CREATE | TCE_PCI_SWINV_FREE);
|
||||
}
|
||||
iommu_init_table(tbl, phb->hose->node);
|
||||
iommu_register_group(tbl, pci_domain_nr(pe->pbus), pe->pe_number);
|
||||
|
|
Loading…
Reference in New Issue