linux-kernel - Re: [PATCH v1 07/11] dt-bindings: clk: microchip: mpfs: remove first reg region

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] [day] [month] [year] [list]

Message-ID: <172791218968.1549326.10092947836585074554.robh@kernel.org>
Date: Wed, 2 Oct 2024 18:36:30 -0500
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Conor Dooley <conor@...nel.org>
Cc: linux-clk@...r.kernel.org, Neil Armstrong <neil.armstrong@...aro.org>,
	valentina.fernandezalanis@...rochip.com,
	Conor Dooley <conor.dooley@...rochip.com>,
	Kevin Hilman <khilman@...libre.com>,
	linux-riscv@...ts.infradead.org, pierre-henry.moussay@...rochip.com,
	Philipp Zabel <p.zabel@...gutronix.de>,
	Jerome Brunet <jbrunet@...libre.com>,
	Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
	Daire McNamara <daire.mcnamara@...rochip.com>,
	Albert Ou <aou@...s.berkeley.edu>,
	Michael Turquette <mturquette@...libre.com>,
	linux-kernel@...r.kernel.org,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	devicetree@...r.kernel.org, Palmer Dabbelt <palmer@...belt.com>,
	linux-amlogic@...ts.infradead.org, Stephen Boyd <sboyd@...nel.org>,
	Jassi Brar <jassisinghbrar@...il.com>,
	Paul Walmsley <paul.walmsley@...ive.com>,
	Lee Jones <lee@...nel.org>, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v1 07/11] dt-bindings: clk: microchip: mpfs: remove first
 reg region


On Wed, 02 Oct 2024 11:48:05 +0100, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@...rochip.com>
> 
> The first reg region in this binding is not exclusively for clocks, as
> evidenced by the dual role of this device as a reset controller at
> present. The first region is however better described by a simple-mfd
> syscon, but this would have require a significant re-write of the
> devicetree for the platform, so the easy way out was chosen when reset
> support was first introduced. The region doesn't just contain clock and
> reset registers, it also contains pinctrl and interrupt controller
> functionality, so drop the region from the clock binding so that it can
> be described instead by a simple-mfd syscon rather than propagate this
> incorrect description of the hardware to the new pic64gx SoC.
> 
> Signed-off-by: Conor Dooley <conor.dooley@...rochip.com>
> ---
>  .../bindings/clock/microchip,mpfs-clkcfg.yaml | 36 +++++++++++--------
>  1 file changed, 22 insertions(+), 14 deletions(-)
> 

Acked-by: Rob Herring (Arm) <robh@...nel.org>