Re: [PATCH 00/19] Tegra210 DFLL support

Thierry Reding
2018-12-04T15:10:59Z

[-- Attachment #1.1: Type: text/plain, Size: 3920 bytes --]

On Tue, Dec 04, 2018 at 05:25:29PM +0800, Joseph Lo wrote:
> This series introduces support for the DFLL as a CPU clock source
> on Tegra210. As Jetson TX1 uses a PWM controlled regulator IC which
> is driven directly by the DFLLs PWM output, we also introduce support
> for PWM regulators next to I2C controlled regulators. The DFLL output
> frequency is directly controlled by the regulator voltage. The registers
> for controlling the PWM are part of the DFLL IP block, so there's no
> separate linux regulator object involved because the regulator IC only
> supplies the rail powering the CPUs. It doesn't have any other controls.
>
> The patch 1~4 are the patches of DT bindings update for DFLL clock and
> Tegra124 cpufreq, which add PWM and Tegra210 support for DFLL clock and
> remove deprecate properties for Tegra124 cpufreq bindings.
>
> The patch 5~10 are the patches for DFLL clock driver update for PWM-mode
> DFLL support.
>
> The patch 11 and 12 are the Tegra124 cpufreq driver update to make it
> work with Tegra210.
>
> The patch 13~18 are the devicetree files update for Tegra210 SoC and
> platforms. Two platforms are updated here for different DFLL mode usage.
> The Tegra210-p2371-2180 (a.k.a. Jetson Tx1) uses DFLL-PWM and the
> Tegra210-smaug (a.k.a. Pixel C) uses DFLL-I2C. So two different modes
> are verified with this series.
>
> The patch 19 is the patch for enabling the CPU regulator for Smaug
> board.
>
> Joseph Lo (16):
> dt-bindings: clock: tegra124-dfll: add Tegra210 support
> dt-bindings: cpufreq: tegra124: remove vdd-cpu-supply from required
> properties
> dt-bindings: cpufreq: tegra124: remove cpu_lp clock from required
> properties
> clk: tegra: dfll: CVB calculation alignment with the regulator
> clk: tegra: dfll: support PWM regulator control
> clk: tegra: dfll: round down voltages based on alignment
> clk: tegra: dfll: add CVB tables for Tegra210
> cpufreq: tegra124: do not handle the CPU rail
> cpufreq: tegra124: extend to support Tegra210
> arm64: dts: tegra210: add DFLL clock
> arm64: dts: tegra210: add CPU clocks
> arm64: dts: tegra210-p2597: add pinmux for PWM-based DFLL support
> arm64: dts: tegra210-p2371-2180: enable DFLL clock
> arm64: dts: tegra210-smaug: add CPU power rail regulator
> arm64: dts: tegra210-smaug: enable DFLL clock
> arm64: defconfig: Enable MAX8973 regulator
>
> Peter De Schrijver (3):
> dt-bindings: clock: tegra124-dfll: Update DFLL binding for PWM
> regulator
> clk: tegra: dfll: registration for multiple SoCs
> clk: tegra: dfll: build clk-dfll.c for Tegra124 and Tegra210
>
> .../bindings/clock/nvidia,tegra124-dfll.txt | 77 ++-
> .../cpufreq/nvidia,tegra124-cpufreq.txt | 6 +-
> .../boot/dts/nvidia/tegra210-p2371-2180.dts | 20 +
> .../arm64/boot/dts/nvidia/tegra210-p2597.dtsi | 14 +
> arch/arm64/boot/dts/nvidia/tegra210-smaug.dts | 31 +
> arch/arm64/boot/dts/nvidia/tegra210.dtsi | 25 +
> arch/arm64/configs/defconfig | 1 +
> drivers/clk/tegra/Kconfig | 5 +
> drivers/clk/tegra/Makefile | 2 +-
> drivers/clk/tegra/clk-dfll.c | 455 ++++++++++++---
> drivers/clk/tegra/clk-dfll.h | 6 +-
> drivers/clk/tegra/clk-tegra124-dfll-fcpu.c | 536 +++++++++++++++++-
> drivers/clk/tegra/cvb.c | 12 +-
> drivers/clk/tegra/cvb.h | 7 +-
> drivers/cpufreq/Kconfig.arm | 2 +-
> drivers/cpufreq/tegra124-cpufreq.c | 29 +-
> 16 files changed, 1095 insertions(+), 133 deletions(-)

Hi Joseph,

can you highlight the build and runtime dependencies between the various
patches? For example, can I pick up all the arch/arm64 patches into the
Tegra tree without breaking anything?

Thierry

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org