From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Tue, 26 Jun 2012 14:47:53 +0200
Subject: [U-Boot] [PATCH 2/3] KW: Add kw_adjust_dram() weak function
In-Reply-To: <F766E4F80769BD478052FB6533FA745D1A2FDB39AD@SC-VEXCH4.marvell.com>
References: <1340712914-3193-1-git-send-email-marex@denx.de>
	<1340712914-3193-2-git-send-email-marex@denx.de>
	<F766E4F80769BD478052FB6533FA745D1A2FDB39AD@SC-VEXCH4.marvell.com>
Message-ID: <201206261447.53537.marex@denx.de>
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

Dear Prafulla Wadaskar,

> > -----Original Message-----
> > From: Marek Vasut [mailto:marex at denx.de]
> > Sent: 26 June 2012 17:45
> > To: u-boot at lists.denx.de
> > Cc: Marek Vasut; Prafulla Wadaskar; Wolfgang Denk
> > Subject: [PATCH 2/3] KW: Add kw_adjust_dram() weak function
> > 
> > This function shall allow a board to adjust DRAM parameters in case
> > there are multiple versions of the board with different DRAM sizes.
> > 
> > Signed-off-by: Marek Vasut <marex@denx.de>
> > Cc: Prafulla Wadaskar <prafulla@marvell.com>
> > Cc: Wolfgang Denk <wd@denx.de>
> > ---
> > 
> >  arch/arm/cpu/arm926ejs/kirkwood/dram.c |   11 +++++++++++
> 
> Instead I would like to suggest to add function name as
> __kw_post_kwbconfig()

what about using board_early_init_f() ?

> This typically addresses to reconfigure the default configuration done by
> kwbimage.cfg, so the scope is not limited to DRAM configuration.
> 
> Regards..
> Prafulla . . .

Best regards,
Marek Vasut