From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: gabriel.fernandez@foss.st.com,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Alexandre Torgue <alexandre.torgue@st.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
Etienne Carriere <etienne.carriere@st.com>,
marex@denx.de
Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-stm32@st-md-mailman.stormreply.com
Subject: Re: [Linux-stm32] [PATCH v2 12/14] ARM: dts: stm32: move clocks/resets to SCMI resources for stm32mp15
Date: Thu, 18 Feb 2021 10:45:47 +0100 [thread overview]
Message-ID: <c74e8909-84a3-f6dc-50fb-18349fd79ece@pengutronix.de> (raw)
In-Reply-To: <20210126090120.19900-13-gabriel.fernandez@foss.st.com>
Hello Gabriel,
On 26.01.21 10:01, gabriel.fernandez@foss.st.com wrote:
> From: Gabriel Fernandez <gabriel.fernandez@foss.st.com>
>
> This change reflects board hardware configuration where RCC security
> features are configured for RCC[TZEN]=1 and RCC[MCKPROT]=0, that is
> RCC TrustZone is hardened and RCC MCKPROT is disabled.
>
> Clock and reset controllers that relate to SoC secure resources are
> moved from a RCC clock/reset handle to a SCMI clock/reset_domain handle.
>
> These clocks are all the platform oscillators (HSI/LSI/CSI/HSE/LSE),
> clocks for few subsystem and peripheral interfaces.
>
> This change add a SCMI clock dependency on RCC clock device since it
> registers clocks which parent clocks are provided by the SCMI clock
> driver. This change allows the RCC clock device probe to be deferred
> until SCMI clocks are fully registered in the system.
This would break existing boards, right? If so, you should move the
last patch in the series before this one and patch all existing boards
to use it. Maintainers of individual boards can then opt-in later by
deleting the arch/arm/boot/dts/stm32mp15-no-scmi.dtsi inclusion.
Cheers,
Ahmad
>
> Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@foss.st.com>
> ---
> arch/arm/boot/dts/stm32mp151.dtsi | 77 +++++++++++-------------------
> arch/arm/boot/dts/stm32mp153.dtsi | 4 +-
> arch/arm/boot/dts/stm32mp157.dtsi | 2 +-
> arch/arm/boot/dts/stm32mp15xc.dtsi | 4 +-
> 4 files changed, 32 insertions(+), 55 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
> index da3647373365..e06882e0611d 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -102,38 +102,6 @@
> interrupt-parent = <&intc>;
> };
>
> - clocks {
> - clk_hse: clk-hse {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <24000000>;
> - };
> -
> - clk_hsi: clk-hsi {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <64000000>;
> - };
> -
> - clk_lse: clk-lse {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <32768>;
> - };
> -
> - clk_lsi: clk-lsi {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <32000>;
> - };
> -
> - clk_csi: clk-csi {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <4000000>;
> - };
> - };
> -
> thermal-zones {
> cpu_thermal: cpu-thermal {
> polling-delay-passive = <0>;
> @@ -595,7 +563,7 @@
> compatible = "st,stm32-cec";
> reg = <0x40016000 0x400>;
> interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc CEC_K>, <&clk_lse>;
> + clocks = <&rcc CEC_K>, <&scmi0_clk CK_SCMI0_LSE>;
> clock-names = "cec", "hdmi-cec";
> status = "disabled";
> };
> @@ -1156,10 +1124,17 @@
> };
>
> rcc: rcc@50000000 {
> - compatible = "st,stm32mp1-rcc", "syscon";
> + compatible = "st,stm32mp1-rcc-secure", "st,stm32mp1-rcc", "syscon";
> reg = <0x50000000 0x1000>;
> #clock-cells = <1>;
> #reset-cells = <1>;
> +
> + clock-names = "hse", "hsi", "csi", "lse", "lsi";
> + clocks = <&scmi0_clk CK_SCMI0_HSE>,
> + <&scmi0_clk CK_SCMI0_HSI>,
> + <&scmi0_clk CK_SCMI0_CSI>,
> + <&scmi0_clk CK_SCMI0_LSE>,
> + <&scmi0_clk CK_SCMI0_LSI>;
> };
>
> pwr_regulators: pwr@50001000 {
> @@ -1342,8 +1317,8 @@
> compatible = "st,stm32f756-hash";
> reg = <0x54002000 0x400>;
> interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc HASH1>;
> - resets = <&rcc HASH1_R>;
> + clocks = <&scmi0_clk CK_SCMI0_HASH1>;
> + resets = <&scmi0_reset RST_SCMI0_HASH1>;
> dmas = <&mdma1 31 0x2 0x1000A02 0x0 0x0>;
> dma-names = "in";
> dma-maxburst = <2>;
> @@ -1353,8 +1328,8 @@
> rng1: rng@54003000 {
> compatible = "st,stm32-rng";
> reg = <0x54003000 0x400>;
> - clocks = <&rcc RNG1_K>;
> - resets = <&rcc RNG1_R>;
> + clocks = <&scmi0_clk CK_SCMI0_RNG1>;
> + resets = <&scmi0_reset RST_SCMI0_RNG1>;
> status = "disabled";
> };
>
> @@ -1363,7 +1338,7 @@
> reg = <0x58000000 0x1000>;
> interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&rcc MDMA>;
> - resets = <&rcc MDMA_R>;
> + resets = <&scmi0_reset RST_SCMI0_MDMA>;
> #dma-cells = <5>;
> dma-channels = <32>;
> dma-requests = <48>;
> @@ -1524,7 +1499,7 @@
> iwdg2: watchdog@5a002000 {
> compatible = "st,stm32mp1-iwdg";
> reg = <0x5a002000 0x400>;
> - clocks = <&rcc IWDG2>, <&rcc CK_LSI>;
> + clocks = <&rcc IWDG2>, <&scmi0_clk CK_SCMI0_LSI>;
> clock-names = "pclk", "lsi";
> status = "disabled";
> };
> @@ -1553,7 +1528,8 @@
> compatible = "st,stm32h7-uart";
> reg = <0x5c000000 0x400>;
> interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc USART1_K>;
> + clocks = <&scmi0_clk CK_SCMI0_USART1>;
> + resets = <&scmi0_reset RST_SCMI0_USART1>;
> status = "disabled";
> };
>
> @@ -1563,8 +1539,8 @@
> compatible = "st,stm32h7-spi";
> reg = <0x5c001000 0x400>;
> interrupts = <GIC_SPI 86 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc SPI6_K>;
> - resets = <&rcc SPI6_R>;
> + clocks = <&scmi0_clk CK_SCMI0_SPI6>;
> + resets = <&scmi0_reset RST_SCMI0_SPI6>;
> dmas = <&mdma1 34 0x0 0x40008 0x0 0x0>,
> <&mdma1 35 0x0 0x40002 0x0 0x0>;
> dma-names = "rx", "tx";
> @@ -1577,8 +1553,8 @@
> interrupt-names = "event", "error";
> interrupts = <GIC_SPI 95 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc I2C4_K>;
> - resets = <&rcc I2C4_R>;
> + clocks = <&scmi0_clk CK_SCMI0_I2C4>;
> + resets = <&scmi0_reset RST_SCMI0_I2C4>;
> #address-cells = <1>;
> #size-cells = <0>;
> st,syscfg-fmp = <&syscfg 0x4 0x8>;
> @@ -1589,7 +1565,8 @@
> rtc: rtc@5c004000 {
> compatible = "st,stm32mp1-rtc";
> reg = <0x5c004000 0x400>;
> - clocks = <&rcc RTCAPB>, <&rcc RTC>;
> + clocks = <&scmi0_clk CK_SCMI0_RTCAPB>,
> + <&scmi0_clk CK_SCMI0_RTC>;
> clock-names = "pclk", "rtc_ck";
> interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
> status = "disabled";
> @@ -1614,8 +1591,8 @@
> interrupt-names = "event", "error";
> interrupts = <GIC_SPI 135 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 136 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc I2C6_K>;
> - resets = <&rcc I2C6_R>;
> + clocks = <&scmi0_clk CK_SCMI0_I2C6>;
> + resets = <&scmi0_reset RST_SCMI0_I2C6>;
> #address-cells = <1>;
> #size-cells = <0>;
> st,syscfg-fmp = <&syscfg 0x4 0x20>;
> @@ -1778,7 +1755,7 @@
> interrupt-controller;
> #interrupt-cells = <2>;
> reg = <0 0x400>;
> - clocks = <&rcc GPIOZ>;
> + clocks = <&scmi0_clk CK_SCMI0_GPIOZ>;
> st,bank-name = "GPIOZ";
> st,bank-ioport = <11>;
> status = "disabled";
> @@ -1800,7 +1777,7 @@
> reg = <0x10000000 0x40000>,
> <0x30000000 0x40000>,
> <0x38000000 0x10000>;
> - resets = <&rcc MCU_R>;
> + resets = <&scmi0_reset RST_SCMI0_MCU>;
> st,syscfg-holdboot = <&rcc 0x10C 0x1>;
> st,syscfg-tz = <&rcc 0x000 0x1>;
> st,syscfg-pdds = <&pwr_mcu 0x0 0x1>;
> diff --git a/arch/arm/boot/dts/stm32mp153.dtsi b/arch/arm/boot/dts/stm32mp153.dtsi
> index 1c1889b194cf..db1273854675 100644
> --- a/arch/arm/boot/dts/stm32mp153.dtsi
> +++ b/arch/arm/boot/dts/stm32mp153.dtsi
> @@ -30,7 +30,7 @@
> interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 21 IRQ_TYPE_LEVEL_HIGH>;
> interrupt-names = "int0", "int1";
> - clocks = <&rcc CK_HSE>, <&rcc FDCAN_K>;
> + clocks = <&scmi0_clk CK_SCMI0_HSE>, <&rcc FDCAN_K>;
> clock-names = "hclk", "cclk";
> bosch,mram-cfg = <0x0 0 0 32 0 0 2 2>;
> status = "disabled";
> @@ -43,7 +43,7 @@
> interrupts = <GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
> interrupt-names = "int0", "int1";
> - clocks = <&rcc CK_HSE>, <&rcc FDCAN_K>;
> + clocks = <&scmi0_clk CK_SCMI0_HSE>, <&rcc FDCAN_K>;
> clock-names = "hclk", "cclk";
> bosch,mram-cfg = <0x1400 0 0 32 0 0 2 2>;
> status = "disabled";
> diff --git a/arch/arm/boot/dts/stm32mp157.dtsi b/arch/arm/boot/dts/stm32mp157.dtsi
> index 54e73ccea446..7b06c08e3a23 100644
> --- a/arch/arm/boot/dts/stm32mp157.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157.dtsi
> @@ -20,7 +20,7 @@
> dsi: dsi@5a000000 {
> compatible = "st,stm32-dsi";
> reg = <0x5a000000 0x800>;
> - clocks = <&rcc DSI_K>, <&clk_hse>, <&rcc DSI_PX>;
> + clocks = <&rcc DSI_K>, <&scmi0_clk CK_SCMI0_HSE>, <&rcc DSI_PX>;
> clock-names = "pclk", "ref", "px_clk";
> resets = <&rcc DSI_R>;
> reset-names = "apb";
> diff --git a/arch/arm/boot/dts/stm32mp15xc.dtsi b/arch/arm/boot/dts/stm32mp15xc.dtsi
> index b06a55a2fa18..435846883f25 100644
> --- a/arch/arm/boot/dts/stm32mp15xc.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xc.dtsi
> @@ -10,8 +10,8 @@
> compatible = "st,stm32mp1-cryp";
> reg = <0x54001000 0x400>;
> interrupts = <GIC_SPI 79 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc CRYP1>;
> - resets = <&rcc CRYP1_R>;
> + clocks = <&scmi0_clk CK_SCMI0_CRYP1>;
> + resets = <&scmi0_reset RST_SCMI0_CRYP1>;
> status = "disabled";
> };
> };
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
WARNING: multiple messages have this Message-ID (diff)
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: gabriel.fernandez@foss.st.com,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Alexandre Torgue <alexandre.torgue@st.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
Etienne Carriere <etienne.carriere@st.com>,
marex@denx.de
Cc: devicetree@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org
Subject: Re: [Linux-stm32] [PATCH v2 12/14] ARM: dts: stm32: move clocks/resets to SCMI resources for stm32mp15
Date: Thu, 18 Feb 2021 10:45:47 +0100 [thread overview]
Message-ID: <c74e8909-84a3-f6dc-50fb-18349fd79ece@pengutronix.de> (raw)
In-Reply-To: <20210126090120.19900-13-gabriel.fernandez@foss.st.com>
Hello Gabriel,
On 26.01.21 10:01, gabriel.fernandez@foss.st.com wrote:
> From: Gabriel Fernandez <gabriel.fernandez@foss.st.com>
>
> This change reflects board hardware configuration where RCC security
> features are configured for RCC[TZEN]=1 and RCC[MCKPROT]=0, that is
> RCC TrustZone is hardened and RCC MCKPROT is disabled.
>
> Clock and reset controllers that relate to SoC secure resources are
> moved from a RCC clock/reset handle to a SCMI clock/reset_domain handle.
>
> These clocks are all the platform oscillators (HSI/LSI/CSI/HSE/LSE),
> clocks for few subsystem and peripheral interfaces.
>
> This change add a SCMI clock dependency on RCC clock device since it
> registers clocks which parent clocks are provided by the SCMI clock
> driver. This change allows the RCC clock device probe to be deferred
> until SCMI clocks are fully registered in the system.
This would break existing boards, right? If so, you should move the
last patch in the series before this one and patch all existing boards
to use it. Maintainers of individual boards can then opt-in later by
deleting the arch/arm/boot/dts/stm32mp15-no-scmi.dtsi inclusion.
Cheers,
Ahmad
>
> Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@foss.st.com>
> ---
> arch/arm/boot/dts/stm32mp151.dtsi | 77 +++++++++++-------------------
> arch/arm/boot/dts/stm32mp153.dtsi | 4 +-
> arch/arm/boot/dts/stm32mp157.dtsi | 2 +-
> arch/arm/boot/dts/stm32mp15xc.dtsi | 4 +-
> 4 files changed, 32 insertions(+), 55 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
> index da3647373365..e06882e0611d 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -102,38 +102,6 @@
> interrupt-parent = <&intc>;
> };
>
> - clocks {
> - clk_hse: clk-hse {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <24000000>;
> - };
> -
> - clk_hsi: clk-hsi {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <64000000>;
> - };
> -
> - clk_lse: clk-lse {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <32768>;
> - };
> -
> - clk_lsi: clk-lsi {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <32000>;
> - };
> -
> - clk_csi: clk-csi {
> - #clock-cells = <0>;
> - compatible = "fixed-clock";
> - clock-frequency = <4000000>;
> - };
> - };
> -
> thermal-zones {
> cpu_thermal: cpu-thermal {
> polling-delay-passive = <0>;
> @@ -595,7 +563,7 @@
> compatible = "st,stm32-cec";
> reg = <0x40016000 0x400>;
> interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc CEC_K>, <&clk_lse>;
> + clocks = <&rcc CEC_K>, <&scmi0_clk CK_SCMI0_LSE>;
> clock-names = "cec", "hdmi-cec";
> status = "disabled";
> };
> @@ -1156,10 +1124,17 @@
> };
>
> rcc: rcc@50000000 {
> - compatible = "st,stm32mp1-rcc", "syscon";
> + compatible = "st,stm32mp1-rcc-secure", "st,stm32mp1-rcc", "syscon";
> reg = <0x50000000 0x1000>;
> #clock-cells = <1>;
> #reset-cells = <1>;
> +
> + clock-names = "hse", "hsi", "csi", "lse", "lsi";
> + clocks = <&scmi0_clk CK_SCMI0_HSE>,
> + <&scmi0_clk CK_SCMI0_HSI>,
> + <&scmi0_clk CK_SCMI0_CSI>,
> + <&scmi0_clk CK_SCMI0_LSE>,
> + <&scmi0_clk CK_SCMI0_LSI>;
> };
>
> pwr_regulators: pwr@50001000 {
> @@ -1342,8 +1317,8 @@
> compatible = "st,stm32f756-hash";
> reg = <0x54002000 0x400>;
> interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc HASH1>;
> - resets = <&rcc HASH1_R>;
> + clocks = <&scmi0_clk CK_SCMI0_HASH1>;
> + resets = <&scmi0_reset RST_SCMI0_HASH1>;
> dmas = <&mdma1 31 0x2 0x1000A02 0x0 0x0>;
> dma-names = "in";
> dma-maxburst = <2>;
> @@ -1353,8 +1328,8 @@
> rng1: rng@54003000 {
> compatible = "st,stm32-rng";
> reg = <0x54003000 0x400>;
> - clocks = <&rcc RNG1_K>;
> - resets = <&rcc RNG1_R>;
> + clocks = <&scmi0_clk CK_SCMI0_RNG1>;
> + resets = <&scmi0_reset RST_SCMI0_RNG1>;
> status = "disabled";
> };
>
> @@ -1363,7 +1338,7 @@
> reg = <0x58000000 0x1000>;
> interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&rcc MDMA>;
> - resets = <&rcc MDMA_R>;
> + resets = <&scmi0_reset RST_SCMI0_MDMA>;
> #dma-cells = <5>;
> dma-channels = <32>;
> dma-requests = <48>;
> @@ -1524,7 +1499,7 @@
> iwdg2: watchdog@5a002000 {
> compatible = "st,stm32mp1-iwdg";
> reg = <0x5a002000 0x400>;
> - clocks = <&rcc IWDG2>, <&rcc CK_LSI>;
> + clocks = <&rcc IWDG2>, <&scmi0_clk CK_SCMI0_LSI>;
> clock-names = "pclk", "lsi";
> status = "disabled";
> };
> @@ -1553,7 +1528,8 @@
> compatible = "st,stm32h7-uart";
> reg = <0x5c000000 0x400>;
> interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc USART1_K>;
> + clocks = <&scmi0_clk CK_SCMI0_USART1>;
> + resets = <&scmi0_reset RST_SCMI0_USART1>;
> status = "disabled";
> };
>
> @@ -1563,8 +1539,8 @@
> compatible = "st,stm32h7-spi";
> reg = <0x5c001000 0x400>;
> interrupts = <GIC_SPI 86 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc SPI6_K>;
> - resets = <&rcc SPI6_R>;
> + clocks = <&scmi0_clk CK_SCMI0_SPI6>;
> + resets = <&scmi0_reset RST_SCMI0_SPI6>;
> dmas = <&mdma1 34 0x0 0x40008 0x0 0x0>,
> <&mdma1 35 0x0 0x40002 0x0 0x0>;
> dma-names = "rx", "tx";
> @@ -1577,8 +1553,8 @@
> interrupt-names = "event", "error";
> interrupts = <GIC_SPI 95 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc I2C4_K>;
> - resets = <&rcc I2C4_R>;
> + clocks = <&scmi0_clk CK_SCMI0_I2C4>;
> + resets = <&scmi0_reset RST_SCMI0_I2C4>;
> #address-cells = <1>;
> #size-cells = <0>;
> st,syscfg-fmp = <&syscfg 0x4 0x8>;
> @@ -1589,7 +1565,8 @@
> rtc: rtc@5c004000 {
> compatible = "st,stm32mp1-rtc";
> reg = <0x5c004000 0x400>;
> - clocks = <&rcc RTCAPB>, <&rcc RTC>;
> + clocks = <&scmi0_clk CK_SCMI0_RTCAPB>,
> + <&scmi0_clk CK_SCMI0_RTC>;
> clock-names = "pclk", "rtc_ck";
> interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
> status = "disabled";
> @@ -1614,8 +1591,8 @@
> interrupt-names = "event", "error";
> interrupts = <GIC_SPI 135 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 136 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc I2C6_K>;
> - resets = <&rcc I2C6_R>;
> + clocks = <&scmi0_clk CK_SCMI0_I2C6>;
> + resets = <&scmi0_reset RST_SCMI0_I2C6>;
> #address-cells = <1>;
> #size-cells = <0>;
> st,syscfg-fmp = <&syscfg 0x4 0x20>;
> @@ -1778,7 +1755,7 @@
> interrupt-controller;
> #interrupt-cells = <2>;
> reg = <0 0x400>;
> - clocks = <&rcc GPIOZ>;
> + clocks = <&scmi0_clk CK_SCMI0_GPIOZ>;
> st,bank-name = "GPIOZ";
> st,bank-ioport = <11>;
> status = "disabled";
> @@ -1800,7 +1777,7 @@
> reg = <0x10000000 0x40000>,
> <0x30000000 0x40000>,
> <0x38000000 0x10000>;
> - resets = <&rcc MCU_R>;
> + resets = <&scmi0_reset RST_SCMI0_MCU>;
> st,syscfg-holdboot = <&rcc 0x10C 0x1>;
> st,syscfg-tz = <&rcc 0x000 0x1>;
> st,syscfg-pdds = <&pwr_mcu 0x0 0x1>;
> diff --git a/arch/arm/boot/dts/stm32mp153.dtsi b/arch/arm/boot/dts/stm32mp153.dtsi
> index 1c1889b194cf..db1273854675 100644
> --- a/arch/arm/boot/dts/stm32mp153.dtsi
> +++ b/arch/arm/boot/dts/stm32mp153.dtsi
> @@ -30,7 +30,7 @@
> interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 21 IRQ_TYPE_LEVEL_HIGH>;
> interrupt-names = "int0", "int1";
> - clocks = <&rcc CK_HSE>, <&rcc FDCAN_K>;
> + clocks = <&scmi0_clk CK_SCMI0_HSE>, <&rcc FDCAN_K>;
> clock-names = "hclk", "cclk";
> bosch,mram-cfg = <0x0 0 0 32 0 0 2 2>;
> status = "disabled";
> @@ -43,7 +43,7 @@
> interrupts = <GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
> interrupt-names = "int0", "int1";
> - clocks = <&rcc CK_HSE>, <&rcc FDCAN_K>;
> + clocks = <&scmi0_clk CK_SCMI0_HSE>, <&rcc FDCAN_K>;
> clock-names = "hclk", "cclk";
> bosch,mram-cfg = <0x1400 0 0 32 0 0 2 2>;
> status = "disabled";
> diff --git a/arch/arm/boot/dts/stm32mp157.dtsi b/arch/arm/boot/dts/stm32mp157.dtsi
> index 54e73ccea446..7b06c08e3a23 100644
> --- a/arch/arm/boot/dts/stm32mp157.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157.dtsi
> @@ -20,7 +20,7 @@
> dsi: dsi@5a000000 {
> compatible = "st,stm32-dsi";
> reg = <0x5a000000 0x800>;
> - clocks = <&rcc DSI_K>, <&clk_hse>, <&rcc DSI_PX>;
> + clocks = <&rcc DSI_K>, <&scmi0_clk CK_SCMI0_HSE>, <&rcc DSI_PX>;
> clock-names = "pclk", "ref", "px_clk";
> resets = <&rcc DSI_R>;
> reset-names = "apb";
> diff --git a/arch/arm/boot/dts/stm32mp15xc.dtsi b/arch/arm/boot/dts/stm32mp15xc.dtsi
> index b06a55a2fa18..435846883f25 100644
> --- a/arch/arm/boot/dts/stm32mp15xc.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xc.dtsi
> @@ -10,8 +10,8 @@
> compatible = "st,stm32mp1-cryp";
> reg = <0x54001000 0x400>;
> interrupts = <GIC_SPI 79 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&rcc CRYP1>;
> - resets = <&rcc CRYP1_R>;
> + clocks = <&scmi0_clk CK_SCMI0_CRYP1>;
> + resets = <&scmi0_reset RST_SCMI0_CRYP1>;
> status = "disabled";
> };
> };
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-02-18 11:17 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-26 9:01 [PATCH v2 00/14] Introduce STM32MP1 RCC in secured mode gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 01/14] clk: stm32mp1: merge 'clk-hsi-div' and 'ck_hsi' into one clock gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 02/14] clk: stm32mp1: merge 'ck_hse_rtc' and 'ck_rtc' " gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-02-09 8:00 ` Stephen Boyd
2021-02-09 8:00 ` Stephen Boyd
2021-02-12 8:08 ` gabriel.fernandez
2021-02-12 8:08 ` gabriel.fernandez
[not found] ` <161369805767.1254594.5233096495913117772@swboyd.mtv.corp.google.com>
2021-02-23 16:34 ` gabriel.fernandez
2021-02-23 16:34 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 03/14] clk: stm32mp1: remove intermediate pll clocks gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 04/14] clk: stm32mp1: convert to module driver gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 05/14] clk: stm32mp1: move RCC reset controller into RCC clock driver gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 06/14] reset: stm32mp1: remove stm32mp1 reset gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 07/14] dt-bindings: clock: add IDs for SCMI clocks on stm32mp15 gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-02-09 17:52 ` Rob Herring
2021-02-09 17:52 ` Rob Herring
2021-01-26 9:01 ` [PATCH v2 08/14] dt-bindings: reset: add IDs for SCMI reset domains " gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-02-09 17:53 ` Rob Herring
2021-02-09 17:53 ` Rob Herring
2021-01-26 9:01 ` [PATCH v2 09/14] dt-bindings: reset: add MCU HOLD BOOT ID " gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-02-09 17:54 ` Rob Herring
2021-02-09 17:54 ` Rob Herring
2021-01-26 9:01 ` [PATCH v2 10/14] clk: stm32mp1: new compatible for secure RCC support gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 11/14] ARM: dts: stm32: define SCMI resources on stm32mp15 gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-01-26 9:01 ` [PATCH v2 12/14] ARM: dts: stm32: move clocks/resets to SCMI resources for stm32mp15 gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-02-18 9:45 ` Ahmad Fatoum [this message]
2021-02-18 9:45 ` [Linux-stm32] " Ahmad Fatoum
2021-01-26 9:01 ` [PATCH v2 13/14] dt-bindings: clock: stm32mp1 new compatible for secure rcc gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-02-09 17:56 ` Rob Herring
2021-02-09 17:56 ` Rob Herring
2021-01-26 9:01 ` [PATCH v2 14/14] ARM: dts: stm32: introduce basic boot include on stm32mp15x board gabriel.fernandez
2021-01-26 9:01 ` gabriel.fernandez
2021-03-09 21:50 ` [PATCH v2 00/14] Introduce STM32MP1 RCC in secured mode Alex G.
2021-03-09 21:50 ` Alex G.
2021-03-11 8:08 ` Alexandre TORGUE
2021-03-11 11:43 ` Marek Vasut
2021-03-11 11:43 ` Marek Vasut
2021-03-11 13:15 ` Alexandre TORGUE
2021-03-11 13:23 ` Marek Vasut
2021-03-11 13:23 ` Marek Vasut
2021-03-11 14:02 ` Alexandre TORGUE
2021-03-11 14:41 ` [Linux-stm32] " Ahmad Fatoum
2021-03-11 14:41 ` Ahmad Fatoum
2021-03-11 15:18 ` Alexandre TORGUE
2021-03-11 15:49 ` Ahmad Fatoum
2021-03-11 15:49 ` Ahmad Fatoum
2021-03-11 16:11 ` Marek Vasut
2021-03-11 16:11 ` Marek Vasut
2021-03-11 18:10 ` Alexandre TORGUE
2021-03-11 18:23 ` Alex G.
2021-03-11 18:23 ` Alex G.
2021-03-11 20:09 ` Marek Vasut
2021-03-11 20:09 ` Marek Vasut
2021-03-12 8:22 ` Alexandre TORGUE
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=c74e8909-84a3-f6dc-50fb-18349fd79ece@pengutronix.de \
--to=a.fatoum@pengutronix.de \
--cc=alexandre.torgue@st.com \
--cc=devicetree@vger.kernel.org \
--cc=etienne.carriere@st.com \
--cc=gabriel.fernandez@foss.st.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=marex@denx.de \
--cc=mcoquelin.stm32@gmail.com \
--cc=mturquette@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=sboyd@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.