From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hadli, Manjunath <manjunath.hadli@ti.com>
Date: Wed, 19 Oct 2011 18:08:29 +0530
Subject: [U-Boot] [PATCH 2/2] dm646x: pass board revision info to kernel
In-Reply-To: <4E7C9893.8010505@mvista.com>
References: <1316608831-31016-1-git-send-email-nagabhushana.netagunte@ti.com>
	<1316608831-31016-3-git-send-email-nagabhushana.netagunte@ti.com>
	<4E7C9893.8010505@mvista.com>
Message-ID: <B85A65D85D7EB246BE421B3FB0FBB5930328A9BD1B@dbde02.ent.ti.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

Hi Sergei,

Thanks for the comments.

On Fri, Sep 23, 2011 at 20:02:51, Sergei Shtylyov wrote:
> Hello.
> 
> On 09/21/2011 04:40 PM, nagabhushana.netagunte at ti.com wrote:
> 
> > From: Nagabhushana Netagunte <nagabhushana.netagunte@ti.com>
> 
> > add a function in board file to pass board revision info to kernel. 
> > Revision number 0 and 1 are passed in case of DM6467 and DM6467T 
> > respectively.
> 
> > Signed-off-by: Nagabhushana Netagunte <nagabhushana.netagunte@ti.com>
> > ---
> >   board/davinci/dm6467evm/dm6467evm.c  |   19 +++++++++++++++++++
> >   include/configs/davinci_dm6467Tevm.h |    2 ++
> >   include/configs/davinci_dm6467evm.h  |    1 +
> >   3 files changed, 22 insertions(+), 0 deletions(-)
> 
> > diff --git a/board/davinci/dm6467evm/dm6467evm.c 
> > b/board/davinci/dm6467evm/dm6467evm.c
> > index 1a01c3c..b582e97 100644
> > --- a/board/davinci/dm6467evm/dm6467evm.c
> > +++ b/board/davinci/dm6467evm/dm6467evm.c
> > @@ -24,6 +24,25 @@
> >
> >   DECLARE_GLOBAL_DATA_PTR;
> >
> > +#define REV_DM6446EVM		0
> > +#define REV_DM6467TEVM		1
> > +/*
> > + * get_board_rev() - setup to pass kernel board revision information
> > + * Returns:
> > + * bit[0-3]	System clock frequency
> > + * 0000b	- 27 MHz
> > + * 0001b	- 33 MHz
> > + */
> > +u32 get_board_rev(void)
> > +{
> > +
> > +#ifdef DAVINCI_DM6467TEVM
> > +	return REV_DM6467TEVM;
> > +#endif
> 
>     Maybe #else here?..
Ok.
> 
> > +
> > +	return REV_DM6446EVM;
> 
>     ... and #endif here?
Sure.

Regards,
--Manju
> 
> > +}
> > +
> >   int board_init(void)
> >   {
> >   	gd->bd->bi_arch_number = MACH_TYPE_DAVINCI_DM6467_EVM;
> 
> WBR, Sergei
>