From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 01/17] omap3/omap4/omap5/am33xx: Use a common running_from_sdram function
Date: Tue, 31 Jul 2012 08:42:35 -0700 [thread overview]
Message-ID: <5017FCEB.1070806@ti.com> (raw)
In-Reply-To: <CAJ7qFSfm05=bRMpS1Jy3eUtz0R_RsCYo2W5P9oFqm7r0_J9xCQ@mail.gmail.com>
On 07/31/2012 08:27 AM, R, Sricharan wrote:
> Hi Tom,
>
> On Tue, Jul 31, 2012 at 8:43 PM, Tom Rini <trini@ti.com> wrote:
>> On 07/31/2012 01:33 AM, R, Sricharan wrote:
>>> Hi Tom,
>>> [snip..]
>>>> diff --git a/arch/arm/include/asm/arch-omap5/omap.h b/arch/arm/include/asm/arch-omap5/omap.h
>>>> index 7f05cb5..c697e0b 100644
>>>> --- a/arch/arm/include/asm/arch-omap5/omap.h
>>>> +++ b/arch/arm/include/asm/arch-omap5/omap.h
>>>> @@ -39,11 +39,6 @@
>>>> #define OMAP54XX_L4_WKUP_BASE 0x4Ae00000
>>>> #define OMAP54XX_L4_PER_BASE 0x48000000
>>>>
>>>> -#define OMAP54XX_DRAM_ADDR_SPACE_START 0x80000000
>>>> -#define OMAP54XX_DRAM_ADDR_SPACE_END 0xFFFFFFFF
>>>> -#define DRAM_ADDR_SPACE_START OMAP54XX_DRAM_ADDR_SPACE_START
>>>> -#define DRAM_ADDR_SPACE_END OMAP54XX_DRAM_ADDR_SPACE_END
>>>> -
>>> This is a problem for OMAP5, which has a trap section at 0xFF000000
>>> with in the sdram boundary. OMAP5 evm board has 2GB of memory from
>>> 0x80000000 - 0xFFFFFFFF. Size of the trap section should not be
>>> included in the
>>> total sdram size.
>>
>> But it's not sdram size. What happens when you're executing at the trap
>> section, or rather, where are you executing code from?
>
> When we execute at trap section address, the system aborts.
> EMIF returns a exception. This is to catch the unmapped tiler
> entries.
> So total size of sdram size calculated should subtract the size
> of trap section if that falls with in the sdram boundary,
> as in case of omap5. This is taken care in omap_sdram_size
> function. But with this change the trap section will go un-noticed.
So you're saying the problem is that 0xFF... needs to be included in
DRAM_ADDR_SPACE on omap5?
--
Tom
next prev parent reply other threads:[~2012-07-31 15:42 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-30 16:48 [U-Boot] [PATCH 0/17]: Improve am33xx support Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 01/17] omap3/omap4/omap5/am33xx: Use a common running_from_sdram function Tom Rini
2012-07-31 8:33 ` R, Sricharan
2012-07-31 15:13 ` Tom Rini
2012-07-31 15:27 ` R, Sricharan
2012-07-31 15:42 ` Tom Rini [this message]
2012-07-31 15:46 ` R, Sricharan
2012-07-31 20:57 ` Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 02/17] am335x_evm: Update config for common usage Tom Rini
2012-08-02 18:04 ` Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 03/17] am33xx: Remove DMM_BASE define Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 04/17] am33xx: Convert to using <asm/emif.h> to describe the EMIF Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 05/17] am33xx: Remove extra check in enable_ddr_clocks Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 06/17] am33xx: Bugfix to config_sdram() Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 07/17] am33xx: Rework DDR2 EMIF initalization slightly Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 08/17] am33xx: Make config_cmd_ctrl / config_ddr_data take const structs Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 09/17] am33xx: Pass to config_ddr the type of memory that is connected Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 10/17] am33xx: Move the call to ddr_pll_config, make it take the frequency Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 11/17] am33xx: Clean up unused DDR defines, prefix more with 'DDR2' Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 12/17] am335x: ddr_defs: Update EMIF parameters Tom Rini
2012-07-30 16:48 ` [U-Boot] [PATCH 13/17] am33xx: Document what we're doing with ddrctrl->ddrckectrl Tom Rini
2012-07-30 16:49 ` [U-Boot] [PATCH 14/17] am33xx: Turn a number of 'int' functions to 'void' Tom Rini
2012-07-30 16:49 ` [U-Boot] [PATCH 15/17] am33xx: Use emif_regs struct for storing initialization values Tom Rini
2012-07-30 16:49 ` [U-Boot] [PATCH 16/17] am33xx: Rework config_io_ctrl slightly Tom Rini
2012-07-30 16:49 ` [U-Boot] [PATCH 17/17] am33xx: Do not touch 'ratio1' fields Tom Rini
2012-07-30 19:29 ` [U-Boot] [PATCH 18/17] am33xx: Correct and clean up ddr_regs struct Tom Rini
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=5017FCEB.1070806@ti.com \
--to=trini@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