From mboxrd@z Thu Jan 1 00:00:00 1970 From: gerald@granddesignsgroup.com (Gerald Coley) Date: Thu, 6 Nov 2014 10:51:04 -0600 Subject: [PATCH] ARM: dts: Add am57xx-beagle-x15 In-Reply-To: <20141106164843.GY31454@atomide.com> References: <1415290702-11353-1-git-send-email-nm@ti.com> <20141106163207.GF16896@saruman> <545BA3E6.90105@ti.com> <545BA513.2090907@ti.com> <20141106164843.GY31454@atomide.com> Message-ID: <006701cff9e1$dc0cfbf0$9426f3d0$@granddesignsgroup.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Convert 12V to 5V and 3.3V. 5V supplies are too expensive at the required wattage level. Gerald -----Original Message----- From: Tony Lindgren [mailto:tony at atomide.com] Sent: Thursday, November 6, 2014 10:49 AM To: Nishanth Menon Cc: Tom Rini; balbi at ti.com; devicetree at vger.kernel.org; beagleboard at googlegroups.com; linux-kernel at vger.kernel.org; gerald at granddesignsgroup.com; fcooper at ti.com; linux-omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org Subject: Re: [PATCH] ARM: dts: Add am57xx-beagle-x15 * 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