From: Conor Dooley <conor@kernel.org>
To: Michal Simek <michal.simek@amd.com>
Cc: linux-kernel@vger.kernel.org, monstr@monstr.eu,
michal.simek@xilinx.com, git@xilinx.com,
Anand Ashok Dumbre <anand.ashok.dumbre@xilinx.com>,
Andrew Lunn <andrew+netdev@lunn.ch>,
Claudiu Beznea <claudiu.beznea@tuxon.dev>,
Conor Dooley <conor+dt@kernel.org>,
Damien Le Moal <dlemoal@kernel.org>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Harini Katakam <harini.katakam@amd.com>,
Jakub Kicinski <kuba@kernel.org>,
Jonathan Cameron <jic23@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Lars-Peter Clausen <lars@metafoo.de>,
Mark Brown <broonie@kernel.org>,
Michael Tretter <m.tretter@pengutronix.de>,
Michael Turquette <mturquette@baylibre.com>,
Mubin Sayyed <mubin.sayyed@amd.com>,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Niklas Cassel <cassel@kernel.org>,
Paolo Abeni <pabeni@redhat.com>, Rob Herring <robh@kernel.org>,
Shyam Pandey <radhey.shyam.pandey@amd.com>,
Stephen Boyd <sboyd@kernel.org>, Vinod Koul <vkoul@kernel.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
"open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM"
<dmaengine@vger.kernel.org>,
"moderated list:ARM/ZYNQ ARCHITECTURE"
<linux-arm-kernel@lists.infradead.org>,
"open list:COMMON CLK FRAMEWORK" <linux-clk@vger.kernel.org>,
"open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers)"
<linux-ide@vger.kernel.org>,
"open list:XILINX AMS DRIVER" <linux-iio@vger.kernel.org>,
"open list:SPI SUBSYSTEM" <linux-spi@vger.kernel.org>,
"open list:USB SUBSYSTEM" <linux-usb@vger.kernel.org>,
"open list:NETWORKING DRIVERS" <netdev@vger.kernel.org>
Subject: Re: [PATCH 0/2] arm64: zynqmp: Move firmware constants from binding to platform
Date: Tue, 4 Feb 2025 19:43:14 +0000 [thread overview]
Message-ID: <20250204-onto-gliding-465a003ae66a@spud> (raw)
In-Reply-To: <cover.1738600745.git.michal.simek@amd.com>
[-- Attachment #1: Type: text/plain, Size: 725 bytes --]
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@microchip.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Conor Dooley <conor@kernel.org>
To: Michal Simek <michal.simek@amd.com>
Cc: "open list:XILINX AMS DRIVER" <linux-iio@vger.kernel.org>,
Michael Turquette <mturquette@baylibre.com>,
linux-kernel@vger.kernel.org, Niklas Cassel <cassel@kernel.org>,
Eric Dumazet <edumazet@google.com>,
Claudiu Beznea <claudiu.beznea@tuxon.dev>,
"open list:COMMON CLK FRAMEWORK" <linux-clk@vger.kernel.org>,
Shyam Pandey <radhey.shyam.pandey@amd.com>,
Michael Tretter <m.tretter@pengutronix.de>,
Rob Herring <robh@kernel.org>,
Lars-Peter Clausen <lars@metafoo.de>,
michal.simek@xilinx.com, Jakub Kicinski <kuba@kernel.org>,
Paolo Abeni <pabeni@redhat.com>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>, Conor Dooley <conor+dt@kernel.org>,
Anand Ashok Dumbre <anand.ashok.dumbre@xilinx.com>,
Mark Brown <broonie@kernel.org>,
git@xilinx.com,
"moderated list:ARM/ZYNQ ARCHITECTURE"
<linux-arm-kernel@lists.infradead.org>,
Harini Katakam <harini.katakam@amd.com>,
monstr@monstr.eu,
"open list:LIBATA SUBSYSTEM \(Serial and Parallel ATA drivers\)"
<linux-ide@vger.kernel.org>, Stephen Boyd <sboyd@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"open list:USB SUBSYSTEM" <linux-usb@vger.kernel.org>,
"open list:SPI SUBSYSTEM" <linux-spi@vger.kernel.org>,
Andrew Lunn <andrew+netdev@lunn.ch>,
Vinod Koul <vkoul@kernel.org>,
Damien Le Moal <dlemoal@kernel.org>,
"open list:NETWORKING DRIVERS" <netdev@vger.kernel.org>,
"open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM"
<dmaengine@vger.kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
"David S. Miller" <davem@davemloft.net>,
Jonathan Cameron <jic23@kernel.org>,
Mubin Sayyed <mubin.sayyed@amd.com>
Subject: Re: [PATCH 0/2] arm64: zynqmp: Move firmware constants from binding to platform
Date: Tue, 4 Feb 2025 19:43:14 +0000 [thread overview]
Message-ID: <20250204-onto-gliding-465a003ae66a@spud> (raw)
In-Reply-To: <cover.1738600745.git.michal.simek@amd.com>
[-- Attachment #1: Type: text/plain, Size: 725 bytes --]
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@microchip.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2025-02-04 19:43 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-03 16:39 [PATCH 0/2] arm64: zynqmp: Move firmware constants from binding to platform Michal Simek
2025-02-03 16:39 ` Michal Simek
2025-02-03 16:39 ` [PATCH 1/2] arm64: zynqmp: Use DT header for firmware constants Michal Simek
2025-02-03 16:39 ` [PATCH 2/2] dt-bindings: xilinx: Deprecate header with " Michal Simek
2025-02-03 16:39 ` Michal Simek
2025-02-04 20:04 ` Jonathan Cameron
2025-02-04 20:04 ` Jonathan Cameron
2025-02-18 10:54 ` Vinod Koul
2025-02-18 10:54 ` Vinod Koul
2025-02-04 19:43 ` Conor Dooley [this message]
2025-02-04 19:43 ` [PATCH 0/2] arm64: zynqmp: Move firmware constants from binding to platform Conor Dooley
2025-02-17 15:25 ` Michal Simek
2025-02-17 15:25 ` Michal Simek
2025-02-20 11:21 ` Michal Simek
2025-02-20 11:21 ` Michal Simek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250204-onto-gliding-465a003ae66a@spud \
--to=conor@kernel.org \
--cc=anand.ashok.dumbre@xilinx.com \
--cc=andrew+netdev@lunn.ch \
--cc=broonie@kernel.org \
--cc=cassel@kernel.org \
--cc=claudiu.beznea@tuxon.dev \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=dlemoal@kernel.org \
--cc=dmaengine@vger.kernel.org \
--cc=edumazet@google.com \
--cc=git@xilinx.com \
--cc=gregkh@linuxfoundation.org \
--cc=harini.katakam@amd.com \
--cc=jic23@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=m.tretter@pengutronix.de \
--cc=michal.simek@amd.com \
--cc=michal.simek@xilinx.com \
--cc=monstr@monstr.eu \
--cc=mturquette@baylibre.com \
--cc=mubin.sayyed@amd.com \
--cc=netdev@vger.kernel.org \
--cc=nicolas.ferre@microchip.com \
--cc=pabeni@redhat.com \
--cc=radhey.shyam.pandey@amd.com \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.