[U-Boot] [PATCH 05/10] am33xx: add ti814x specific register definitions
Peter Korsgaard
jacmet at sunsite.dk
Sun Feb 17 21:28:33 CET 2013
>>>>> "Matt" == Matt Porter <mporter at ti.com> writes:
Matt> Support the ti814x specific register definitions within
Matt> arch-am33xx.
Matt> Signed-off-by: Matt Porter <mporter at ti.com>
Matt> ---
Matt> arch/arm/cpu/armv7/am33xx/sys_info.c | 3 +++
Matt> arch/arm/include/asm/arch-am33xx/cpu.h | 11 +++++----
Matt> arch/arm/include/asm/arch-am33xx/hardware.h | 32 +++++++++++++++++++++++++++
Matt> arch/arm/include/asm/arch-am33xx/omap.h | 7 ++++++
Matt> arch/arm/include/asm/arch-am33xx/spl.h | 5 +++++
Matt> 5 files changed, 54 insertions(+), 4 deletions(-)
Matt> diff --git a/arch/arm/include/asm/arch-am33xx/hardware.h b/arch/arm/include/asm/arch-am33xx/hardware.h
Matt> index 41ab2c0..786c159 100644
Matt> --- a/arch/arm/include/asm/arch-am33xx/hardware.h
Matt> +++ b/arch/arm/include/asm/arch-am33xx/hardware.h
Matt> @@ -20,9 +20,14 @@
Matt> #define __AM33XX_HARDWARE_H
Matt> #include <asm/arch/omap.h>
Matt> +#include <config.h>
Quite some of the base addresses are similar, but I wonder if it
wouldn't be cleaner to simply have a hardware-am33xx.h /
hardware-ti814x.h instead of all these ifdef / elif?
Matt> /* Control Module Base Address */
Matt> +#ifdef CONFIG_AM33XX
Matt> #define CTRL_BASE 0x44E10000
Matt> #define CTRL_DEVICE_BASE 0x44E10600
Matt> +#elif defined(CONFIG_TI814X)
Matt> +#define CTRL_BASE 0x48140000
Matt> +#endif
No CTRL_DEVICE_BASE on ti814x?
Matt> --- a/arch/arm/include/asm/arch-am33xx/spl.h
Matt> +++ b/arch/arm/include/asm/arch-am33xx/spl.h
Matt> @@ -25,8 +25,13 @@
Matt> #define BOOT_DEVICE_XIP 2
Matt> #define BOOT_DEVICE_NAND 5
Matt> +#ifdef CONFIG_AM33XX
Matt> #define BOOT_DEVICE_MMC1 8
Matt> #define BOOT_DEVICE_MMC2 9 /* eMMC or daughter card */
Matt> +#elif defined(CONFIG_TI814X)
Matt> +#define BOOT_DEVICE_MMC1 9
Matt> +#define BOOT_DEVICE_MMC2 8 /* ROM only supports 2nd instance */
Argh! Couldn't we just swap the meaning of mmc1/mmc2 or would that be
too confusing?
Matt> +#endif
Matt> #define BOOT_DEVICE_SPI 11
Matt> #define BOOT_DEVICE_UART 65
Matt> #define BOOT_DEVICE_CPGMAC 70
Matt> --
Matt> 1.7.9.5
Matt> _______________________________________________
Matt> U-Boot mailing list
Matt> U-Boot at lists.denx.de
Matt> http://lists.denx.de/mailman/listinfo/u-boot
--
Bye, Peter Korsgaard
More information about the U-Boot
mailing list