Re: [PATCH v4 15/19] ARM: dts: tegra: Add clock-cells property to pmc

Dmitry Osipenko
2019-12-18T21:36:13Z

17.12.2019 23:04, Sowjanya Komatineni пишет:
> Tegra PMC has clk_out_1, clk_out_2, clk_out_3, and blink clock.
>
> These clocks are moved from clock driver to pmc driver with pmc
> as the clock provider for these clocks.
>
> This patch adds #clock-cells property with 1 clock specifier to
> the Tegra PMC node in device tree.
>
> Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
> ---
> arch/arm/boot/dts/tegra114.dtsi | 4 +++-
> arch/arm/boot/dts/tegra124.dtsi | 4 +++-
> arch/arm/boot/dts/tegra20.dtsi | 4 +++-
> arch/arm/boot/dts/tegra30.dtsi | 4 +++-
> 4 files changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi
> index 0d7a6327e404..b8f12f24f314 100644
> --- a/arch/arm/boot/dts/tegra114.dtsi
> +++ b/arch/arm/boot/dts/tegra114.dtsi
> @@ -4,6 +4,7 @@
> #include <dt-bindings/memory/tegra114-mc.h>
> #include <dt-bindings/pinctrl/pinctrl-tegra.h>
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/soc/tegra-pmc.h>
>
> / {
> compatible = "nvidia,tegra114";
> @@ -514,11 +515,12 @@
> status = "disabled";
> };
>
> - pmc@7000e400 {
> + pmc: pmc@7000e400 {
> compatible = "nvidia,tegra114-pmc";
> reg = <0x7000e400 0x400>;
> clocks = <&tegra_car TEGRA114_CLK_PCLK>, <&clk32k_in>;
> clock-names = "pclk", "clk32k_in";
> + #clock-cells = <1>;
> };
>
> fuse@7000f800 {
> diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi
> index 413bfb981de8..d0802c4ae3bf 100644
> --- a/arch/arm/boot/dts/tegra124.dtsi
> +++ b/arch/arm/boot/dts/tegra124.dtsi
> @@ -6,6 +6,7 @@
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> #include <dt-bindings/reset/tegra124-car.h>
> #include <dt-bindings/thermal/tegra124-soctherm.h>
> +#include <dt-bindings/soc/tegra-pmc.h>
>
> / {
> compatible = "nvidia,tegra124";
> @@ -595,11 +596,12 @@
> clocks = <&tegra_car TEGRA124_CLK_RTC>;
> };
>
> - pmc@7000e400 {
> + pmc: pmc@7000e400 {
> compatible = "nvidia,tegra124-pmc";
> reg = <0x0 0x7000e400 0x0 0x400>;
> clocks = <&tegra_car TEGRA124_CLK_PCLK>, <&clk32k_in>;
> clock-names = "pclk", "clk32k_in";
> + #clock-cells = <1>;
> };
>
> fuse@7000f800 {
> diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi
> index 9c58e7fcf5c0..85a64747bec6 100644
> --- a/arch/arm/boot/dts/tegra20.dtsi
> +++ b/arch/arm/boot/dts/tegra20.dtsi
> @@ -4,6 +4,7 @@
> #include <dt-bindings/memory/tegra20-mc.h>
> #include <dt-bindings/pinctrl/pinctrl-tegra.h>
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/soc/tegra-pmc.h>
>
> / {
> compatible = "nvidia,tegra20";
> @@ -608,11 +609,12 @@
> status = "disabled";
> };
>
> - pmc@7000e400 {
> + pmc: pmc@7000e400 {
> compatible = "nvidia,tegra20-pmc";
> reg = <0x7000e400 0x400>;
> clocks = <&tegra_car TEGRA20_CLK_PCLK>, <&clk32k_in>;
> clock-names = "pclk", "clk32k_in";
> + #clock-cells = <1>;
> };
>
> mc: memory-controller@7000f000 {
> diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi
> index 55ae050042ce..4d5e9d0001d3 100644
> --- a/arch/arm/boot/dts/tegra30.dtsi
> +++ b/arch/arm/boot/dts/tegra30.dtsi
> @@ -4,6 +4,7 @@
> #include <dt-bindings/memory/tegra30-mc.h>
> #include <dt-bindings/pinctrl/pinctrl-tegra.h>
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/soc/tegra-pmc.h>
>
> / {
> compatible = "nvidia,tegra30";
> @@ -714,11 +715,12 @@
> status = "disabled";
> };
>
> - pmc@7000e400 {
> + pmc: pmc@7000e400 {

I think will be a bit better to name the phandle as "tegra_pmc", it has
the same vibe as the "tegra_car".

> compatible = "nvidia,tegra30-pmc";
> reg = <0x7000e400 0x400>;
> clocks = <&tegra_car TEGRA30_CLK_PCLK>, <&clk32k_in>;
> clock-names = "pclk", "clk32k_in";
> + #clock-cells = <1>;
> };
>
> mc: memory-controller@7000f000 {
>