Messages in this thread | | | Subject | Re: [PATCH 2/3] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster | From | Mikko Perttunen <> | Date | Mon, 3 Apr 2017 18:18:43 +0300 |
| |
On 04/03/2017 05:06 PM, Thierry Reding wrote: > On Mon, Apr 03, 2017 at 03:42:24PM +0300, Mikko Perttunen wrote: >> The Tegra186 CCPLEX_CLUSTER area contains memory-mapped >> registers that initiate CPU frequency/voltage transitions. >> >> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com> >> --- >> .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt | 22 ++++++++++++++++++++++ >> 1 file changed, 22 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt >> >> diff --git a/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt >> new file mode 100644 >> index 000000000000..50cd615219e9 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt >> @@ -0,0 +1,22 @@ >> +NVIDIA Tegra CCPLEX_CLUSTER area >> + >> +Required properties: >> +- compatible: Should contain one of the following: >> + - "nvidia,tegra186-ccplex-cluster": for Tegra186 >> +- reg: Must contain an (offset, length) pair of the register set for each >> + entry in reg-names. >> +- reg-names: Must include the following entries: >> + - "a57": Public aperture for A57 CPU cluster >> + - "denver": Public aperture for Denver CPU cluster >> +- nvidia,bpmp: Phandle to BPMP device that can be queried for OPP tables > > "phandle"
Will fix.
> >> +Example: >> + >> + ccplex@e000000 { >> + compatible = "nvidia,tegra186-ccplex-cluster"; >> + reg = <0x0 0x0e060000 0x0 0x1000>, >> + <0x0 0x0e070000 0x0 0x1000>; >> + reg-names = "a57", "denver"; >> + >> + nvidia,bpmp = <&bpmp>; >> + }; > > Where's the information about the register offsets coming from? The TRM > says that CCPLEX_CLUSTER has a single aperture from 0x0e000000 to > 0x0e3fffff.
Some internal document with a name related to Denver power management, IIRC. I'll link it to you tomorrow.
> > Thierry >
| |