From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 3/3] Add support for the LaCie ED Mini V2 board
Date: Sat, 28 Nov 2009 23:21:04 +0100 [thread overview]
Message-ID: <4B11A250.3050205@free.fr> (raw)
In-Reply-To: <20091118222158.A05573F6C2@gemini.denx.de>
Wolfgang Denk a ?crit :
> Dear Albert Aribaud,
>
> In message <1258239796-21528-4-git-send-email-albert.aribaud@free.fr> you wrote:
>> This board is based on the Marvell Orion5x SoC.
>> Current support is limited to console and Flash.
>> Flash support uses LEGACY as the Macronix 29LV400
>> used on ED Mini V2 is not 100% CFI compliant.
>
> And the differences are really so big that we cannot tweak the CFI
> driver?
Apparently not.
The MX29LV400CB is 8-bit/16-bit and it configured on the ED Mini as
8-bit, but presents a pure 16-bit QRY (with zeroes between qry bytes)
which causes cfi_flash to mistakenly detect it as 16-bit wide, and thus
use 16-bit accesses at 16-bit addresses to read IDs and write
instructions, which of course does not work.
>> +int dram_init(void)
>> +{
>> + int i;
>> +
>> + for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
>> + gd->bd->bi_dram[i].start = orion5x_sdram_bar(i);
>> + gd->bd->bi_dram[i].size = orion5x_sdram_bs(i);
>> + }
>> + return 0;
>> +}
>
> Cannot you use get_ram_size() for auto-sizing and checking?
As this is SoC code, not board code, and as the SoC allows to have up to
four non-contiguous DRAM banks, I prefer to read the four DRAM base/size
registers.
Amicalement,
--
Albert.
next prev parent reply other threads:[~2009-11-28 22:21 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-14 23:03 [U-Boot] [PATCH V2 0/3] Support for Marvell Orion5x SoC and LaCie ED Mini V2 board Albert Aribaud
2009-11-14 23:03 ` [U-Boot] [PATCH V2 1/3] Initial support for Marvell Orion5x SoC Albert Aribaud
2009-11-14 23:03 ` [U-Boot] [PATCH V2 2/3] Add Orion5x support to 16550 serial device driver Albert Aribaud
2009-11-14 23:03 ` [U-Boot] [PATCH V2 3/3] Add support for the LaCie ED Mini V2 board Albert Aribaud
2009-11-16 9:05 ` Prafulla Wadaskar
2009-11-29 11:22 ` Albert ARIBAUD
2009-11-18 22:21 ` Wolfgang Denk
2009-11-19 7:51 ` Stefan Roese
2009-11-19 12:48 ` Albert ARIBAUD
2009-11-19 13:10 ` Stefan Roese
2009-11-28 22:21 ` Albert ARIBAUD [this message]
2009-11-29 12:56 ` Albert ARIBAUD
2009-12-05 0:35 ` Wolfgang Denk
2009-12-05 7:03 ` Albert ARIBAUD
2009-12-07 21:56 ` Wolfgang Denk
2009-12-11 15:51 ` Albert ARIBAUD
2009-12-02 8:31 ` [U-Boot] Mixed 8/16 bits NOR FLASH support (was: [PATCH V2 3/3] Add support for the LaCie ED Mini V2 board) Albert ARIBAUD
2009-12-02 9:15 ` Stefan Roese
2009-12-02 9:38 ` Wolfgang Denk
2009-12-02 11:07 ` [U-Boot] Mixed 8/16 bits NOR FLASH support Albert ARIBAUD
2009-12-02 10:57 ` Albert ARIBAUD
2009-11-18 22:17 ` [U-Boot] [PATCH V2 1/3] Initial support for Marvell Orion5x SoC Wolfgang Denk
2009-11-18 22:51 ` Albert ARIBAUD
2009-11-29 8:55 ` Albert ARIBAUD
2009-12-05 0:37 ` Wolfgang Denk
2009-12-23 6:07 ` Prafulla Wadaskar
2009-11-29 10:34 ` Albert ARIBAUD
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=4B11A250.3050205@free.fr \
--to=albert.aribaud@free.fr \
--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.