mbox series

[0/4] iommu: Remove iommu_fwspec ops

Message ID cover.1713523251.git.robin.murphy@arm.com
Headers show
Series iommu: Remove iommu_fwspec ops | expand

Message

Robin Murphy April 19, 2024, 4:55 p.m. UTC
Hi All,

Building on top of the arch_setup_dma_ops() cleanup[1], the next step
down the chain is {acpi,of}_dma_configure()... There's plenty to do
here, but it may as well start with this fairly self-contained little
cleanup, pruning yet more redundancy and exposed API surface.

Thanks,
Robin.

[1] https://lore.kernel.org/linux-iommu/cover.1713523152.git.robin.murphy@arm.com


Robin Murphy (4):
  iommu: Resolve fwspec ops automatically
  ACPI: Retire acpi_iommu_fwspec_ops()
  OF: Simplify of_iommu_configure()
  iommu: Remove iommu_fwspec ops

 drivers/acpi/arm64/iort.c             | 19 +++-------
 drivers/acpi/scan.c                   | 38 +++++---------------
 drivers/acpi/viot.c                   | 11 ++----
 drivers/iommu/arm/arm-smmu/arm-smmu.c |  3 +-
 drivers/iommu/iommu-priv.h            |  7 ++++
 drivers/iommu/iommu.c                 | 20 +++++------
 drivers/iommu/mtk_iommu_v1.c          |  2 +-
 drivers/iommu/of_iommu.c              | 50 ++++++++++-----------------
 drivers/iommu/tegra-smmu.c            |  2 +-
 drivers/of/device.c                   | 30 ++++++----------
 include/acpi/acpi_bus.h               |  3 +-
 include/linux/iommu.h                 | 15 ++------
 12 files changed, 66 insertions(+), 134 deletions(-)

Comments

Rob Herring April 19, 2024, 10 p.m. UTC | #1
On Fri, 19 Apr 2024 17:56:01 +0100, Robin Murphy wrote:
> We no longer have a notion of partially-initialised fwspecs existing,
> and we also no longer need to use an iommu_ops pointer to return status
> to of_dma_configure(). Clean up the remains of those, which lends itself
> to clarifying the logic around the dma_range_map allocation as well.
> 
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> ---
>  drivers/iommu/of_iommu.c | 29 ++++++++++-------------------
>  drivers/of/device.c      | 30 +++++++++++-------------------
>  2 files changed, 21 insertions(+), 38 deletions(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>