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? -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
