<div dir="ltr"><br><br><div class="gmail_quote">On Fri, May 8, 2015 at 8:31 AM Aaron Durbin <<a href="mailto:adurbin@google.com">adurbin@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I personally feel that changing device_t type based on stage makes the<br>
code non-obvious and hard to follow.<br>
<br>
I'd rather we *always* provide simple u32 device_t functions in all<br>
stages while allowing struct device IO functions for use in ramstage.<br><br></blockquote><div><br></div><div><br></div><div>that's pretty much the approach I tried to take in v3. While the device_t stuff is cool it's too hard for my brain. </div><div><br></div><div>BUT, it's there, been in use for a long time, not sure this is the best time to change it.</div><div><br></div><div>But The Will of the People will decide.</div><div><br></div><div>ron </div></div></div>