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 /arch/arm/include/asm/arch-mx6/clock.h | |
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 'arch/arm/include/asm/arch-mx6/clock.h')
-rw-r--r-- | arch/arm/include/asm/arch-mx6/clock.h | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/arch/arm/include/asm/arch-mx6/clock.h b/arch/arm/include/asm/arch-mx6/clock.h index f7760541a4..769917679e 100644 --- a/arch/arm/include/asm/arch-mx6/clock.h +++ b/arch/arm/include/asm/arch-mx6/clock.h @@ -7,7 +7,7 @@ #ifndef __ASM_ARCH_CLOCK_H #define __ASM_ARCH_CLOCK_H -#include <common.h> +#include <linux/types.h> #ifdef CONFIG_SYS_MX6_HCLK #define MXC_HCLK CONFIG_SYS_MX6_HCLK @@ -21,6 +21,8 @@ #define MXC_CLK32 32768 #endif +struct cmd_tbl_s; + enum mxc_clock { MXC_ARM_CLK = 0, MXC_PER_CLK, @@ -80,5 +82,6 @@ void enable_thermal_clk(void); void mxs_set_lcdclk(u32 base_addr, u32 freq); void select_ldb_di_clock_source(enum ldb_di_clock clk); void enable_eim_clk(unsigned char enable); -int do_mx6_showclocks(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); +int do_mx6_showclocks(struct cmd_tbl_s *cmdtp, int flag, int argc, + char *const argv[]); #endif /* __ASM_ARCH_CLOCK_H */ |