Re: [PATCH] dt-bindings: arm-smmu: disallow clocks when not used

From: Marijn Suijten
Date: Thu Dec 22 2022 - 09:48:02 EST


On 2022-12-22 15:03:20, Krzysztof Kozlowski wrote:
> On 22/12/2022 14:33, Marijn Suijten wrote:
> > On 2022-12-22 11:36:16, Krzysztof Kozlowski wrote:
> >> On 22/12/2022 11:16, Marijn Suijten wrote:
> >>> Is this missing a cc to linux-arm-msm?
> >>
> >> No, it is not (or maybe but then fix MAINTAINERS). The policy is to use
> >> get_maintainers.pl to CC people.
> >
> > Yes, that is the question: is it in MANTAINERS and if not, why not?
>
> You can check by yourself if it is there.

It's not there.

> Why not? I don't know. Could be that no one ever added it there.

Let's leave it like that then :)

<snip>

> > sc7280 has two IOMMU nodes. One with clocks (should not be in this
> > list), the other doesn't have clocks (should be in this list).
> >
> > How do you want to address that?
>
> No, because it is the same compatible.

That is the point. We can tell them apart based on the presence of
"qcom,adreno-smmu" though. But if it is not spitting out any errors
right now, let's not bother.

- Marijn