Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1663403 > unrolled thread

[PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

Started byMikko Perttunen <mperttunen@nvidia.com>
First post2017-06-12 12:30 +0200
Last post2017-06-13 14:50 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster Mikko Perttunen <mperttunen@nvidia.com> - 2017-06-12 12:30 +0200
    Re: [PATCH v3 1/2] dt-bindings: Add bindings for  nvidia,tegra186-ccplex-cluster Mikko Perttunen <cyndis@kapsi.fi> - 2017-06-13 14:50 +0200
      Re: [PATCH v3 1/2] dt-bindings: Add bindings for  nvidia,tegra186-ccplex-cluster Thierry Reding <thierry.reding@gmail.com> - 2017-06-13 16:20 +0200
    Re: [PATCH v3 1/2] dt-bindings: Add bindings for  nvidia,tegra186-ccplex-cluster Mikko Perttunen <cyndis@kapsi.fi> - 2017-06-13 14:50 +0200
    Re: [PATCH v3 1/2] dt-bindings: Add bindings for  nvidia,tegra186-ccplex-cluster Thierry Reding <thierry.reding@gmail.com> - 2017-06-13 14:50 +0200

#1663403 — [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

FromMikko Perttunen <mperttunen@nvidia.com>
Date2017-06-12 12:30 +0200
Subject[PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster
Message-ID<tRuLD-Ua-13@gated-at.bofh.it>
The Tegra186 CCPLEX_CLUSTER area contains memory-mapped
registers that initiate CPU frequency/voltage transitions.

Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
Acked-by: Rob Herring <robh@kernel.org>
---
 .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt     | 20 ++++++++++++++++++++
 1 file changed, 20 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..0c80cd8ee839
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt
@@ -0,0 +1,20 @@
+NVIDIA Tegra CCPLEX_CLUSTER area
+
+The Tegra186 CCPLEX_CLUSTER area contains memory-mapped
+registers that initiate CPU frequency/voltage transitions.
+
+Required properties:
+- compatible: Should contain one of the following:
+  - "nvidia,tegra186-ccplex-cluster": for Tegra186
+- reg: Must contain an (offset, length) pair of the device's MMIO
+  register area
+- nvidia,bpmp: phandle to BPMP device that can be queried for OPP tables
+
+Example:
+
+	ccplex@e000000 {
+		compatible = "nvidia,tegra186-ccplex-cluster";
+		reg = <0x0 0x0e000000 0x0 0x3fffff>,
+
+		nvidia,bpmp = <&bpmp>;
+	};
-- 
2.1.4

[toc] | [next] | [standalone]

#1664805 — Re: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

FromMikko Perttunen <cyndis@kapsi.fi>
Date2017-06-13 14:50 +0200
SubjectRe: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster
Message-ID<tRTqG-81T-5@gated-at.bofh.it>
In reply to#1663403

On 13.06.2017 15:46, Mikko Perttunen wrote:
> On 13.06.2017 15:42, Thierry Reding wrote:
>> On Mon, Jun 12, 2017 at 01:23:04PM +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>
>>> Acked-by: Rob Herring <robh@kernel.org>
>>> ---
>>>  .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt     | 20
>>> ++++++++++++++++++++
>>>  1 file changed, 20 insertions(+)
>>>  create mode 100644
>>> Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt
>>>
>>
>> The ARM SoC maintainers don't like to pick up device tree bindings, so
>> I'd prefer this to go through the cpufreq tree that also contains the
>> driver patches. Presumably this wasn't merged through that tree because
>> of the missing Acked-by by a device tree maintainer? Given that Rob's
>> acked it now, maybe you can resend this to Viresh, who I think had
>> picked up the driver?
>
> Sure, I'll do that.

> I guess the .dts change should then also go in that
> way?

Or I guess not, since you applied it :)

Cheers,
Mikko

>
> Mikko
>
>>
>> Thanks,
>> Thierry
>>

[toc] | [prev] | [next] | [standalone]

#1664919 — Re: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

FromThierry Reding <thierry.reding@gmail.com>
Date2017-06-13 16:20 +0200
SubjectRe: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster
Message-ID<tRUPM-zh-9@gated-at.bofh.it>
In reply to#1664805

[Multipart message — attachments visible in raw view] — view raw

On Tue, Jun 13, 2017 at 03:47:39PM +0300, Mikko Perttunen wrote:
> 
> 
> On 13.06.2017 15:46, Mikko Perttunen wrote:
> > On 13.06.2017 15:42, Thierry Reding wrote:
> > > On Mon, Jun 12, 2017 at 01:23:04PM +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>
> > > > Acked-by: Rob Herring <robh@kernel.org>
> > > > ---
> > > >  .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt     | 20
> > > > ++++++++++++++++++++
> > > >  1 file changed, 20 insertions(+)
> > > >  create mode 100644
> > > > Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt
> > > > 
> > > 
> > > The ARM SoC maintainers don't like to pick up device tree bindings, so
> > > I'd prefer this to go through the cpufreq tree that also contains the
> > > driver patches. Presumably this wasn't merged through that tree because
> > > of the missing Acked-by by a device tree maintainer? Given that Rob's
> > > acked it now, maybe you can resend this to Viresh, who I think had
> > > picked up the driver?
> > 
> > Sure, I'll do that.
> 
> > I guess the .dts change should then also go in that
> > way?
> 
> Or I guess not, since you applied it :)

So the rule of thumb is that .dts changes should go through ARM SoC and
device tree binding changes should be going through the same tree as the
driver changes that implement the binding.

Thierry

[toc] | [prev] | [next] | [standalone]

#1664810 — Re: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

FromMikko Perttunen <cyndis@kapsi.fi>
Date2017-06-13 14:50 +0200
SubjectRe: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster
Message-ID<tRTqG-81T-9@gated-at.bofh.it>
In reply to#1663403
On 13.06.2017 15:42, Thierry Reding wrote:
> On Mon, Jun 12, 2017 at 01:23:04PM +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>
>> Acked-by: Rob Herring <robh@kernel.org>
>> ---
>>  .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt     | 20 ++++++++++++++++++++
>>  1 file changed, 20 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt
>
> The ARM SoC maintainers don't like to pick up device tree bindings, so
> I'd prefer this to go through the cpufreq tree that also contains the
> driver patches. Presumably this wasn't merged through that tree because
> of the missing Acked-by by a device tree maintainer? Given that Rob's
> acked it now, maybe you can resend this to Viresh, who I think had
> picked up the driver?

Sure, I'll do that. I guess the .dts change should then also go in that way?

Mikko

>
> Thanks,
> Thierry
>

[toc] | [prev] | [next] | [standalone]

#1664811 — Re: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

FromThierry Reding <thierry.reding@gmail.com>
Date2017-06-13 14:50 +0200
SubjectRe: [PATCH v3 1/2] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster
Message-ID<tRTqG-81T-7@gated-at.bofh.it>
In reply to#1663403

[Multipart message — attachments visible in raw view] — view raw

On Mon, Jun 12, 2017 at 01:23:04PM +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>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
>  .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt     | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-ccplex-cluster.txt

The ARM SoC maintainers don't like to pick up device tree bindings, so
I'd prefer this to go through the cpufreq tree that also contains the
driver patches. Presumably this wasn't merged through that tree because
of the missing Acked-by by a device tree maintainer? Given that Rob's
acked it now, maybe you can resend this to Viresh, who I think had
picked up the driver?

Thanks,
Thierry

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web