Re: [PATCH 4/4] iommu: make inclusion of riscv directory conditional

From: Joerg Roedel
Date: Thu Mar 20 2025 - 04:07:50 EST


On Tue, Mar 11, 2025 at 11:15:12AM +0100, Rolf Eike Beer wrote:
> Nothing in there is active if CONFIG_RISCV_IOMMU is not enabled, so the whole
> directory can depend on that switch as well.
>
> Fixes: 5c0ebbd3c6c6 ("iommu/riscv: Add RISC-V IOMMU platform device driver")
> Signed-off-by: Rolf Eike Beer <eb@xxxxxxxxx>
> ---
> drivers/iommu/Makefile | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/iommu/Makefile b/drivers/iommu/Makefile
> index 41b00cdc8621..17133876f777 100644
> --- a/drivers/iommu/Makefile
> +++ b/drivers/iommu/Makefile
> @@ -1,7 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0
> -obj-y += arm/ iommufd/ riscv/
> +obj-y += arm/ iommufd/
> obj-$(CONFIG_AMD_IOMMU) += amd/
> obj-$(CONFIG_INTEL_IOMMU) += intel/
> +obj-$(CONFIG_RISCV_IOMMU) += riscv/
> obj-$(CONFIG_IOMMU_API) += iommu.o
> obj-$(CONFIG_IOMMU_API) += iommu-traces.o
> obj-$(CONFIG_IOMMU_API) += iommu-sysfs.o

This allows for some additional cleanups in the Makefiles of the
sub-directory, no? Same for the other patches in this series.

Regards,

Joerg