linux-kernel - Re: [PATCH 1/4] dt-bindings: mailbox: qcom,apcs: Add separate node for clock-controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <175053907628.4372.13105365536734444855@lazor>
Date: Sat, 21 Jun 2025 13:51:16 -0700
From: Stephen Boyd <sboyd@...nel.org>
To: Bjorn Andersson <andersson@...nel.org>, Krzysztof Kozlowski <krzk@...nel.org>
Cc: Stephan Gerhold <stephan.gerhold@...aro.org>, Saravana Kannan <saravanak@...gle.com>, Rob Herring <robh@...nel.org>, Jassi Brar <jassisinghbrar@...il.com>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Michael Turquette <mturquette@...libre.com>, linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, linux-clk@...r.kernel.org, Georgi Djakov <djakov@...nel.org>, Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Subject: Re: [PATCH 1/4] dt-bindings: mailbox: qcom,apcs: Add separate node for clock-controller

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.