Re: [PATCHv3 10/19] iommu/tegra: smmu: Get "nvidia,swgroups" from DT
From: Mark Rutland <hidden>
Date: 2013-10-31 17:15:31
Also in:
linux-iommu, linux-tegra
On Fri, Oct 18, 2013 at 11:26:51AM +0100, Hiroshi Doyu wrote:
quoted hunk
This provides the info about which H/W Accelerators are supported on Tegra SoC. This info is passed from DT. This is necessary to have the unified SMMU driver among Tegra SoCs. Instead of using platform data, DT passes "nvidia,swgroups" now. DT is mandatory in Tegra. Signed-off-by: Hiroshi Doyu <redacted> --- .../bindings/iommu/nvidia,tegra30-smmu.txt | 6 +++++ drivers/iommu/tegra-smmu.c | 31 ++++++++++------------ 2 files changed, 20 insertions(+), 17 deletions(-)diff --git a/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt b/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt index 89fb543..6a844b3 100644 --- a/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt +++ b/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt@@ -8,6 +8,11 @@ Required properties: - nvidia,#asids : # of ASIDs - dma-window : IOVA start address and length. - nvidia,ahb : phandle to the ahb bus connected to SMMU. +- nvidia,swgroups: A bitmap of supported HardWare Accelerators(HWA). + Each bit represents one swgroup. The assignments may be found in header + file <dt-bindings/memory/tegra-swgroup.h>. Its max is 64. 2 cells + are required. This unique ID info can be used to calculate + MC_SMMU_<SWGROUP name>_ASID_0 offset and HOTRESET bit.
I'd prefer that you pointed out the type of the property explicitly at the beginning of the description. It's difficult to spot it in the middle, and while the current description defines that the property is two cells, it doesn't point out that they are expected to form a (big endian) u64 (rather than an array of two u32s). While we don't yet have a formal set of rules for consistent binding documentation, defining the explicitly is one of the requirements I'd like to push for in new bindings. Placing this at the start of the description makes it far easier to skim a binding document and perform basic sanity checks of the binding itself, dts using the binding, and kernel code parsing it. Cheers, Mark.