* [PATCH v4] dt-bindings: opp: Allow multi-worded OPP entry name
@ 2021-10-28 22:15 Dmitry Osipenko
2021-11-02 3:55 ` Viresh Kumar
0 siblings, 1 reply; 2+ messages in thread
From: Dmitry Osipenko @ 2021-10-28 22:15 UTC (permalink / raw)
To: Viresh Kumar, Stephen Boyd, Nishanth Menon, Rob Herring,
David Heidelberg, Thierry Reding, Jonathan Hunter
Cc: linux-kernel, linux-pm, devicetree, linux-tegra
Not all OPP entries fit into a single word. In particular NVIDIA Tegra OPP
tables use multi-word names. Allow OPP entry to have multi-worded name
separated by hyphen. This silences DT checker warnings about wrong naming
scheme.
Reviewed-by: David Heidelberg <david@ixit.cz>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
Changelog:
v4: - Corrected pattern to retain matching of variants like 'opp1'.
Was spotted by Rob Herring.
v3: - Viresh Kumar suggested that we shouldn't support postfix variant.
I dropped variant with postfix naming scheme and multi-wording
support of table names.
v2: - In addition to a multi-wording support, support of the postfix
*-opp-table naming scheme is added to the updated pattern.
Documentation/devicetree/bindings/opp/opp-v2-base.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
index ae3ae4d39843..15a76bcd6d42 100644
--- a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
+++ b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
@@ -33,7 +33,7 @@ properties:
type: boolean
patternProperties:
- '^opp-?[0-9]+$':
+ '^opp(-?[0-9]+)*$':
type: object
description:
One or more OPP nodes describing voltage-current-frequency combinations.
--
2.33.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH v4] dt-bindings: opp: Allow multi-worded OPP entry name
2021-10-28 22:15 [PATCH v4] dt-bindings: opp: Allow multi-worded OPP entry name Dmitry Osipenko
@ 2021-11-02 3:55 ` Viresh Kumar
0 siblings, 0 replies; 2+ messages in thread
From: Viresh Kumar @ 2021-11-02 3:55 UTC (permalink / raw)
To: Dmitry Osipenko
Cc: Viresh Kumar, Stephen Boyd, Nishanth Menon, Rob Herring,
David Heidelberg, Thierry Reding, Jonathan Hunter, linux-kernel,
linux-pm, devicetree, linux-tegra
On 29-10-21, 01:15, Dmitry Osipenko wrote:
> Not all OPP entries fit into a single word. In particular NVIDIA Tegra OPP
> tables use multi-word names. Allow OPP entry to have multi-worded name
> separated by hyphen. This silences DT checker warnings about wrong naming
> scheme.
>
> Reviewed-by: David Heidelberg <david@ixit.cz>
> Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Applied. Thanks.
--
viresh
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-11-02 3:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-28 22:15 [PATCH v4] dt-bindings: opp: Allow multi-worded OPP entry name Dmitry Osipenko
2021-11-02 3:55 ` Viresh Kumar
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).