Thread (11 messages) 11 messages, 4 authors, 2020-06-30

RE: [PATCH v7 1/3] iommu/arm-smmu: add NVIDIA implementation for dual ARM MMU-500 usage

From: Krishna Reddy <vdumpa@nvidia.com>
Date: 2020-06-29 22:49:42
Also in: linux-iommu, linux-tegra, lkml

quoted
+     if (!nvidia_smmu->bases[0])
+             nvidia_smmu->bases[0] = smmu->base;
+
+     return nvidia_smmu->bases[inst] + (page << smmu->pgshift); }
Not critical -- just a nit: why not put the bases[0] in init()?
smmu->base is not available during nvidia_smmu_impl_init() call. It is set afterwards in arm-smmu.c.
It can't be avoided without changing the devm_ioremap() and impl_init() call order in arm-smmu.c. 

-KR

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help