--- v7
+++ v3
@@ -1,132 +1,65 @@
-This is a part of moving DMA window programming to an iommu_ops
-callback.
-
-This is a mechanical patch.
+This changes few functions to receive a powerpc_iommu pointer
+rather than PE as they are going to be a part of upcoming
+powerpc_iommu_ops callback set.
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
---
- arch/powerpc/platforms/powernv/pci-ioda.c | 85 ++++++++++++++++++++-----------
- 1 file changed, 56 insertions(+), 29 deletions(-)
+ arch/powerpc/platforms/powernv/pci-ioda.c | 13 ++++++++-----
+ 1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c
-index 908863a..64b7cfe 100644
+index f542819..29bd7a4 100644
--- a/arch/powerpc/platforms/powernv/pci-ioda.c
+++ b/arch/powerpc/platforms/powernv/pci-ioda.c
-@@ -1391,6 +1391,57 @@ static void pnv_pci_free_table(struct iommu_table *tbl)
- memset(tbl, 0, sizeof(struct iommu_table));
+@@ -1360,10 +1360,12 @@ static __be64 *pnv_alloc_tce_table(int nid,
+ return addr;
}
-+static long pnv_pci_ioda2_set_window(struct pnv_ioda_pe *pe,
-+ struct iommu_table *tbl)
-+{
-+ struct pnv_phb *phb = pe->phb;
-+ const __be64 *swinvp;
-+ int64_t rc;
-+ const __u64 start_addr = tbl->it_offset << tbl->it_page_shift;
-+ const __u64 win_size = tbl->it_size << tbl->it_page_shift;
-+
-+ pe_info(pe, "Setting up window at %llx..%llx pagesize=0x%x tablesize=0x%lx\n",
-+ start_addr, start_addr + win_size - 1,
-+ 1UL << tbl->it_page_shift, tbl->it_size << 3);
-+
-+ pe->table_group.tables[0] = *tbl;
-+ tbl = &pe->table_group.tables[0];
-+ tbl->it_group = &pe->table_group;
-+
-+ /*
-+ * Map TCE table through TVT. The TVE index is the PE number
-+ * shifted by 1 bit for 32-bits DMA space.
-+ */
-+ rc = opal_pci_map_pe_dma_window(phb->opal_id, pe->pe_number,
-+ pe->pe_number << 1, 1, __pa(tbl->it_base),
-+ tbl->it_size << 3, 1ULL << tbl->it_page_shift);
-+ if (rc) {
-+ pe_err(pe, "Failed to configure TCE table, err %ld\n", rc);
-+ goto fail;
-+ }
-+
-+ /* OPAL variant of PHB3 invalidated TCEs */
-+ swinvp = of_get_property(phb->hose->dn, "ibm,opal-tce-kill", NULL);
-+ if (swinvp) {
-+ /* We need a couple more fields -- an address and a data
-+ * to or. Since the bus is only printed out on table free
-+ * errors, and on the first pass the data will be a relative
-+ * bus number, print that out instead.
-+ */
-+ 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);
-+ }
-+
-+ return 0;
-+fail:
-+ if (pe->tce32_seg >= 0)
-+ pe->tce32_seg = -1;
-+
-+ return rc;
-+}
-+
- static void pnv_pci_ioda2_set_bypass(struct pnv_ioda_pe *pe, bool enable)
+-static long pnv_pci_ioda2_create_table(struct pnv_ioda_pe *pe,
++static long pnv_pci_ioda2_create_table(struct powerpc_iommu *iommu,
+ __u32 page_shift, __u32 window_shift, __u32 levels,
+ struct iommu_table *tbl)
{
- uint16_t window_id = (pe->pe_number << 1 ) + 1;
-@@ -1463,7 +1514,6 @@ static struct iommu_table_group_ops pnv_pci_ioda2_ops = {
- static void pnv_pci_ioda2_setup_dma_pe(struct pnv_phb *phb,
- struct pnv_ioda_pe *pe)
++ struct pnv_ioda_pe *pe = container_of(iommu, struct pnv_ioda_pe,
++ iommu);
+ int nid = pe->phb->hose->node;
+ void *addr;
+ unsigned long tce_table_size, left;
+@@ -1419,9 +1421,11 @@ static void pnv_pci_ioda2_free_table(struct iommu_table *tbl)
+ iommu_reset_table(tbl, "ioda2");
+ }
+
+-static long pnv_pci_ioda2_set_window(struct pnv_ioda_pe *pe,
++static long pnv_pci_ioda2_set_window(struct powerpc_iommu *iommu,
+ struct iommu_table *tbl)
{
-- const __be64 *swinvp;
- unsigned int end;
- struct iommu_table *tbl = &pe->table_group.tables[0];
++ struct pnv_ioda_pe *pe = container_of(iommu, struct pnv_ioda_pe,
++ iommu);
+ struct pnv_phb *phb = pe->phb;
+ const __be64 *swinvp;
int64_t rc;
-@@ -1493,31 +1543,14 @@ static void pnv_pci_ioda2_setup_dma_pe(struct pnv_phb *phb,
- pe->table_group.ops = &pnv_pci_ioda2_ops;
- #endif
+@@ -1554,12 +1558,11 @@ static void pnv_pci_ioda2_setup_dma_pe(struct pnv_phb *phb,
-- /*
-- * Map TCE table through TVT. The TVE index is the PE number
-- * shifted by 1 bit for 32-bits DMA space.
-- */
-- rc = opal_pci_map_pe_dma_window(phb->opal_id, pe->pe_number,
-- pe->pe_number << 1, 1, __pa(tbl->it_base),
-- tbl->it_size << 3, 1ULL << tbl->it_page_shift);
-+ rc = pnv_pci_ioda2_set_window(pe, tbl);
+ /* The PE will reserve all possible 32-bits space */
+ pe->tce32_seg = 0;
+-
+ end = (1 << ilog2(phb->ioda.m32_pci_base));
+ pe_info(pe, "Setting up 32-bit TCE table at 0..%08x\n",
+ end);
+
+- rc = pnv_pci_ioda2_create_table(pe, IOMMU_PAGE_SHIFT_4K,
++ rc = pnv_pci_ioda2_create_table(&pe->iommu, IOMMU_PAGE_SHIFT_4K,
+ ilog2(phb->ioda.m32_pci_base),
+ POWERPC_IOMMU_DEFAULT_LEVELS, tbl);
+ if (rc) {
+@@ -1571,7 +1574,7 @@ static void pnv_pci_ioda2_setup_dma_pe(struct pnv_phb *phb,
+ pe->iommu.tables[0].it_iommu = &pe->iommu;
+ pe->iommu.ops = &pnv_pci_ioda2_ops;
+
+- rc = pnv_pci_ioda2_set_window(pe, tbl);
++ rc = pnv_pci_ioda2_set_window(&pe->iommu, tbl);
if (rc) {
pe_err(pe, "Failed to configure 32-bit TCE table,"
" err %ld\n", rc);
-- goto fail;
-- }
--
-- /* OPAL variant of PHB3 invalidated TCEs */
-- swinvp = of_get_property(phb->hose->dn, "ibm,opal-tce-kill", NULL);
-- if (swinvp) {
-- /* We need a couple more fields -- an address and a data
-- * to or. Since the bus is only printed out on table free
-- * errors, and on the first pass the data will be a relative
-- * bus number, print that out instead.
-- */
-- 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);
-+ pnv_pci_free_table(tbl);
-+ if (pe->tce32_seg >= 0)
-+ pe->tce32_seg = -1;
-+ return;
- }
- iommu_register_group(&pe->table_group, phb->hose->global_number,
- pe->pe_number);
-@@ -1531,12 +1564,6 @@ static void pnv_pci_ioda2_setup_dma_pe(struct pnv_phb *phb,
- /* Also create a bypass window */
- if (!pnv_iommu_bypass_disabled)
- pnv_pci_ioda2_setup_bypass_pe(phb, pe);
--
-- return;
--fail:
-- if (pe->tce32_seg >= 0)
-- pe->tce32_seg = -1;
-- pnv_pci_free_table(tbl);
- }
-
- static void pnv_ioda_setup_dma(struct pnv_phb *phb)
--
2.0.0