Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752835AbcJMItm convert rfc822-to-8bit (ORCPT ); Thu, 13 Oct 2016 04:49:42 -0400 Received: from mailapp02.imgtec.com ([217.156.133.132]:57389 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752220AbcJMItk (ORCPT ); Thu, 13 Oct 2016 04:49:40 -0400 From: James Hartley To: Rahul Bedarkar , Rob Herring CC: Ralf Baechle , Mark Rutland , "linux-mips@linux-mips.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board Thread-Topic: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board Thread-Index: AQHSH83MFJLEHIy+xEmmOrinu3/Ha6ChsusAgAQ8JACAAC050A== Date: Thu, 13 Oct 2016 08:49:34 +0000 Message-ID: <72BC0C8BD7BB6F45988A99382E5FBAE5889ED617@HHMAIL01.hh.imgtec.org> References: <1475757094-31089-1-git-send-email-rahul.bedarkar@imgtec.com> <1475757094-31089-2-git-send-email-rahul.bedarkar@imgtec.com> <20161010142152.GA7920@rob-hp-laptop> <57FF3172.4010709@imgtec.com> In-Reply-To: <57FF3172.4010709@imgtec.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.40.7.55] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1406 Lines: 38 Hi Rahul, > -----Original Message----- > From: Rahul Bedarkar > Sent: 13 October 2016 08:02 > To: Rob Herring > Cc: Ralf Baechle; Mark Rutland; James Hartley; linux-mips@linux-mips.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board > > Hi, > > On Monday 10 October 2016 07:51 PM, Rob Herring wrote: > >> + > >> + memory { > > > > Is 0 the actual base, or that gets filled in by bootloader? If the > > formet, add unit address. > > > > Bootloader (uboot) can override or fixup memory node. But with version of > bootloader I tested with, base address is hardcoded to 0 and only size may > get changed. But since booloader can override or fixup memory node, I > assume we don't add unit address in this case. > > >> + device_type = "memory"; > >> + reg = <0x00000000 0x08000000>; > > I now realized that size is incorrectly specified in memory node. It should be > 256MB and not 128MB. I will fix this in v2. This is board dependent, and since you have already explained that the boot loader is expected to configure this, I would recommend leaving it at 128MB (which I believe is the size of the smallest DDR part in use on existing boards). This would mean that if for some reason the boot loader did not adjust the settings, it would still boot on all currently known boards. James. > > Thanks, > Rahul