public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
From: Dario Binacchi <dario.binacchi@amarulasolutions.com>
To: linux-kernel@vger.kernel.org
Cc: Peng Fan <peng.fan@nxp.com>, Stephen Boyd <sboyd@kernel.org>,
	Shawn Guo <shawnguo@kernel.org>,
	linux-amarula@amarulasolutions.com,
	Abel Vesa <abelvesa@kernel.org>,
	Dario Binacchi <dario.binacchi@amarulasolutions.com>,
	Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Fabio Estevam <festevam@gmail.com>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Rob Herring <robh@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	devicetree@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org
Subject: [PATCH v12 16/19] dt-bindings: clock: imx8m-clock: add PLLs
Date: Thu, 24 Apr 2025 08:21:46 +0200	[thread overview]
Message-ID: <20250424062154.2999219-17-dario.binacchi@amarulasolutions.com> (raw)
In-Reply-To: <20250424062154.2999219-1-dario.binacchi@amarulasolutions.com>

Though adding the PLLs to clocks and clock-names properties will break
the ABI, it is required to accurately describe the hardware. Indeed,
the Clock Control Module (CCM) receives clocks from the PLLs and
oscillators and generates clocks for on-chip peripherals.

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

(no changes since v11)

Changes in v11:
- Fix conflict while rebasing on master

Changes in v7:
- Add 'Reviewed-by' tag of Krzysztof Kozlowski

Changes in v6:
- New

 .../bindings/clock/imx8m-clock.yaml           | 27 ++++++++++++++-----
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
index 4fec55832702..e83f08abd44c 100644
--- a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
@@ -29,12 +29,12 @@ properties:
     maxItems: 2
 
   clocks:
-    minItems: 6
-    maxItems: 7
+    minItems: 7
+    maxItems: 10
 
   clock-names:
-    minItems: 6
-    maxItems: 7
+    minItems: 7
+    maxItems: 10
 
   '#clock-cells':
     const: 1
@@ -93,6 +93,10 @@ allOf:
             - description: ext2 clock input
             - description: ext3 clock input
             - description: ext4 clock input
+            - description: audio1 PLL input
+            - description: audio2 PLL input
+            - description: dram PLL input
+            - description: video PLL input
 
         clock-names:
           items:
@@ -102,20 +106,31 @@ allOf:
             - const: clk_ext2
             - const: clk_ext3
             - const: clk_ext4
+            - const: audio_pll1
+            - const: audio_pll2
+            - const: dram_pll
+            - const: video_pll
 
 additionalProperties: false
 
 examples:
   # Clock Control Module node:
   - |
+    #include <dt-bindings/clock/imx8mm-clock.h>
+
     clock-controller@30380000 {
         compatible = "fsl,imx8mm-ccm";
         reg = <0x30380000 0x10000>;
         #clock-cells = <1>;
         clocks = <&osc_32k>, <&osc_24m>, <&clk_ext1>, <&clk_ext2>,
-                 <&clk_ext3>, <&clk_ext4>;
+                 <&clk_ext3>, <&clk_ext4>,
+                 <&anatop IMX8MM_ANATOP_AUDIO_PLL1>,
+                 <&anatop IMX8MM_ANATOP_AUDIO_PLL2>,
+                 <&anatop IMX8MM_ANATOP_DRAM_PLL>,
+                 <&anatop IMX8MM_ANATOP_VIDEO_PLL>;
         clock-names = "osc_32k", "osc_24m", "clk_ext1", "clk_ext2",
-                      "clk_ext3", "clk_ext4";
+                      "clk_ext3", "clk_ext4", "audio_pll1", "audio_pll2",
+                      "dram_pll", "video_pll";
         fsl,operating-mode = "nominal";
     };
 
-- 
2.43.0


  parent reply	other threads:[~2025-04-24  6:22 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-24  6:21 [PATCH v12 00/19] Support spread spectrum clocking for i.MX8M PLLs Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 01/19] dt-bindings: clock: imx8mm: add VIDEO_PLL clocks Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 03/19] dt-bindings: clock: imx8mp: " Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 05/19] dt-bindings: clock: imx8m-anatop: add oscillators and PLLs Dario Binacchi
2025-05-13 13:35   ` Krzysztof Kozlowski
2025-04-24  6:21 ` [PATCH v12 06/19] arm64: dts: imx8mm: add anatop clocks Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 07/19] arm64: dts: imx8mn: " Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 08/19] arm64: dts: imx8mp: " Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 09/19] arm64: dts: imx8mq: " Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 15/19] arm64: dts: imx8mp-aristainetos3a-som-v1: don't replicate clk properties Dario Binacchi
2025-04-24  6:21 ` Dario Binacchi [this message]
2025-05-09 16:25   ` [PATCH v12 16/19] dt-bindings: clock: imx8m-clock: add PLLs Rob Herring
2025-05-09 18:22     ` Dario Binacchi
2025-05-13 13:24       ` Abel Vesa
2025-05-13 13:33         ` Krzysztof Kozlowski
2025-05-13 13:31       ` Krzysztof Kozlowski
2025-05-13 13:35   ` Krzysztof Kozlowski
2025-04-24  6:21 ` [PATCH v12 17/19] arm64: dts: imx8mm: add PLLs to clock controller module (CCM) Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 18/19] arm64: dts: imx8mn: " Dario Binacchi
2025-04-24  6:21 ` [PATCH v12 19/19] arm64: dts: imx8mp: " Dario Binacchi
2025-05-05  7:52 ` (subset) [PATCH v12 00/19] Support spread spectrum clocking for i.MX8M PLLs Abel Vesa
2025-05-08  8:32   ` Dario Binacchi
2025-05-09 12:01     ` Shawn Guo
2025-05-09 12:43       ` Dario Binacchi
2025-05-13 13:34         ` Krzysztof Kozlowski
2025-05-23 15:19   ` Dario Binacchi
2025-05-27 18:42     ` Krzysztof Kozlowski
2025-05-27 19:32       ` Dario Binacchi

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=20250424062154.2999219-17-dario.binacchi@amarulasolutions.com \
    --to=dario.binacchi@amarulasolutions.com \
    --cc=abelvesa@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=imx@lists.linux.dev \
    --cc=kernel@pengutronix.de \
    --cc=krzk+dt@kernel.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-amarula@amarulasolutions.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=peng.fan@nxp.com \
    --cc=robh@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=sboyd@kernel.org \
    --cc=shawnguo@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