[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260127-mystify-carmaker-150aa3fcd6c6@spud>
Date: Tue, 27 Jan 2026 19:58:40 +0000
From: Conor Dooley <conor@...nel.org>
To: Anirudh Srinivasan <asrinivasan@....tenstorrent.com>
Cc: Drew Fustini <dfustini@....tenstorrent.com>,
Joel Stanley <jms@....tenstorrent.com>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>,
Philipp Zabel <p.zabel@...gutronix.de>,
linux-riscv@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-clk@...r.kernel.org,
joel@....id.au, fustini@...nel.org, mpe@...nel.org,
mpe@....tenstorrent.com, npiggin@....tenstorrent.com,
agross@...nel.org, agross@....tenstorrent.com, bmasney@...hat.com
Subject: Re: [PATCH v3 1/3] dt-bindings: clk: tenstorrent: Add
tenstorrent,atlantis-prcm
On Mon, Jan 26, 2026 at 03:07:14PM -0600, Anirudh Srinivasan wrote:
> Document bindings for Tenstorrent Atlantis PRCM that manages clocks
> and resets. This block is instantiated 4 times in the SoC.
> This commit documents the clocks from the RCPU PRCM block.
>
> Signed-off-by: Anirudh Srinivasan <asrinivasan@....tenstorrent.com>
> ---
> .../bindings/clock/tenstorrent,atlantis-prcm.yaml | 82 ++++++++++++++++
> MAINTAINERS | 2 +
> .../dt-bindings/clock/tenstorrent,atlantis-prcm.h | 103 +++++++++++++++++++++
> 3 files changed, 187 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/tenstorrent,atlantis-prcm.yaml b/Documentation/devicetree/bindings/clock/tenstorrent,atlantis-prcm.yaml
> new file mode 100644
> index 000000000000..c5716a9928bc
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/tenstorrent,atlantis-prcm.yaml
> @@ -0,0 +1,82 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/clock/tenstorrent,atlantis-prcm.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Tenstorrent Atlantis PRCM (Power, Reset, Clock Management) Module
> +
> +maintainers:
> + - Anirudh Srinivasan <asrinivasan@....tenstorrent.com>
> +
> +description:
> + Multifunctional register block found in Tenstorrent Atlantis SoC whose main
> + function is to control clocks and resets. This block is instantiated multiple
> + times in the SoC, each block controls clock and resets for a different
> + subsystem. RCPU prcm serves low speed IO interfaces. PCIe prcm serves all
> + PCIe related functions, HSIO prcm serves high speed IO interfaces (Ethernet,
> + USB), MM prcm serves GPU, display and video processing functions.
> +
> +properties:
> + compatible:
> + enum:
> + - tenstorrent,atlantis-prcm-rcpu
> + - tenstorrent,atlantis-prcm-pcie
> + - tenstorrent,atlantis-prcm-mm
> + - tenstorrent,atlantis-prcm-hsio
> +
> + reg:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + "#clock-cells":
> + const: 1
> + description:
> + See <dt-bindings/clock/tenstorrent,atlantis-prcm.h> for valid indices.
> +
> + "#reset-cells":
> + const: 1
> +
> + tenstorrent,prcm-rcpu:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description:
> + Phandle reference to RCPU prcm, needed by other 3 prcms (PCIe, MM, HSIO)
> + as the control registers for the PLLs that drive these subsystems are in
> + RCPU prcm's range
This is pretty suspect sounding, if the PLLs for !rcpu are controlled in
the rcpu register region, why is it not a clock parent for the !rcpu
prcms?
> +
> +required:
> + - compatible
> + - reg
> + - clocks
> + - "#clock-cells"
> + - "#reset-cells"
> +
> +allOf:
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - tenstorrent,atlantis-prcm-pcie
> + - tenstorrent,atlantis-prcm-mm
> + - tenstorrent,atlantis-prcm-hsio
> + then:
> + required:
> + - tenstorrent,prcm-rcpu
> + else:
> + properties:
> + tenstorrent,prcm-rcpu: false
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + clock-controller@...00000 {
> + compatible = "tenstorrent,atlantis-prcm-rcpu";
> + reg = <0xa8000000 0x10000>;
> + clocks = <&osc_24m>;
> + #clock-cells = <1>;
> + #reset-cells = <1>;
> + };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index dc731d37c8fe..0fc7bc6d0458 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -22534,8 +22534,10 @@ M: Joel Stanley <jms@....tenstorrent.com>
> L: linux-riscv@...ts.infradead.org
> S: Maintained
> T: git https://github.com/tenstorrent/linux.git
> +F: Documentation/devicetree/bindings/clock/tenstorrent,atlantis-prcm.yaml
> F: Documentation/devicetree/bindings/riscv/tenstorrent.yaml
> F: arch/riscv/boot/dts/tenstorrent/
> +F: include/dt-bindings/clock/tenstorrent,atlantis-prcm.h
>
> RISC-V THEAD SoC SUPPORT
> M: Drew Fustini <fustini@...nel.org>
> diff --git a/include/dt-bindings/clock/tenstorrent,atlantis-prcm.h b/include/dt-bindings/clock/tenstorrent,atlantis-prcm.h
> new file mode 100644
> index 000000000000..3820781127c3
> --- /dev/null
> +++ b/include/dt-bindings/clock/tenstorrent,atlantis-prcm.h
> @@ -0,0 +1,103 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/*
> + * Tenstorrent Atlantis PRCM Clock and Reset Indices
> + *
> + * Copyright (c) 2026 Tenstorrent
> + */
> +
> +#ifndef _DT_BINDINGS_ATLANTIS_PRCM_H
> +#define _DT_BINDINGS_ATLANTIS_PRCM_H
> +
> +/*
> + * RCPU Domain Clock IDs
> + */
> +#define CLK_RCPU_PLL 0
> +#define CLK_RCPU_ROOT 1
> +#define CLK_RCPU_DIV2 2
> +#define CLK_RCPU_DIV4 3
> +#define CLK_RCPU_RTC 4
> +#define CLK_SMNDMA0_ACLK 5
> +#define CLK_SMNDMA1_ACLK 6
> +#define CLK_WDT0_PCLK 7
> +#define CLK_WDT1_PCLK 8
> +#define CLK_TIMER_PCLK 9
> +#define CLK_PVTC_PCLK 10
> +#define CLK_PMU_PCLK 11
> +#define CLK_MAILBOX_HCLK 12
> +#define CLK_SEC_SPACC_HCLK 13
> +#define CLK_SEC_OTP_HCLK 14
> +#define CLK_TRNG_PCLK 15
> +#define CLK_SEC_CRC_HCLK 16
> +#define CLK_SMN_HCLK 17
> +#define CLK_AHB0_HCLK 18
> +#define CLK_SMN_PCLK 19
> +#define CLK_SMN_CLK 20
> +#define CLK_SCRATCHPAD_CLK 21
> +#define CLK_RCPU_CORE_CLK 22
> +#define CLK_RCPU_ROM_CLK 23
> +#define CLK_OTP_LOAD_CLK 24
> +#define CLK_NOC_PLL 25
> +#define CLK_NOCC_CLK 26
> +#define CLK_NOCC_DIV2 27
> +#define CLK_NOCC_DIV4 28
> +#define CLK_NOCC_RTC 29
> +#define CLK_NOCC_CAN 30
> +#define CLK_QSPI_SCLK 31
> +#define CLK_QSPI_HCLK 32
> +#define CLK_I2C0_PCLK 33
> +#define CLK_I2C1_PCLK 34
> +#define CLK_I2C2_PCLK 35
> +#define CLK_I2C3_PCLK 36
> +#define CLK_I2C4_PCLK 37
> +#define CLK_UART0_PCLK 38
> +#define CLK_UART1_PCLK 39
> +#define CLK_UART2_PCLK 40
> +#define CLK_UART3_PCLK 41
> +#define CLK_UART4_PCLK 42
> +#define CLK_SPI0_PCLK 43
> +#define CLK_SPI1_PCLK 44
> +#define CLK_SPI2_PCLK 45
> +#define CLK_SPI3_PCLK 46
> +#define CLK_GPIO_PCLK 47
> +#define CLK_CAN0_HCLK 48
> +#define CLK_CAN0_CLK 49
> +#define CLK_CAN1_HCLK 50
> +#define CLK_CAN1_CLK 51
> +#define CLK_CAN0_TIMER_CLK 52
> +#define CLK_CAN1_TIMER_CLK 53
> +
> +/* RCPU domain reset */
> +#define RST_SMNDMA0 0
> +#define RST_SMNDMA1 1
> +#define RST_WDT0 2
> +#define RST_WDT1 3
> +#define RST_TMR 4
> +#define RST_PVTC 5
> +#define RST_PMU 6
> +#define RST_MAILBOX 7
> +#define RST_SPACC 8
> +#define RST_OTP 9
> +#define RST_TRNG 10
> +#define RST_CRC 11
> +#define RST_QSPI 12
> +#define RST_I2C0 13
> +#define RST_I2C1 14
> +#define RST_I2C2 15
> +#define RST_I2C3 16
> +#define RST_I2C4 17
> +#define RST_UART0 18
> +#define RST_UART1 19
> +#define RST_UART2 20
> +#define RST_UART3 21
> +#define RST_UART4 22
> +#define RST_SPI0 23
> +#define RST_SPI1 24
> +#define RST_SPI2 25
> +#define RST_SPI3 26
> +#define RST_GPIO 27
> +#define RST_CAN0 28
> +#define RST_CAN1 29
> +#define RST_I2S0 30
> +#define RST_I2S1 31
> +
> +#endif /* _DT_BINDINGS_ATLANTIS_PRCM_H */
>
> --
> 2.43.0
>
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists