From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 28 Mar 2013 17:52:47 -0400 Subject: [U-Boot] [PATCH 6/8] Add TI816X specific register definitions In-Reply-To: <874nfv2okn.fsf@dell.be.48ers.dk> References: <1364490881-25921-1-git-send-email-atenart@adeneo-embedded.com> <3465D313FDFB824F9A9C8CD24FA4F6BC0112A9F1@frontmail.adetel.com> <874nfv2okn.fsf@dell.be.48ers.dk> Message-ID: <20130328215247.GG30253@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, Mar 28, 2013 at 10:37:28PM +0100, Peter Korsgaard wrote: > >>>>> "TENART" == TENART Antoine writes: > > TENART> Also move some non common defintions from hardware.h > TENART> --- > TENART> arch/arm/include/asm/arch-am33xx/hardware.h | 7 +-- > TENART> arch/arm/include/asm/arch-am33xx/hardware_am33xx.h | 5 ++ > TENART> arch/arm/include/asm/arch-am33xx/hardware_ti814x.h | 5 ++ > TENART> arch/arm/include/asm/arch-am33xx/hardware_ti816x.h | 58 ++++++++++++++++++++ > TENART> 4 files changed, 70 insertions(+), 5 deletions(-) > TENART> create mode 100644 arch/arm/include/asm/arch-am33xx/hardware_ti816x.h > > .. > > TENART> +#define DDRPHY_0_CONFIG_BASE 0x48198000 > TENART> +#define DDRPHY_1_CONFIG_BASE 0x4819a000 > TENART> +#define DDRPHY_CONFIG_BASE ((emif == 0) ? DDRPHY_0_CONFIG_BASE:DDRPHY_1_CONFIG_BASE) > TENART> + > TENART> +/* CPSW Config space */ > TENART> +#define CPSW_MDIO_BASE 0x4A100800 > > ti816x doesn't have CPSW, so the name is a bit odd (the base address > matches the emac mdio module). The emac driver expects an > asm/arch/emac_defs.h header with a EMAC_MDIO_BASE_ADDR define (among > others). > > Perhaps just drop it from here as you're not adding emac support in this > series anyway. Agreed, and please check the whole series over for other unused defines. We shouldn't import things we aren't using. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: