From: Lokesh Vutla <lokeshvutla@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 01/14] ARM: AM43xx: Update the base addresses of modules
Date: Mon, 25 Nov 2013 14:56:22 +0530 [thread overview]
Message-ID: <529317BE.4090909@ti.com> (raw)
In-Reply-To: <CANacCWydPiieQzG=OZk2nU-54VdrG70rx0dRksuhhcEC-PY7eQ@mail.gmail.com>
On Friday 22 November 2013 01:50 AM, Vaibhav Bedia wrote:
> Hi Lokesh,
>
> On Thu, Nov 21, 2013 at 1:18 AM, Lokesh Vutla <lokeshvutla@ti.com> wrote:
> [...]
>> +#define CM_DPLL 0x44DF4200
>> +#define CM_RTC 0x44df8500
>>
>
> nit: CM_RTC address should be in caps for the sake of consistency
OK ll change it.
>
>> #define PRM_RSTCTRL (PRCM_BASE + 0x4000)
>> #define PRM_RSTST (PRM_RSTCTRL + 4)
>> diff --git a/arch/arm/include/asm/arch-am33xx/hardware_ti814x.h b/arch/arm/include/asm/arch-am33xx/hardware_ti814x.h
>> index 4509a23..9080b6f 100644
>> --- a/arch/arm/include/asm/arch-am33xx/hardware_ti814x.h
>> +++ b/arch/arm/include/asm/arch-am33xx/hardware_ti814x.h
>> @@ -27,6 +27,8 @@
>> #define PRCM_BASE 0x48180000
>> #define CM_PER 0x44E00000
>> #define CM_WKUP 0x44E00400
>> +#define CM_DPLL 0x44E00500
>> +#define CM_RTC 0x44E00800
>>
>
> Looking at these address i strongly suspect the CM_XXX ones are wrong
> for the TI81xx
> family. Does this really match with the TRM?
Yes, you are correct. TI814x code doesn't use these defines at all.
Ill drop these.
Thanks
Lokesh
>
> Regards,
> Vaibhav
>
next prev parent reply other threads:[~2013-11-25 9:26 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-21 6:18 [U-Boot] [PATCH V2 00/14] ARM: AM43xx: Update support for AM4372 SoC Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 01/14] ARM: AM43xx: Update the base addresses of modules Lokesh Vutla
2013-11-21 20:20 ` Vaibhav Bedia
2013-11-25 9:26 ` Lokesh Vutla [this message]
2013-11-21 6:18 ` [U-Boot] [PATCH V2 02/14] ARM: AM43xx: Adapt to ti_armv7_common.h config file Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 03/14] ARM: AM43xx: Add L2 Support Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 04/14] ARM: AM43xx: Add extra ENV settings Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 05/14] ARM: AM43XX: board: add support for reading onboard EEPROM Lokesh Vutla
2013-11-21 20:26 ` Vaibhav Bedia
2013-11-25 4:46 ` Lokesh Vutla
2013-11-26 23:49 ` Vaibhav Bedia
2013-11-21 6:18 ` [U-Boot] [PATCH V2 06/14] ARM: AM43XX: Add CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG support Lokesh Vutla
2013-11-21 20:28 ` Vaibhav Bedia
2013-11-25 4:48 ` Lokesh Vutla
2013-11-26 23:51 ` Vaibhav Bedia
2013-11-21 6:18 ` [U-Boot] [PATCH V2 07/14] ARM: AM43xx: Select clk source for Timer2 Lokesh Vutla
2013-11-21 20:31 ` Vaibhav Bedia
2013-11-25 4:53 ` Lokesh Vutla
2013-11-26 23:56 ` Vaibhav Bedia
2013-11-21 6:18 ` [U-Boot] [PATCH V2 08/14] ARM: AM43xx: Update Current Booting devices list Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 09/14] ARM: AM43xx: mux: Update mux data Lokesh Vutla
2013-11-21 20:34 ` Vaibhav Bedia
2013-11-25 4:59 ` Lokesh Vutla
2013-11-26 23:58 ` Vaibhav Bedia
2013-11-21 6:18 ` [U-Boot] [PATCH V2 10/14] ARM: AM43xx: clocks: Update DPLL details for EPOS EVM Lokesh Vutla
2013-11-21 20:37 ` Vaibhav Bedia
2013-11-25 5:08 ` Lokesh Vutla
2013-11-27 0:06 ` Vaibhav Bedia
2013-11-27 6:58 ` Lokesh Vutla
2013-11-27 22:48 ` Vaibhav Bedia
2013-12-02 3:53 ` Lokesh Vutla
2013-12-04 3:20 ` Vaibhav Bedia
2013-12-04 17:39 ` Sekhar Nori
2013-11-21 6:18 ` [U-Boot] [PATCH V2 11/14] ARM: AM43xx: clocks: Add DPLL data for GP EVM Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 12/14] ARM: AM43xx: EPOS_EVM: Add support for LPDDR2 Lokesh Vutla
2013-11-21 20:46 ` Vaibhav Bedia
2013-11-25 5:13 ` Lokesh Vutla
2013-11-27 0:12 ` Vaibhav Bedia
2013-11-27 4:48 ` Lokesh Vutla
2013-11-21 6:18 ` [U-Boot] [PATCH V2 13/14] ARM: AM43xx: GP_EVM: Add support for DDR3 Lokesh Vutla
2013-11-21 20:52 ` Vaibhav Bedia
2013-11-25 5:18 ` Lokesh Vutla
2013-11-27 0:17 ` Vaibhav Bedia
2013-11-27 9:34 ` Lokesh Vutla
2013-11-27 23:03 ` Vaibhav Bedia
2013-12-02 4:21 ` Lokesh Vutla
2013-12-04 3:24 ` Vaibhav Bedia
2013-11-21 6:18 ` [U-Boot] [PATCH V2 14/14] ARM: AM43xx: Add Maintainer Lokesh Vutla
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=529317BE.4090909@ti.com \
--to=lokeshvutla@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).