[U-Boot] [RFC PATCH 3/3] tegra2: Add support for Paz00 (Toshiba AC100)

[Stephen Warren <swarren@nvidia.com>]

On 11/20/2011 10:39 AM, Marc Dietrich wrote:
> On Sunday 20 November 2011 13:10:08 Thierry Reding wrote:
>> * Stephen Warren wrote:
>>> The Toshiba AC100 (code-name Paz00m a/k/a Dynabook AZ) is an netbook
>>> derived from the NVIDIA Tegra Harmony reference board. It ships with
>>> Ubuntu, but is often repurposed to run Linux. This patch adds just
>>> enough support to get a U-Boot serial console, and the ability access
>>> built-in eMMC and the external SD slot.
...
>>> +#define TEGRA2_SYSMEM		"mem=384M at 0M nvmem=128M at 384M"
>>
>> Can we get rid of this? I think I remember some discussion about the same
>> topic going on the linux-tegra/devicetree mailing lists, where it was
>> decided that /mem-reserve/ entries should be dropped until some driver in
>> the mainline kernel was actually going to use this memory. Perhaps U-Boot
>> should take the same approach.
> 
> I also vote for this. Additionally, most users prefer mem=448M here. The 
> device was original ships with 448 MB user memory, which was later reduced to 
> 384 MB, maybe because of flash player requirements. I'm also not sure if nvmem 
> is needed at all (maybe for android original kernel only)

Sounds like a plan. I've done this, and also moved the files to
board/compal/paz00 as suggested elsewhere.

>>> +#define V_PROMPT		"Tegra2 (Paz00) # "
>>> +#define CONFIG_TEGRA2_BOARD_STRING	"NVIDIA Paz00"
>>> +
>>> +/* Board-specific serial config */
>>> +#define CONFIG_SERIAL_MULTI
>>> +#define CONFIG_TEGRA2_ENABLE_UARTA
>>> +#define CONFIG_SYS_NS16550_COM1		NV_PA_APB_UARTA_BASE
>>> +
>>> +#define CONFIG_MACH_TYPE		MACH_TYPE_PAZ00
>>> +/*
>>> + * The following should be 512MB, but I don't know how to set that
>>> value
>>> + * It doesn't matter, since this isn't really used any more
>>> + */
>>> +#define CONFIG_SYS_BOARD_ODMDATA	0x200d8011 /* lp1, 1GB */
> 
> I don't know what is the effect of this paramter, but paz00 has 512 MB. The 
> original valu was 0x8000c0085, AFAIK, which leaves most stuff on the default 
> value.

OK, I've switched to that value (where did you extract it from, BTW),
and after fixing a bug in
arch/arm/cpu/armv7/tegra2/board.c:query_sdram_size() (default==512M, not
1G), it all works.

-- 
nvpublic