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

Tom Rini trini at konsulko.com
Fri Mar 9 14:29:39 UTC 2018


On Thu, Mar 08, 2018 at 09:57:58AM +0900, Masahiro Yamada wrote:
> 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.)

OK, sure, lets move these and relevant bits over to init.h, or
init_helpers.h which already exists and is in a few of the appropriate
locations already.

> 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.)

Sure, thanks!

-- 
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/20180309/c7dcf43d/attachment.sig>


More information about the U-Boot mailing list