Thread (8 messages) 8 messages, 3 authors, 2014-01-09

Re: [PATCH v2 3/6] ARM: tegra: Add efuse bindings

From: Stephen Warren <hidden>
Date: 2014-01-08 22:42:00
Also in: linux-arm-kernel, linux-tegra, lkml

On 01/08/2014 01:09 PM, Thierry Reding wrote:
On Wed, Jan 08, 2014 at 11:50:35AM -0700, Stephen Warren wrote:
quoted
On 01/08/2014 06:39 AM, Thierry Reding wrote:
quoted
On Mon, Jan 06, 2014 at 01:40:51PM -0700, Stephen Warren wrote:
quoted
On 12/24/2013 06:32 AM, Peter De Schrijver wrote:
[...]
quoted
quoted
diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi
quoted
+	efuse@7000F800 {
"fuse" might be a better node name; "efuse" is presumably the name of
the instance, not the type of object.
There's another occurrence I noticed recently where we haven't followed
that rule. The PMIC node on Venice2 for instance is called as3722.
Perhaps that should also be renamed.
Yes, we should fix that. Care to send a patch?
Ugh... I've just been going through some of the other DTS files and see
that quite a lot of other places aren't following this rule either. The
...
Perhaps it isn't worth fixing them all up after all?
I guess that just means the patch will be a little bigger; it doesn't
seem like that's a good reason not to fix the issue.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help