linux-kernel - Re: [PATCH v1] dts: arm64: freescale: move imx9*-clock.h imx9*-power.h into dt-bindings

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: <694a3d9e-4096-4782-a918-42d033737fbd@freeshell.de>
Date: Wed, 1 Oct 2025 22:27:26 -0700
From: E Shattow <e@...eshell.de>
To: Peng Fan <peng.fan@....com>, Marek Vasut <marek.vasut@...lbox.org>,
 "Peng Fan (OSS)" <peng.fan@....nxp.com>
Cc: Krzysztof Kozlowski <krzk@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
 Sascha Hauer <s.hauer@...gutronix.de>,
 Pengutronix Kernel Team <kernel@...gutronix.de>,
 Fabio Estevam <festevam@...il.com>, Abel Vesa <abelvesa@...nel.org>,
 Michael Turquette <mturquette@...libre.com>, Stephen Boyd
 <sboyd@...nel.org>, "devicetree@...r.kernel.org"
 <devicetree@...r.kernel.org>, "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
 "linux-arm-kernel@...ts.infradead.org"
 <linux-arm-kernel@...ts.infradead.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>
Subject: Re: [PATCH v1] dts: arm64: freescale: move imx9*-clock.h
 imx9*-power.h into dt-bindings



On 9/10/25 00:07, Peng Fan wrote:
> Hi Marek,
> 
>> Subject: Re: [PATCH v1] dts: arm64: freescale: move imx9*-clock.h
>> imx9*-power.h into dt-bindings
>>
>> On 9/4/25 11:34 AM, Peng Fan wrote:
>>
>> Hi,
>>
>> sorry for my late reply.
>>
>>>> Instead of playing this "I found this code somewhere, so I can do
>>>>> whatever the same" answer the first implied question - why these
>> are
>>>>> bindings? Provide arguments what do they bind.
>>>>
>>>> I am not sure how to answer this, but what I can write is, that if I
>>>> scramble these IDs in either the DT or the firmware (which provides
>>>> the SCMI clock service), then the system cannot work. I am not sure
>>>> if this is the answer you are looking for.
>>>
>>> Marek,
>>>    Some U-Boot code indeed directly use the IDs to configure the clock
>> without
>>>    relying on any drivers. Since the SCMI IDs could not be moved to
>>> dt-bindings,
>>
>> Why can they not be moved to DT bindings ?
> 
> DT maintainers suggested to keep the files under
> arch/arm64/boot/dts/freescale/
> 
> https://lore.kernel.org/all/75ab9162-ed02-479d-92a1-7cfabff6b32e@linaro.org/
> 
> Regards
> Peng

Follow up FYI to all this is resolved from U-Boot [1] as not using the
header and duplicating the information.

1:
https://lore.kernel.org/u-boot/20250923-imx94-v1-5-cb3b65169048@oss.nxp.com/

There was yet some IRC discussion about what justification can be for
avoiding this maintenance and duplication.

I am dropping this suggestion to move imx9*-clock.h imx9*-power.h into
dt-bindings. There is some better way of doing this, maybe a future
dt-firmware includes? That is not at all my area of interest or
expertise, however. I just noticed something out-of-place and bring it
to our attention.

My great appreciation is due for the reviews and discussion about this.
Thanks, everyone!

-E