On Wed, Jan 08, 2014 at 03:41:52PM -0700, Stephen Warren wrote: > On 01/08/2014 01:09 PM, Thierry Reding wrote: > > On Wed, Jan 08, 2014 at 11:50:35AM -0700, Stephen Warren wrote: > >> On 01/08/2014 06:39 AM, Thierry Reding wrote: > >>> On Mon, Jan 06, 2014 at 01:40:51PM -0700, Stephen Warren wrote: > >>>> On 12/24/2013 06:32 AM, Peter De Schrijver wrote: > >>> [...] > >>>>> diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi > >>>> > >>>>> + 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. Alright, I'll make a pass over all the DTS files that we have and try to restore some consistency. Thierry
Attachment:
pgpTNYSXJpBik.pgp
Description: PGP signature