On Mon, May 5, 2014 at 7:29 AM, Stefano Babic sbabic@denx.de wrote:
Hi Fabio,
just a minor question:
On 04/05/2014 21:14, Fabio Estevam wrote:
<snip>
+int misc_init_r(void) +{
Do we need misc_init ? Why not to use power_init_board() ? It is already defined as weak.
OT: I confess I am quite losing which functions are already foreseen. Most of them have the prefix board_, and they are easier to find. However, power_init_board() does not follow the same rule.
Hi Fabio,
Yes, I think Stefano is correct in that power_init_board() is the right place to put pmic init. He asked me to do that to my recent series as well and it worked fine. Note that in that series I do have a patch that will rename pmic_init() for the PFUZE100 to power_pfuze100_init() [1]. I'm hoping that will make it in soon.
Tim
[1] - http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/184853/focus=184851