summaryrefslogtreecommitdiff
path: root/arch/arm/include/asm/arch-rk3308
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2020-05-10 11:39:55 -0600
committerTom Rini <trini@konsulko.com>2020-05-18 14:54:24 -0400
commitc3dc39a2f85b16bf590789f7e283cd72275cd168 (patch)
treefd07b448aa4192e1d4fede43e412d5fdfa584b82 /arch/arm/include/asm/arch-rk3308
parentb79fdc76978515fba2c4f3b5b4f2e679c5ff1463 (diff)
downloadu-boot-c3dc39a2f85b16bf590789f7e283cd72275cd168.tar.gz
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 'arch/arm/include/asm/arch-rk3308')
-rw-r--r--arch/arm/include/asm/arch-rk3308/cru_rk3308.h2
-rw-r--r--arch/arm/include/asm/arch-rk3308/grf_rk3308.h2
2 files changed, 0 insertions, 4 deletions
diff --git a/arch/arm/include/asm/arch-rk3308/cru_rk3308.h b/arch/arm/include/asm/arch-rk3308/cru_rk3308.h
index a14b64cdb3..86c906bb0e 100644
--- a/arch/arm/include/asm/arch-rk3308/cru_rk3308.h
+++ b/arch/arm/include/asm/arch-rk3308/cru_rk3308.h
@@ -5,8 +5,6 @@
#ifndef _ASM_ARCH_CRU_RK3308_H
#define _ASM_ARCH_CRU_RK3308_H
-#include <common.h>
-
#define MHz 1000000
#define OSC_HZ (24 * MHz)
diff --git a/arch/arm/include/asm/arch-rk3308/grf_rk3308.h b/arch/arm/include/asm/arch-rk3308/grf_rk3308.h
index 3e68626d3e..a995bb950d 100644
--- a/arch/arm/include/asm/arch-rk3308/grf_rk3308.h
+++ b/arch/arm/include/asm/arch-rk3308/grf_rk3308.h
@@ -5,8 +5,6 @@
#ifndef _ASM_ARCH_GRF_rk3308_H
#define _ASM_ARCH_GRF_rk3308_H
-#include <common.h>
-
struct rk3308_grf {
unsigned int gpio0a_iomux;
unsigned int reserved0;