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
