All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Bießmann" <andreas.devel@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: at91: support for the Calao USB-A9263 board (based on AT91SAM9263)
Date: Mon, 04 Nov 2013 10:36:44 +0100	[thread overview]
Message-ID: <52776AAC.9020307@gmail.com> (raw)
In-Reply-To: <5274004C.3040908@gmail.com>

Dear Mateusz Kulikowski,

On 11/01/2013 08:26 PM, Mateusz Kulikowski wrote:
> On 29.10.2013 10:12, Andreas Bie?mann wrote:
>>>> +
>>>> +/*define the area offsets*/
>>>> +dataflash_protect_t area_list[NB_DATAFLASH_AREA] = {
>>>> +    {0x00000000, 0x00001FFF, FLAG_PROTECT_SET, 0, "Bootstrap"},
>>>> +    {0x00002000, 0x00003FFF, FLAG_PROTECT_CLEAR, 0, "Environment"},
>>>> +    {0x00004000, 0xFFFFFFFF, FLAG_PROTECT_SET, 0, "U-Boot"},
>>>> +};
>>
>> I would really like to see that removed. The dataflash thing is another
>> part on my todo list. The common mtd code can handle these devices too
>> (see for example at91sam9n12ek, at91sam9x5ek, ethernut5 sam5d3xek and
>> top9000). I did some short testing on that for at91sam9263ek (at home,
>> never sent to the list) and it worked basically. Will see if I find it
>> this evening, clean up and post it this evening CET.
> 
> What kind of Dataflash have you used?
> 
> Dataflash on Calao (at45db021b) does not have read-id command.
> It also has different command set than more recent version (*021d).
> It may be tricky to probe and operate it without modifying SPI drivers.

Sorry, haven't realized that. I have to check which dataflash was
attached to that board.

> For now I will move partition code to "main" board file;
> If I figure out how to handle that Dataflash - I'll update the code.

fine!

<snip>

> As for the memory concerns, I did little test:
> - enabled debugs in board_[fr].c (they print nicely memory layout),
> - Filled u-boot binary with commands/garbage to reach max size supported
> by AT91bootstrap (200704)
> - Flashed, booted the board. Below - log (parts):
> <log>
> U-Boot 2013.10-00122-g509dca7-dirty (Nov 01 2013 - 20:09:46)
> 
> U-Boot code: 23F00000 -> 23F2C570  BSS: -> 23F31498

Your code uses 23f00000 up to 23f31498, is 197.15 KiB

> CPU: AT91SAM9263
> Crystal frequency:       12 MHz
> CPU clock        :      180 MHz
> Master clock     :       90 MHz
> DRAM:  Monitor len: 00031498
> Ram size: 04000000
> Ram top: 24000000
> TLB table from 23ff0000 to 23ff4000
> Reserving 197k for U-Boot at: 23fbe000
> Reserving 152k for malloc() at: 23f98000
> Reserving 84 Bytes for Board Info at: 23f97fac
> Reserving 176 Bytes for Global Data at: 23f97efc

The relocated environment will use the top of RAM down to 23f97efc

> 
> RAM Configuration:
> Bank #0: 20000000
> DRAM:  64 MiB
> New Stack Pointer is: 23f97ed0
> Relocation Offset is: 000be000
> Relocating to 23fbe000, new gd at 23f97efc, sp at 23f97ed0
> image_copy: 23f00000 23f2c570 <- @setup_reloc()
> image_copy: 23fbe000 23fea570 <- @board_init_r()
> WARNING: Caches not enabled
> Now running in RAM - U-Boot at: 23fbe000
> </log>
> 
> If I understand correctly, this means that we have plenty of space
> between start of stack, and end of non-relocated U-Boot (.

_plenty_ of space is not true here (in my understanding), but 410 KiB
will be enough ;)

<snip>

Best regards

Andreas Bie?mann

  reply	other threads:[~2013-11-04  9:36 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-27 19:34 [U-Boot] [PATCH] arm: at91: support for the Calao USB-A9263 board (based on AT91SAM9263) Mateusz Kulikowski
2013-10-28  4:57 ` Bo Shen
2013-10-28 21:30   ` Mateusz Kulikowski
2013-10-29  5:20     ` Heiko Schocher
2013-10-30 18:03       ` Mateusz Kulikowski
2013-10-29  5:24     ` Bo Shen
2013-10-29  5:54       ` Heiko Schocher
2013-10-30 18:17       ` Mateusz Kulikowski
2013-10-29  6:26     ` Andreas Bießmann
2013-10-29  6:35       ` Heiko Schocher
2013-10-29  9:12   ` Andreas Bießmann
2013-11-01 19:26     ` Mateusz Kulikowski
2013-11-04  9:36       ` Andreas Bießmann [this message]
2013-11-01 23:19     ` [U-Boot] [PATCH v2] " Mateusz Kulikowski
2013-11-11 11:03       ` Andreas Bießmann
2013-11-12  8:53         ` Heiko Schocher
2013-11-28 21:29         ` Mateusz Kulikowski
2013-11-29 11:31           ` Andreas Bießmann
2013-12-02 22:30             ` [U-Boot] [PATCH v3] " Mateusz Kulikowski
2013-12-09 12:39               ` [U-Boot] [U-Boot, " Andreas Bießmann

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=52776AAC.9020307@gmail.com \
    --to=andreas.devel@googlemail.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.