[PATCH v3 04/22] arm: Don't include common.h in header files

Tom Rini trini at konsulko.com
Tue May 19 19:32:11 CEST 2020


On Sun, May 10, 2020 at 11:39:55AM -0600, Simon Glass wrote:

> It is bad practice to include common.h in other header files since it can
> bring in any number of superfluous definitions. It implies that some C
> files don't include it and thus may be missing CONFIG options that are set
> up by that file. The C files should include these themselves.
> 
> Update some header files in arch/arm to drop this.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200519/0e2c8a7f/attachment.sig>


More information about the U-Boot mailing list