Re: [PATCH] iommu: fix ARM_SMMU vs QCOM_SCM compilation

2021-10-09 Thread Steev Klimaszewski
On 10/9/21 9:33 PM, Dmitry Baryshkov wrote: After commit 424953cf3c66 ("qcom_scm: hide Kconfig symbol") arm-smmu got qcom_smmu_impl_init() call guarded by IS_ENABLED(CONFIG_ARM_SMMU_QCOM). However the CONFIG_ARM_SMMU_QCOM Kconfig entry does not exist, so the qcom_smmu_impl_init() is never

Re: [PATCH] iommu: fix ARM_SMMU vs QCOM_SCM compilation

2021-10-09 Thread Bjorn Andersson
On Sat 09 Oct 21:33 CDT 2021, Dmitry Baryshkov wrote: > After commit 424953cf3c66 ("qcom_scm: hide Kconfig symbol") arm-smmu got > qcom_smmu_impl_init() call guarded by IS_ENABLED(CONFIG_ARM_SMMU_QCOM). > However the CONFIG_ARM_SMMU_QCOM Kconfig entry does not exist, so the >

[PATCH] iommu: fix ARM_SMMU vs QCOM_SCM compilation

2021-10-09 Thread Dmitry Baryshkov
After commit 424953cf3c66 ("qcom_scm: hide Kconfig symbol") arm-smmu got qcom_smmu_impl_init() call guarded by IS_ENABLED(CONFIG_ARM_SMMU_QCOM). However the CONFIG_ARM_SMMU_QCOM Kconfig entry does not exist, so the qcom_smmu_impl_init() is never called. So, let's fix this by always calling

Re: [PATCH v7 9/9] iommu/dma: Reserve any RMR regions associated with a dev

2021-10-09 Thread Jon Nettleton
On Fri, Oct 8, 2021 at 3:10 PM Robin Murphy wrote: > > On 2021-08-05 09:07, Shameer Kolothum wrote: > > Get ACPI IORT RMR regions associated with a dev reserved > > so that there is a unity mapping for them in SMMU. > > This feels like most of it belongs in the IORT code rather than > iommu-dma

Re: [PATCH v7 2/9] ACPI/IORT: Add support for RMR node parsing

2021-10-09 Thread Jon Nettleton
On Fri, Oct 8, 2021 at 2:49 PM Robin Murphy wrote: > > On 2021-08-05 09:07, Shameer Kolothum wrote: > > Add support for parsing RMR node information from ACPI. > > > > Find the associated streamid and smmu node info from the > > RMR node and populate a linked list with RMR memory > > descriptors.

Re: [PATCH v7 1/9] iommu: Introduce a union to struct iommu_resv_region

2021-10-09 Thread Jon Nettleton
On Fri, Oct 8, 2021 at 2:14 PM Robin Murphy wrote: > > On 2021-08-05 09:07, Shameer Kolothum wrote: > > A union is introduced to struct iommu_resv_region to hold > > any firmware specific data. This is in preparation to add > > support for IORT RMR reserve regions and the union now holds > > the