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 13:57:13 -0700 [thread overview]
Message-ID: <501846A9.1010505@ti.com> (raw)
In-Reply-To: <CAJ7qFSf=HEpWrJ6hgZm5HbRLfk9KtcWJddMaBHb5T+8aOQPWQQ@mail.gmail.com>
On 07/31/2012 08:46 AM, R, Sricharan wrote:
> Correct.
> DRAM_ADDR_SPACE_END should be 0xFFFFFFFF for OMAP5.
OK, I will just drop this.
>
> Thanks,
> Sricharan
>
> On Tue, Jul 31, 2012 at 9:12 PM, Tom Rini <trini@ti.com> wrote:
>> 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 20:57 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
2012-07-31 15:46 ` R, Sricharan
2012-07-31 20:57 ` Tom Rini [this message]
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=501846A9.1010505@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