On Wed, Jun 26, 2013 at 12:28:05PM +0300, Hiroshi Doyu wrote: > This is a bitmap that indicates which HardWare Accelerators(HWA) are > supported on Tegra30 SoC. > > Signed-off-by: Hiroshi Doyu <[email protected]> > --- > Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt | 6 +++++- > arch/arm/boot/dts/tegra30.dtsi | 1 + > 2 files changed, 6 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt > b/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt > index 89fb543..6be51f6 100644 > --- a/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt > +++ b/Documentation/devicetree/bindings/iommu/nvidia,tegra30-smmu.txt > @@ -8,14 +8,18 @@ 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 bit map of supported HardWare Accelerators(HWA).
Nit: you use the spelling "bitmap" in the subject but "bit map" here.
Both are correct but perhaps we should stick to one.
> + Each bit represents one sgroup. The assignments may be found in header
Nit: "swgroup"
> - smmu {
> + iommu {
This change isn't really related, but given that the tegra30.dtsi
already uses it I guess we can keep it as part of this patch.
> compatible = "nvidia,tegra30-smmu";
> reg = <0x7000f010 0x02c
> 0x7000f1f0 0x010
> 0x7000f228 0x05c>;
> nvidia,#asids = <4>; /* # of ASIDs */
> dma-window = <0 0x40000000>; /* IOVA start & length */
> + nvidia,swgroups = <0x00000000 0x000779ff>;
Perhaps this should be a symbolic name too? Perhaps something like
TEGRA30_SWGID_ALL?
Thierry
pgpIOn9v4D1s4.pgp
Description: PGP signature
