From mboxrd@z Thu Jan 1 00:00:00 1970 From: andreas.herrmann@calxeda.com (Andreas Herrmann) Date: Thu, 10 Oct 2013 00:38:02 +0200 Subject: [PATCH 3/7] iommu/arm-smmu: Introduce stream ID masking In-Reply-To: <1381358286-27065-1-git-send-email-andreas.herrmann@calxeda.com> References: <1381358286-27065-1-git-send-email-andreas.herrmann@calxeda.com> Message-ID: <1381358286-27065-4-git-send-email-andreas.herrmann@calxeda.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Ie. use a mask based on smr_mask_bits to map all stream IDs of an SMMU to one context. This behaviour is controlled per SMMU node with DT property "arm,smmu-mask-stream-ids" and is only allowed if just a single master is attached to an SMMU. If the option is specified, all stream-ids that are provided in DT for the single master have no relevance. This is useful/convenient if a master has more than 8 stream-ids or if not all stream-ids are known for a master device. Signed-off-by: Andreas Herrmann --- drivers/iommu/arm-smmu.c | 44 ++++++++++++++++++++++++++++++++------------ 1 file changed, 32 insertions(+), 12 deletions(-) diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c index 87239e8..f877d02 100644 --- a/drivers/iommu/arm-smmu.c +++ b/drivers/iommu/arm-smmu.c @@ -50,6 +50,7 @@ /* Driver options */ #define ARM_SMMU_OPT_ISOLATE_DEVICES (1 << 0) +#define ARM_SMMU_OPT_MASK_STREAM_IDS (1 << 1) /* Maximum number of stream IDs assigned to a single device */ #define MAX_MASTER_STREAMIDS 8 @@ -362,6 +363,7 @@ struct arm_smmu_device { u32 num_mapping_groups; DECLARE_BITMAP(smr_map, ARM_SMMU_MAX_SMRS); + u32 smr_mask_bits; unsigned long input_size; unsigned long s1_output_size; @@ -373,6 +375,7 @@ struct arm_smmu_device { struct list_head list; struct rb_root masters; + u32 num_masters; }; struct arm_smmu_cfg { @@ -410,6 +413,7 @@ struct arm_smmu_option_prop { static struct arm_smmu_option_prop arm_smmu_options [] = { { ARM_SMMU_OPT_ISOLATE_DEVICES, "arm,smmu-isolate-devices" }, + { ARM_SMMU_OPT_MASK_STREAM_IDS, "arm,smmu-mask-stream-ids" }, { 0, NULL}, }; @@ -475,6 +479,7 @@ static int insert_smmu_master(struct arm_smmu_device *smmu, rb_link_node(&master->node, parent, new); rb_insert_color(&master->node, &smmu->masters); + smmu->num_masters++; return 0; } @@ -507,8 +512,20 @@ static int register_smmu_master(struct arm_smmu_device *smmu, master->of_node = masterspec->np; master->num_streamids = masterspec->args_count; - for (i = 0; i < master->num_streamids; ++i) - master->streamids[i] = masterspec->args[i]; + if (smmu->options & ARM_SMMU_OPT_MASK_STREAM_IDS) { + if (smmu->num_masters) { + dev_err(dev, "%s not supported with multiple masters\n", + get_driver_option_property( + ARM_SMMU_OPT_MASK_STREAM_IDS)); + return -EINVAL; + } + /* set fixed streamid (0) that will be used for masking */ + master->num_streamids = 1; + master->streamids[0] = 0; + } else { + for (i = 0; i < master->num_streamids; ++i) + master->streamids[i] = masterspec->args[i]; + } return insert_smmu_master(smmu, master); } @@ -1050,17 +1067,20 @@ static int arm_smmu_master_configure_smrs(struct arm_smmu_device *smmu, goto err_free_smrs; } - smrs[i] = (struct arm_smmu_smr) { - .idx = idx, - .mask = 0, /* We don't currently share SMRs */ - .id = master->streamids[i], - }; + smrs[i].idx = idx; + smrs[i].id = master->streamids[i]; + + if (smmu->options & ARM_SMMU_OPT_MASK_STREAM_IDS) + smrs[i].mask = smmu->smr_mask_bits; + else + smrs[i].mask = 0; } /* It worked! Now, poke the actual hardware */ for (i = 0; i < master->num_streamids; ++i) { u32 reg = SMR_VALID | smrs[i].id << SMR_ID_SHIFT | smrs[i].mask << SMR_MASK_SHIFT; + dev_dbg(smmu->dev, "SMR%d: 0x%x\n", smrs[i].idx, reg); writel_relaxed(reg, gr0_base + ARM_SMMU_GR0_SMR(smrs[i].idx)); } @@ -1713,7 +1733,7 @@ static int arm_smmu_device_cfg_probe(struct arm_smmu_device *smmu) } if (id & ID0_SMS) { - u32 smr, sid, mask; + u32 smr, sid; smmu->features |= ARM_SMMU_FEAT_STREAM_MATCH; smmu->num_mapping_groups = (id >> ID0_NUMSMRG_SHIFT) & @@ -1729,18 +1749,18 @@ static int arm_smmu_device_cfg_probe(struct arm_smmu_device *smmu) writel_relaxed(smr, gr0_base + ARM_SMMU_GR0_SMR(0)); smr = readl_relaxed(gr0_base + ARM_SMMU_GR0_SMR(0)); - mask = (smr >> SMR_MASK_SHIFT) & SMR_MASK_MASK; + smmu->smr_mask_bits = (smr >> SMR_MASK_SHIFT) & SMR_MASK_MASK; sid = (smr >> SMR_ID_SHIFT) & SMR_ID_MASK; - if ((mask & sid) != sid) { + if ((smmu->smr_mask_bits & sid) != sid) { dev_err(smmu->dev, "SMR mask bits (0x%x) insufficient for ID field (0x%x)\n", - mask, sid); + smmu->smr_mask_bits, sid); return -ENODEV; } dev_notice(smmu->dev, "\tstream matching with %u register groups, mask 0x%x", - smmu->num_mapping_groups, mask); + smmu->num_mapping_groups, smmu->smr_mask_bits); } /* ID1 */ -- 1.7.9.5