On 13/11/2024 06:00, Caleb Connolly wrote:
It turns out this is a very real stream ID. Who woulda thought?
Drop the 0 check on the SID, there's no reason for it to be there in the first
place.
Signed-off-by: Caleb Connolly <[email protected]>
---
drivers/iommu/qcom-hyp-smmu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iommu/qcom-hyp-smmu.c b/drivers/iommu/qcom-hyp-smmu.c
index 1b5a09bb7b39..7e07487ad384 100644
--- a/drivers/iommu/qcom-hyp-smmu.c
+++ b/drivers/iommu/qcom-hyp-smmu.c
@@ -133,9 +133,9 @@ static int get_stream_id(struct udevice *dev)
struct ofnode_phandle_args args;
int count = ofnode_parse_phandle_with_args(node, "iommus",
"#iommu-cells", 0, 0, &args);
- if (count < 0 || args.args[0] == 0) {
+ if (count < 0) {
printf("Error: %s: iommus property not found or wrong number of
cells\n",
__func__);
return -EINVAL;
}
Reviewed-by: Neil Armstrong <[email protected]>
