From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Thu, 6 Nov 2014 08:48:44 -0800 Subject: [PATCH] ARM: dts: Add am57xx-beagle-x15 In-Reply-To: <545BA513.2090907@ti.com> References: <1415290702-11353-1-git-send-email-nm@ti.com> <20141106163207.GF16896@saruman> <545BA3E6.90105@ti.com> <545BA513.2090907@ti.com> Message-ID: <20141106164843.GY31454@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Nishanth Menon [141106 08:44]: > On 11/06/2014 10:37 AM, Tom Rini wrote: > > >>> device_type = "memory"; + reg = <0x80000000 0x40000000>; /* 1GB > >>> to start. Target 2GB */ > >> > >> 1GiB ? Why would you put this here btw ? u-boot fills this one up. > > > > Yes, it should either be the full and correct value or 0x0 (like a > > number of PowerPC platforms do) so it's clear something else gives us > > the right value here. > > Honest mistake, my bad.. I thought I cleaned up the patch! Sigh!!! i > will repost with proper 2GB. it is better that way in case being used > with "other bootloaders" which are not exactly too good like u-boot. Can you also describe why all the always-on regulators are needed? Regards, Tony