[U-Boot] [U-Boot,v2] Convert CONFIG_BOARD_LATE_INIT to Kconfig

[Tom Rini]

On Mon, Aug 20, 2018 at 10:02:29AM +0200, Marek Vasut wrote:
> On 08/20/2018 04:42 AM, Tom Rini wrote:
> > On Sat, Aug 18, 2018 at 08:18:25PM +0300, Ramon Fried wrote:
> > 
> >> This converts the remaining board defined
> >> CONFIG_BOARD_LATE_INIT to Kconfig.
> >>
> >> Signed-off-by: Ramon Fried <ramon.fried at gmail.com>
> >> ---
> >> Changes in v2:
> >> * Removed config from Kconfig as it already exists on
> >>   common/Kconfig.
> >> * Changed the commit message.
> > 
> > OK, I went to apply this and I see where I sent you in the wrong
> > direction, sorry.  BOARD_LATE_INIT is something that the boards (or
> > SoCs) in question should be select'ing, which is what everything else
> > that uses it is doing.  Thanks!
> 
> Again, wouldn't it rather make sense to convert the board_late_init to
> __weak symbol instead and drop the Kconfig option ?

Despite being what I thought was the only fan of weak functions left,
no, this hook isn't nearly widely used enough to justify being used that
way.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180820/dace68fd/attachment.sig>