From: Ivan Khoronzhuk <hidden> Date: 2014-05-21 14:18:35
The pll controller register set includes sets of registers with
different purposes, so it's logically to add syscon entry to be able
to access them from appropriate places.
Signed-off-by: Ivan Khoronzhuk <redacted>
---
Based on linux-next/master
arch/arm/boot/dts/keystone.dtsi | 5 +++++
1 file changed, 5 insertions(+)
From: Ivan Khoronzhuk <hidden> Date: 2014-05-21 14:57:00
Ignore this. It'll be included in "Introduce keystone reset driver"
patch series
On 05/21/2014 05:17 PM, Ivan Khoronzhuk wrote:
quoted hunk
The pll controller register set includes sets of registers with
different purposes, so it's logically to add syscon entry to be able
to access them from appropriate places.
Signed-off-by: Ivan Khoronzhuk <redacted>
---
Based on linux-next/master
arch/arm/boot/dts/keystone.dtsi | 5 +++++
1 file changed, 5 insertions(+)
From: Sergei Shtylyov <hidden> Date: 2014-05-21 18:18:58
Hello.
On 05/21/2014 06:17 PM, Ivan Khoronzhuk wrote:
The pll controller register set includes sets of registers with
different purposes, so it's logically to add syscon entry to be able
to access them from appropriate places.
I think it's better to follow the pattern established by "reset-controller"
and use hyphen ISO dash in the node name. This is also a common practice to
use hyphens in the device trees overall.
From: Sergei Shtylyov <hidden> Date: 2014-05-21 18:22:48
On 05/21/2014 10:18 PM, Sergei Shtylyov wrote:
quoted
The pll controller register set includes sets of registers with
different purposes, so it's logically to add syscon entry to be able
to access them from appropriate places.
I think it's better to follow the pattern established by "reset-controller"
and use hyphen ISO dash in the node name. This is also a common practice to
Oops, hyphen and dash are basically the same thing; I meant
s/dash/underscore/.
From: Ivan Khoronzhuk <hidden> Date: 2014-05-22 13:52:01
On 05/21/2014 09:22 PM, Sergei Shtylyov wrote:
On 05/21/2014 10:18 PM, Sergei Shtylyov wrote:
quoted
quoted
The pll controller register set includes sets of registers with
different purposes, so it's logically to add syscon entry to be able
to access them from appropriate places.
I think it's better to follow the pattern established by
"reset-controller"
and use hyphen ISO dash in the node name. This is also a common
practice to
Oops, hyphen and dash are basically the same thing; I meant
s/dash/underscore/.
You meant s/underscore/dash.
I'll replace.
FYI: the change is included in new series [Patch v5 0/7] Introduce
keystone reset driver
And this patch is squashed in:
[Patch v5 6/7] ARM: dts: keystone: update reset node to work with reset
driver
Thanks.