From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Rob Herring" <robh@kernel.org>,
"Saravana Kannan" <saravanak@google.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
"Viresh Kumar" <viresh.kumar@linaro.org>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"Yangtao Li" <tiny.windzz@gmail.com>,
"Chen-Yu Tsai" <wens@kernel.org>,
"Jernej Skrabec" <jernej.skrabec@gmail.com>,
"Samuel Holland" <samuel@sholland.org>,
"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
"Daniel Lezcano" <daniel.lezcano@linaro.org>,
"Bjorn Andersson" <andersson@kernel.org>,
"Konrad Dybcio" <konradybcio@kernel.org>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Nicolas Ferre" <nicolas.ferre@microchip.com>,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
"Claudiu Beznea" <claudiu.beznea@tuxon.dev>,
"Maximilian Luz" <luzmaximilian@gmail.com>,
"Hans de Goede" <hansg@kernel.org>,
"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
"Daniel Lezcano" <daniel.lezcano@kernel.org>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Jonathan Hunter" <jonathanh@nvidia.com>
Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org, linux-sunxi@lists.linux.dev,
linux-arm-msm@vger.kernel.org,
platform-driver-x86@vger.kernel.org,
linux-tegra@vger.kernel.org,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>,
Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Subject: [PATCH v2 00/11] of: Add wrappers to match root node with OF device ID tables
Date: Wed, 12 Nov 2025 11:28:45 +0100 [thread overview]
Message-ID: <20251112-b4-of-match-matchine-data-v2-0-d46b72003fd6@linaro.org> (raw)
Changes in v2:
- Drop cpufreq/ti change: not correct.
- Drop soc/qcom/qcom_pd_mapper.c - objections from Dmitry and I think
better to drop the patch in such case.
- I did not implement feedback for first patch to make the
of_machine_compatible_match() matching machines in arbitrary nodes,
because there is no such use case possible and no arguments were provided.
I also did not use cleanup.h in first patch because existing code
of_device_get_match_data() does not use it and I prefer uniformity.
- Add Ack/Rb tags.
- Link to v1: https://patch.msgid.link/20251106-b4-of-match-matchine-data-v1-0-d780ea1780c2@linaro.org
Dependency/merging
==================
All patches depend on the first patch, thus everything could go via
Rob's tree with people's acks.
Description
===========
Several drivers duplicate same code for getting reference to the root
node, matching it against 'struct of_device_id' table and getting out
the match data from the table entry.
There is a of_machine_compatible_match() wrapper but it takes array of
strings, which is not suitable for many drivers since they want the
driver data associated with each compatible.
Add two wrappers, similar to existing of_device_get_match_data():
1. of_machine_device_match() doing only matching against 'struct
of_device_id' and returning bool.
2. of_machine_get_match_data() doing the matching and returning
associated driver data for found compatible.
Best regards,
Krzysztof
---
Krzysztof Kozlowski (11):
of: Add wrappers to match root node with OF device ID tables
cpufreq: dt-platdev: Simplify with of_machine_get_match_data()
cpufreq: mediatek: Simplify with of_machine_get_match_data()
cpufreq: sun50i: Simplify with of_machine_device_match()
cpuidle: big_little: Simplify with of_machine_device_match()
firmware: qcom: scm: Simplify with of_machine_device_match()
irqchip/atmel-aic: Simplify with of_machine_get_match_data()
platform: surface: Simplify with of_machine_get_match_data()
powercap: dtpm: Simplify with of_machine_get_match_data()
soc: qcom: ubwc: Simplify with of_machine_get_match_data()
soc: tegra: Simplify with of_machine_device_match()
drivers/cpufreq/cpufreq-dt-platdev.c | 15 ++-----
drivers/cpufreq/mediatek-cpufreq.c | 12 +-----
drivers/cpufreq/sun50i-cpufreq-nvmem.c | 11 +----
drivers/cpuidle/cpuidle-big_little.c | 11 +----
drivers/firmware/qcom/qcom_scm.c | 17 +-------
drivers/irqchip/irq-atmel-aic-common.c | 15 ++-----
drivers/of/base.c | 47 ++++++++++++++++++++++
.../platform/surface/surface_aggregator_registry.c | 13 +-----
drivers/powercap/dtpm.c | 16 +-------
drivers/soc/qcom/ubwc_config.c | 14 ++-----
drivers/soc/tegra/common.c | 12 +-----
include/linux/of.h | 13 ++++++
12 files changed, 79 insertions(+), 117 deletions(-)
---
base-commit: a4ebba34e722123f1c09ce3282e26f052fc8b27f
change-id: 20251106-b4-of-match-matchine-data-4a64bf046814
Best regards,
--
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
next reply other threads:[~2025-11-12 10:29 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-12 10:28 Krzysztof Kozlowski [this message]
2025-11-12 10:28 ` [PATCH v2 01/11] of: Add wrappers to match root node with OF device ID tables Krzysztof Kozlowski
2025-11-13 9:55 ` Lukasz Luba
2025-11-12 10:28 ` [PATCH v2 02/11] cpufreq: dt-platdev: Simplify with of_machine_get_match_data() Krzysztof Kozlowski
2025-11-12 11:44 ` AngeloGioacchino Del Regno
2025-11-12 10:28 ` [PATCH v2 03/11] cpufreq: mediatek: " Krzysztof Kozlowski
2025-11-12 11:44 ` AngeloGioacchino Del Regno
2025-11-12 10:28 ` [PATCH v2 04/11] cpufreq: sun50i: Simplify with of_machine_device_match() Krzysztof Kozlowski
2025-11-12 10:28 ` [PATCH v2 05/11] cpuidle: big_little: " Krzysztof Kozlowski
2025-11-12 11:44 ` AngeloGioacchino Del Regno
2025-11-12 10:28 ` [PATCH v2 06/11] firmware: qcom: scm: " Krzysztof Kozlowski
2025-11-12 11:44 ` AngeloGioacchino Del Regno
2025-11-12 10:28 ` [PATCH v2 07/11] irqchip/atmel-aic: Simplify with of_machine_get_match_data() Krzysztof Kozlowski
2025-11-12 13:06 ` Alexandre Belloni
2025-11-12 10:28 ` [PATCH v2 08/11] platform: surface: " Krzysztof Kozlowski
2025-11-12 10:28 ` [PATCH v2 09/11] powercap: dtpm: " Krzysztof Kozlowski
2025-11-13 9:50 ` Lukasz Luba
2025-11-12 10:28 ` [PATCH v2 10/11] soc: qcom: ubwc: " Krzysztof Kozlowski
2025-11-12 10:28 ` [PATCH v2 11/11] soc: tegra: Simplify with of_machine_device_match() Krzysztof Kozlowski
2025-11-12 11:44 ` AngeloGioacchino Del Regno
2025-11-14 13:17 ` Thierry Reding
2025-11-12 11:52 ` [PATCH v2 00/11] of: Add wrappers to match root node with OF device ID tables AngeloGioacchino Del Regno
2025-11-14 13:24 ` Thierry Reding
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=20251112-b4-of-match-matchine-data-v2-0-d46b72003fd6@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=alexandre.belloni@bootlin.com \
--cc=andersson@kernel.org \
--cc=angelogioacchino.delregno@collabora.com \
--cc=claudiu.beznea@tuxon.dev \
--cc=daniel.lezcano@kernel.org \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.baryshkov@oss.qualcomm.com \
--cc=hansg@kernel.org \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=jernej.skrabec@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=konrad.dybcio@oss.qualcomm.com \
--cc=konradybcio@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-sunxi@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=luzmaximilian@gmail.com \
--cc=matthias.bgg@gmail.com \
--cc=nicolas.ferre@microchip.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=samuel@sholland.org \
--cc=saravanak@google.com \
--cc=tglx@linutronix.de \
--cc=thierry.reding@gmail.com \
--cc=tiny.windzz@gmail.com \
--cc=viresh.kumar@linaro.org \
--cc=wens@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).