linux-kernel - Re: [PATCH 0/2] arm64: zynqmp: Move firmware constants from binding to platform

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: <db3af9b9-37b9-4a29-a480-ddb1df497530@amd.com>
Date: Mon, 17 Feb 2025 16:25:11 +0100
From: Michal Simek <michal.simek@....com>
To: Conor Dooley <conor@...nel.org>
Cc: linux-kernel@...r.kernel.org, monstr@...str.eu, michal.simek@...inx.com,
 git@...inx.com, Anand Ashok Dumbre <anand.ashok.dumbre@...inx.com>,
 Andrew Lunn <andrew+netdev@...n.ch>,
 Claudiu Beznea <claudiu.beznea@...on.dev>, Conor Dooley
 <conor+dt@...nel.org>, Damien Le Moal <dlemoal@...nel.org>,
 "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
 Harini Katakam <harini.katakam@....com>, Jakub Kicinski <kuba@...nel.org>,
 Jonathan Cameron <jic23@...nel.org>, Krzysztof Kozlowski
 <krzk+dt@...nel.org>, Lars-Peter Clausen <lars@...afoo.de>,
 Mark Brown <broonie@...nel.org>, Michael Tretter <m.tretter@...gutronix.de>,
 Michael Turquette <mturquette@...libre.com>,
 Mubin Sayyed <mubin.sayyed@....com>,
 Nicolas Ferre <nicolas.ferre@...rochip.com>,
 Niklas Cassel <cassel@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
 Rob Herring <robh@...nel.org>, Shyam Pandey <radhey.shyam.pandey@....com>,
 Stephen Boyd <sboyd@...nel.org>, Vinod Koul <vkoul@...nel.org>,
 "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
 <devicetree@...r.kernel.org>,
 "open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM"
 <dmaengine@...r.kernel.org>,
 "moderated list:ARM/ZYNQ ARCHITECTURE"
 <linux-arm-kernel@...ts.infradead.org>,
 "open list:COMMON CLK FRAMEWORK" <linux-clk@...r.kernel.org>,
 "open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers)"
 <linux-ide@...r.kernel.org>,
 "open list:XILINX AMS DRIVER" <linux-iio@...r.kernel.org>,
 "open list:SPI SUBSYSTEM" <linux-spi@...r.kernel.org>,
 "open list:USB SUBSYSTEM" <linux-usb@...r.kernel.org>,
 "open list:NETWORKING DRIVERS" <netdev@...r.kernel.org>
Subject: Re: [PATCH 0/2] arm64: zynqmp: Move firmware constants from binding
 to platform

On 2/4/25 20:43, Conor Dooley wrote:
> On Mon, Feb 03, 2025 at 05:39:09PM +0100, Michal Simek wrote:
>> Hi,
>>
>> Based on discussion done long time ago
>> https://lore.kernel.org/all/5353872c-56a3-98f9-7f22-ec1f6c2ccdc8@linaro.org/
>> it is better to deprecate firmware contants which are not used in any code
>> and it's only purpose is to use macros in dt files.
>>
>> There is no reason to be the part of dt bindings but it should be kept
>> there with adding deprecated warning.
>>
>> I want to see if this is the right way to go. If yes, I will also do the
>> same changes in other headers which contain only constans which are not
>> used in any code.
> 
> Ye, I think this is a good approach to take.
> Acked-by: Conor Dooley <conor.dooley@...rochip.com>

Any other comment?

M