Re: [PATCH v6 17/25] iommu/qcom_iommu: Add an IOMMU_IDENTITIY_DOMAIN

Jason Gunthorpe
2023-08-14T15:37:03Z

On Mon, Aug 14, 2023 at 02:32:33PM +0800, Baolu Lu wrote:

> > + pm_runtime_get_sync(qcom_iommu->dev);
> > + for (i = 0; i < fwspec->num_ids; i++) {
> > + struct qcom_iommu_ctx *ctx = to_ctx(qcom_domain, fwspec->ids[i]);
> > +
> > + /* Disable the context bank: */
> > + iommu_writel(ctx, ARM_SMMU_CB_SCTLR, 0);
> > +
> > + ctx->domain = NULL;
>
> Does setting ctx->domain to NULL still match this semantics?

Yes, I did not try to fix this driver. NULL means identity in the
ctx->domain.

Thanks,
Jason