Re: [PATCH 1/4] dt-bindings: mailbox: qcom,apcs: Add separate node for clock-controller
From: Stephen Boyd
Date: Sat Jun 21 2025 - 16:51:30 EST
Quoting Bjorn Andersson (2025-06-10 20:31:57)
>
> I'm still sceptical here.
>
> In the first snippet above, we describe a single IP block which provides
> mailboxes and clocks.
>
> In the second snippet we're saying that the IP block is a mailbox, and
> then it somehow have a subcomponent which is a clock provider.
>
> It seems to me that we're choosing the second option because it better
> fits the Linux implementation, rather than that it would be a better
> representation of the hardware. To the point that we can't even describe
> the register range of the subcomponent...
>
Agreed. Don't workaround problems in the kernel by changing the binding
to have sub-nodes.