[U-Boot] [PATCH v2] common: add a prototype for mach_cpu_init()

[Masahiro Yamada]

2018-03-07 23:45 GMT+09:00 Tom Rini <trini at konsulko.com>:
> On Wed, Mar 07, 2018 at 03:28:20PM +0100, Patrick Delaunay wrote:
>
>> avoid warning: no previous prototype for ‘mach_cpu_init’
>>
>> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
>
> Reviewed-by: Tom Rini <trini at konsulko.com>
>
> --


People tend to put all sort of misc thingy into common.h
but this is one of the ugliest parts in U-Boot.

Most of files parse <common.h> that contains
unrelated / unnecessary defines.
Please realize this madness.

I have tried to slim it down several times,
but people have added more and more.  So this task never ends.

How about splitting out platform init hooks?
(<init.h> or anything suitable is OK.)

Also, I'd like to ban new additions to <common.h>
like we do for config_whitelist.txt
(for example, record the number of lines of <common.h>
then build fails if somebody increases it.)




-- 
Best Regards
Masahiro Yamada