On Thu, Nov 03, 2022 at 05:03:01AM +0000, Tian, Kevin wrote:
> > @@ -4458,7 +4458,11 @@ static bool intel_iommu_capable(struct device
> > *dev, enum iommu_cap cap)
> > return irq_remapping_enabled == 1;
> > if (cap == IOMMU_CAP_PRE_BOOT_PROTECTION)
> > return dmar_platform_optin();
> > + if (cap == IOMMU_CAP_ENFORCE_CACHE_COHERENCY) {
> > + struct device_domain_info *info = dev_iommu_priv_get(dev);
> >
> > + return ecap_sc_support(info->iommu->ecap);
> > + }
> > return false;
> > }
>
> use switch/case while making this change.
Done:
static bool intel_iommu_capable(struct device *dev, enum iommu_cap cap)
{
struct device_domain_info *info = dev_iommu_priv_get(dev);
switch (cap) {
case IOMMU_CAP_CACHE_COHERENCY:
return true;
case IOMMU_CAP_INTR_REMAP:
return irq_remapping_enabled == 1;
case IOMMU_CAP_PRE_BOOT_PROTECTION:
return dmar_platform_optin();
case IOMMU_CAP_ENFORCE_CACHE_COHERENCY:
return ecap_sc_support(info->iommu->ecap);
default:
return false;
}
}
Thanks,
Jason
