Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753903AbcCYQgl (ORCPT ); Fri, 25 Mar 2016 12:36:41 -0400 Received: from mail-lb0-f177.google.com ([209.85.217.177]:36123 "EHLO mail-lb0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753463AbcCYQgj (ORCPT ); Fri, 25 Mar 2016 12:36:39 -0400 Subject: Re: [PATCH v3 01/16] dt: davinci: use proper address after @ To: Rob Herring References: <1458863503-31121-1-git-send-email-david@lechnology.com> <1458863503-31121-2-git-send-email-david@lechnology.com> <56F53D20.6030709@cogentembedded.com> Cc: David Lechner , Petr Kulhavy , David.Laight@aculab.com, Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Sekhar Nori , Kevin Hilman , Kishon Vijay Abraham I , Greg Kroah-Hartman , Alan Stern , Bin Liu , Lee Jones , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , "moderated list:ARM PORT" , "open list:USB SUBSYSTEM" From: Sergei Shtylyov Organization: Cogent Embedded Message-ID: <56F56911.6020603@cogentembedded.com> Date: Fri, 25 Mar 2016 19:36:33 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; 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: 1327 Lines: 43 Hello. On 03/25/2016 06:13 PM, Rob Herring wrote: >>> TI has been using the physical address in DT after the @ in device nodes. >>> The device tree convention is to use the same address that is used for >>> the reg property. This updates all davinci DT files to use the proper >>> convention. >>> >>> Signed-off-by: David Lechner >> >> [...] >> >>> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi >>> index 226cda7..4294849 100644 >>> --- a/arch/arm/boot/dts/da850.dtsi >>> +++ b/arch/arm/boot/dts/da850.dtsi >>> @@ -15,7 +15,7 @@ >>> #address-cells = <1>; >>> #size-cells = <1>; >>> ranges; >>> - intc: interrupt-controller { >>> + intc: interrupt-controller@fffee000 { >>> compatible = "ti,cp-intc"; >>> interrupt-controller; >>> #interrupt-cells = <1>; >>> @@ -23,7 +23,7 @@ >>> reg = <0xfffee000 0x2000>; >>> }; >>> }; >>> - soc { >>> + soc@1c00000 { >> >> >> This node doesn't have the "reg" prop. > But probably has a ranges property which is correct. Yes, it does. Didn't know that "ranges" can be used that way, thanks... > Rob MBR, Sergei