Re: [PATCH] ARM: dts: Add am57xx-beagle-x15

[Nishanth Menon]

On 10:53-20141106, Nishanth Menon wrote:
> On 11/06/2014 10:48 AM, Tony Lindgren wrote:
> > * Nishanth Menon <nm@xxxxxx> [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?
> yep - will do that similar to
> https://patchwork.kernel.org/patch/5125431/ - same rationale unless
> there is any specific voltage rail that you are explicitly interested
> in that needs additional explanation. Is there additional rails of
> interest?

Here is what I have in mind: (updated with comments from Felipe/Tom and
above) - would you like more details added - if you could point out
areas of interest, I can help improve information provided here.

8<---