On Tue, Apr 29, 2025 at 11:34:13AM -0300, Jason Gunthorpe wrote:
> These drivers all set the domain->pgsize_bitmap in their
> domain_alloc_paging() functions, so the ops value is never used. Delete
> it.
>
> Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxx>
> ---
> drivers/iommu/apple-dart.c | 1 -
> drivers/iommu/intel/iommu.c | 1 -
> drivers/iommu/iommufd/selftest.c | 1 -
> drivers/iommu/riscv/iommu.c | 1 -
> drivers/iommu/virtio-iommu.c | 6 ++----
> 5 files changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/iommu/riscv/iommu.c b/drivers/iommu/riscv/iommu.c
> index bb57092ca90110..2d0d31ba28860a 100644
> --- a/drivers/iommu/riscv/iommu.c
> +++ b/drivers/iommu/riscv/iommu.c
> @@ -1533,7 +1533,6 @@ static void riscv_iommu_release_device(struct device *dev)
> }
>
> static const struct iommu_ops riscv_iommu_ops = {
> - .pgsize_bitmap = SZ_4K,
> .of_xlate = riscv_iommu_of_xlate,
> .identity_domain = &riscv_iommu_identity_domain,
> .blocked_domain = &riscv_iommu_blocking_domain,
Reviewed-by: Tomasz Jeznach <tjeznach@xxxxxxxxxxxx> # for RISC-V
Sorry for the delayed response.
- Tomasz
