
9 Oct
2012
9 Oct
'12
2:06 p.m.
Hi Stefano,
On 09/10/2012 12:30, Lukasz Majewski wrote:
This enforces my comment in previous patch. We have two CONFIG_ options, both must be turned on. So at least one is redundant.IMHO you can drop both of them if power_board_init() is declared weak.
I can define power_board_init() as __weak if you are OK with this :-).
IMHO yes, and call it board_power_init(). We know directly from the name that board_* function are often declared weak.
Ok, I will prepare v3 of this patch set.
In this case for sure CONFIG_POWER_INIT could be removed.
I agree that CONFIG_PMIC can be removed from this piece of code.
Best regards, Stefano Babic
--
Best regards,
Lukasz Majewski
Samsung Poland R&D Center | Linux Platform Group