Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752336AbcDBQMf (ORCPT ); Sat, 2 Apr 2016 12:12:35 -0400 Received: from mail-lf0-f49.google.com ([209.85.215.49]:33587 "EHLO mail-lf0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751550AbcDBQMd (ORCPT ); Sat, 2 Apr 2016 12:12:33 -0400 Subject: Re: [PATCH] ARM: dts: armv7-m: add address to unit name To: Stefan Agner , arnd@arndb.de References: <1459577299-2818-1-git-send-email-stefan@agner.ch> Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, mcoquelin.stm32@gmail.com, pawel.moll@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, kernel@pengutronix.de, u.kleine-koenig@pengutronix.de, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org From: Sergei Shtylyov Message-ID: <56FFEF6E.2000002@cogentembedded.com> Date: Sat, 2 Apr 2016 19:12:30 +0300 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.1 MIME-Version: 1.0 In-Reply-To: <1459577299-2818-1-git-send-email-stefan@agner.ch> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 956 Lines: 36 Hello. On 4/2/2016 9:08 AM, Stefan Agner wrote: > Add address to unit name to remove the following warning: > Warning (unit_address_vs_reg): Node /nv-interrupt-controller has a > reg or ranges property, but no unit name > > Signed-off-by: Stefan Agner > --- > Hi Arnd, > > Not sure through which tree this should go but an earlier patch > seemd to be applied directly by you... > > -- > Stefan > > arch/arm/boot/dts/armv7-m.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/armv7-m.dtsi b/arch/arm/boot/dts/armv7-m.dtsi > index b1ad7cf..8126bec 100644 > --- a/arch/arm/boot/dts/armv7-m.dtsi > +++ b/arch/arm/boot/dts/armv7-m.dtsi > @@ -1,7 +1,7 @@ > #include "skeleton.dtsi" > > / { > - nvic: nv-interrupt-controller { > + nvic: nv-interrupt-controller@0xe000e100 { Drop "0x" from please. And the name should be just "interrupt-controller@..." WBR, Sergei