iommu/mediatek: Delete the leaf in the tlb_flush
In our tlb range flush, we don't care the "leaf". Remove it to simplify the code. no functional change. "granule" also is unnecessary for us, Keep it satisfy the format of tlb_flush_walk. Signed-off-by: Yong Wu <yong.wu@mediatek.com> Reviewed-by: Robin Murphy <robin.murphy@arm.com> Signed-off-by: Joerg Roedel <jroedel@suse.de>
This commit is contained in:
parent
a7a04ea34e
commit
67caf7e2b5
|
@ -174,8 +174,7 @@ static void mtk_iommu_tlb_flush_all(void *cookie)
|
||||||
}
|
}
|
||||||
|
|
||||||
static void mtk_iommu_tlb_add_flush_nosync(unsigned long iova, size_t size,
|
static void mtk_iommu_tlb_add_flush_nosync(unsigned long iova, size_t size,
|
||||||
size_t granule, bool leaf,
|
size_t granule, void *cookie)
|
||||||
void *cookie)
|
|
||||||
{
|
{
|
||||||
struct mtk_iommu_data *data = cookie;
|
struct mtk_iommu_data *data = cookie;
|
||||||
|
|
||||||
|
@ -223,19 +222,7 @@ static void mtk_iommu_tlb_flush_walk(unsigned long iova, size_t size,
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
|
||||||
spin_lock_irqsave(&data->tlb_lock, flags);
|
spin_lock_irqsave(&data->tlb_lock, flags);
|
||||||
mtk_iommu_tlb_add_flush_nosync(iova, size, granule, false, cookie);
|
mtk_iommu_tlb_add_flush_nosync(iova, size, granule, cookie);
|
||||||
mtk_iommu_tlb_sync(cookie);
|
|
||||||
spin_unlock_irqrestore(&data->tlb_lock, flags);
|
|
||||||
}
|
|
||||||
|
|
||||||
static void mtk_iommu_tlb_flush_leaf(unsigned long iova, size_t size,
|
|
||||||
size_t granule, void *cookie)
|
|
||||||
{
|
|
||||||
struct mtk_iommu_data *data = cookie;
|
|
||||||
unsigned long flags;
|
|
||||||
|
|
||||||
spin_lock_irqsave(&data->tlb_lock, flags);
|
|
||||||
mtk_iommu_tlb_add_flush_nosync(iova, size, granule, true, cookie);
|
|
||||||
mtk_iommu_tlb_sync(cookie);
|
mtk_iommu_tlb_sync(cookie);
|
||||||
spin_unlock_irqrestore(&data->tlb_lock, flags);
|
spin_unlock_irqrestore(&data->tlb_lock, flags);
|
||||||
}
|
}
|
||||||
|
@ -253,7 +240,7 @@ static void mtk_iommu_tlb_flush_page_nosync(struct iommu_iotlb_gather *gather,
|
||||||
static const struct iommu_flush_ops mtk_iommu_flush_ops = {
|
static const struct iommu_flush_ops mtk_iommu_flush_ops = {
|
||||||
.tlb_flush_all = mtk_iommu_tlb_flush_all,
|
.tlb_flush_all = mtk_iommu_tlb_flush_all,
|
||||||
.tlb_flush_walk = mtk_iommu_tlb_flush_walk,
|
.tlb_flush_walk = mtk_iommu_tlb_flush_walk,
|
||||||
.tlb_flush_leaf = mtk_iommu_tlb_flush_leaf,
|
.tlb_flush_leaf = mtk_iommu_tlb_flush_walk,
|
||||||
.tlb_add_page = mtk_iommu_tlb_flush_page_nosync,
|
.tlb_add_page = mtk_iommu_tlb_flush_page_nosync,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
@ -475,7 +462,7 @@ static void mtk_iommu_iotlb_sync(struct iommu_domain *domain,
|
||||||
|
|
||||||
spin_lock_irqsave(&data->tlb_lock, flags);
|
spin_lock_irqsave(&data->tlb_lock, flags);
|
||||||
mtk_iommu_tlb_add_flush_nosync(gather->start, length, gather->pgsize,
|
mtk_iommu_tlb_add_flush_nosync(gather->start, length, gather->pgsize,
|
||||||
false, data);
|
data);
|
||||||
mtk_iommu_tlb_sync(data);
|
mtk_iommu_tlb_sync(data);
|
||||||
spin_unlock_irqrestore(&data->tlb_lock, flags);
|
spin_unlock_irqrestore(&data->tlb_lock, flags);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue