From mboxrd@z Thu Jan  1 00:00:00 1970
From: Gerlando Falauto <gerlando.falauto@keymile.com>
Date: Thu, 05 Jul 2012 17:44:49 +0200
Subject: [U-Boot] [PATCH 1/2] kirkwood: implement kw_sdram_bs_set()
In-Reply-To: <201207051738.14366.marex@denx.de>
References: <1341498492-2648-1-git-send-email-gerlando.falauto@keymile.com>
	<201207051738.14366.marex@denx.de>
Message-ID: <4FF5B671.4040602@keymile.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 07/05/2012 05:38 PM, Marek Vasut wrote:
> Dear Gerlando Falauto,
>
>> Some boards might be equipped with different SDRAM configurations.
>> When that is the case, CPU CS Window Size Register (CS[0]n Size) should
>> be set to the biggest value through board.cfg file; then its value
>> can be fixed at runtime according to the detected SDRAM size.
>>
>> Therefore, implement kw_sdram_bs_set(), to be called for instance within
>> board_early_init_f().
>>
> [..]
>
>> +/*
>> + * kw_sdram_bs_set - writes SDRAM Bank size
>> + */
>> +void kw_sdram_bs_set(enum memory_bank bank, u32 size)
>
> The enum is declared somewhere, right ?
Yes, in arch/arm/include/asm/arch-kirkwood/cpu.h.

It's the same enum also used by other functions {declared, defined} 
within the same {header, source} file, like:

u32 kw_sdram_bs(enum memory_bank bank)

Best regards,
Gerlando