diff options
| author | Simon Glass <sjg@chromium.org> | 2020-05-10 20:39:55 +0300 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2020-05-18 21:54:24 +0300 |
| commit | c3dc39a2f85b16bf590789f7e283cd72275cd168 (patch) | |
| tree | fd07b448aa4192e1d4fede43e412d5fdfa584b82 /board/technexion | |
| parent | b79fdc76978515fba2c4f3b5b4f2e679c5ff1463 (diff) | |
| download | u-boot-c3dc39a2f85b16bf590789f7e283cd72275cd168.tar.xz | |
arm: Don't include common.h in header files
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@chromium.org>
Diffstat (limited to 'board/technexion')
| -rw-r--r-- | board/technexion/pico-imx6/spl.c | 1 | ||||
| -rw-r--r-- | board/technexion/pico-imx7d/spl.c | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/board/technexion/pico-imx6/spl.c b/board/technexion/pico-imx6/spl.c index 06ad0a8c32..bafe9ba6b7 100644 --- a/board/technexion/pico-imx6/spl.c +++ b/board/technexion/pico-imx6/spl.c @@ -6,6 +6,7 @@ * Fabio Estevam <festevam@gmail.com> */ +#include <common.h> #include <asm/arch/clock.h> #include <asm/arch/imx-regs.h> #include <asm/arch/iomux.h> diff --git a/board/technexion/pico-imx7d/spl.c b/board/technexion/pico-imx7d/spl.c index 6c432ca5a6..69db77412c 100644 --- a/board/technexion/pico-imx7d/spl.c +++ b/board/technexion/pico-imx7d/spl.c @@ -5,6 +5,7 @@ * Author: Richard Hu <richard.hu@technexion.com> */ +#include <common.h> #include <cpu_func.h> #include <asm/arch/clock.h> #include <asm/arch/imx-regs.h> |