From: Rob Herring <robh@kernel.org>
To: "Miquel Raynal (Schneider Electric)" <miquel.raynal@bootlin.com>
Cc: Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Thomas Gleixner <tglx@kernel.org>,
Olivia Mackall <olivia@selenic.com>,
Herbert Xu <herbert@gondor.apana.org.au>,
Jayesh Choudhary <j-choudhary@ti.com>,
"David S. Miller" <davem@davemloft.net>,
Christian Marangi <ansuelsmth@gmail.com>,
Antoine Tenart <atenart@kernel.org>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Magnus Damm <magnus.damm@gmail.com>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
Pascal EBERHARD <pascal.eberhard@se.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-crypto@vger.kernel.org,
linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH 01/16] dt-bindings: clock: Introduce nexus nodes
Date: Tue, 7 Apr 2026 14:29:00 -0500 [thread overview]
Message-ID: <20260407192900.GA3531350-robh@kernel.org> (raw)
In-Reply-To: <20260327-schneider-v7-0-rc1-crypto-v1-1-5e6ff7853994@bootlin.com>
On Fri, Mar 27, 2026 at 09:09:23PM +0100, Miquel Raynal (Schneider Electric) wrote:
> Hardware containers can just decouple external resources like clock
> without any more control. Nexus nodes already exist for PWM and GPIOs,
> add a binding to allow them for clocks as well.
>
> No examples are given, the file is litteraly a copy-paste from Hervé
> Codina's work on PWM Nexus nodes, hence we just point to the examples
> there which already illustrate very clearly the concept of the various
> properties.
>
> Signed-off-by: Miquel Raynal (Schneider Electric) <miquel.raynal@bootlin.com>
> ---
> .../bindings/clock/clock-nexus-node.yaml | 39 ++++++++++++++++++++++
> 1 file changed, 39 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/clock-nexus-node.yaml b/Documentation/devicetree/bindings/clock/clock-nexus-node.yaml
> new file mode 100644
> index 000000000000..f07e2972e8aa
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/clock-nexus-node.yaml
> @@ -0,0 +1,39 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/clock/clock-nexus-node.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Clock Nexus
> +
> +description: >
> + A nexus node allows to remap a phandle list in a consumer node through a
> + container or a connector node in a generic way. With this remapping,
> + the consumer node needs to know only about the nexus node. Resources
> + behind the nexus node are decoupled by the nexus node itself.
> +
> +maintainers:
> + - Miquel Raynal <miquel.raynal@bootlin.com>
> +
> +select: true
> +
> +properties:
> + '#clock-cells': true
No need for this.
> +
> + clock-map:
> + $ref: /schemas/types.yaml#/definitions/uint32-matrix
> +
> + clock-map-mask:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
> +
> + clock-map-pass-thru:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
I think this can be omitted because there aren't common cell definitions
for clocks like there are for GPIO and PWM.
> +
> +dependentRequired:
> + clock-map: ['#clock-cells']
> + clock-map-mask: [ clock-map ]
> + clock-map-pass-thru: [ clock-map ]
> +
> +additionalProperties: true
> +
> +# See the original pwm-nexus-node.yaml description for examples
>
> --
> 2.51.1
>
next prev parent reply other threads:[~2026-04-07 19:29 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260327-schneider-v7-0-rc1-crypto-v1-0-5e6ff7853994@bootlin.com>
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-12-5e6ff7853994@bootlin.com>
2026-03-28 13:10 ` [PATCH 12/16] irqchip/eip201-aic: Add support for Safexcel EIP-201 AIC Thomas Gleixner
2026-04-01 9:10 ` Miquel Raynal
2026-03-30 13:33 ` [PATCH 00/16] Add support for Inside-Secure EIP-150 crypto block Geert Uytterhoeven
2026-04-01 9:02 ` Miquel Raynal
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-6-5e6ff7853994@bootlin.com>
2026-03-30 14:48 ` [PATCH 06/16] clk: tests: Add clk_parse_clkspec() Kunit testing Brian Masney
2026-04-01 8:59 ` Miquel Raynal
2026-04-01 13:55 ` Brian Masney
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-8-5e6ff7853994@bootlin.com>
2026-03-30 14:50 ` [PATCH 08/16] clk: Improve a couple of comments Brian Masney
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-9-5e6ff7853994@bootlin.com>
2026-03-30 15:01 ` [PATCH 09/16] clk: Use the generic OF phandle parsing in only one place Brian Masney
2026-04-01 8:49 ` Miquel Raynal
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-10-5e6ff7853994@bootlin.com>
2026-03-30 15:09 ` [PATCH 10/16] clk: Add support for clock nexus dt bindings Brian Masney
2026-03-30 15:16 ` Brian Masney
2026-04-01 8:47 ` Miquel Raynal
2026-04-01 14:04 ` Brian Masney
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-1-5e6ff7853994@bootlin.com>
2026-04-07 19:29 ` Rob Herring [this message]
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-16-5e6ff7853994@bootlin.com>
2026-04-07 19:33 ` [PATCH 16/16] ARM: dts: renesas: r9a06g032: Describe the EIP-150 block Rob Herring
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-5-5e6ff7853994@bootlin.com>
2026-04-07 19:44 ` [PATCH 05/16] dt-bindings: bus: eip150: Describe the EIP-150 container node Rob Herring
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-2-5e6ff7853994@bootlin.com>
2026-04-07 19:45 ` [PATCH 02/16] dt-bindings: interrupt-controller: Describe EIP-201 AIC Rob Herring (Arm)
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-3-5e6ff7853994@bootlin.com>
2026-04-07 19:47 ` [PATCH 03/16] dt-bindings: rng: Rename the title of the EIP-76 file Rob Herring (Arm)
[not found] ` <20260327-schneider-v7-0-rc1-crypto-v1-4-5e6ff7853994@bootlin.com>
2026-04-07 19:49 ` [PATCH 04/16] dt-bindings: crypto: eip28: Describe EIP-28 PKA Rob Herring
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=20260407192900.GA3531350-robh@kernel.org \
--to=robh@kernel.org \
--cc=ansuelsmth@gmail.com \
--cc=atenart@kernel.org \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=herbert@gondor.apana.org.au \
--cc=j-choudhary@ti.com \
--cc=krzk+dt@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=miquel.raynal@bootlin.com \
--cc=mturquette@baylibre.com \
--cc=olivia@selenic.com \
--cc=pascal.eberhard@se.com \
--cc=sboyd@kernel.org \
--cc=tglx@kernel.org \
--cc=thomas.petazzoni@bootlin.com \
--cc=wsa+renesas@sang-engineering.com \
/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