[U-Boot] [PATCH v4 2/2] ftsdmc021: add register definitions of ftsdmc021

[Wolfgang Denk <wd@denx.de>]

Dear Macpaul Lin,

In message <1303813524-28948-2-git-send-email-macpaul@andestech.com> you wrote:
> Support registers definitions of ftsdmc021 SDRAM controller.
> 
> Signed-off-by: Macpaul Lin <macpaul@andestech.com>
> ---
> Changes for v2:
>  - Add __ASSEMBLY__ protecton to register offset for supporting lowlevel_init
> Changes for v3:
>  - Patch: no change. Changed a mail server to resend this patch
> Changes for v4:
>  - Cleanup.
>  - Replace lines over 80 chararters with proper statement.
>  - Add some bit-field description about important registers.
>  - Replace the define of constant which is power of 2 to (ffs(x - 1))
> 
>  include/faraday/ftsdmc021.h |  151 +++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 151 insertions(+), 0 deletions(-)
>  create mode 100644 include/faraday/ftsdmc021.h
> 
> diff --git a/include/faraday/ftsdmc021.h b/include/faraday/ftsdmc021.h
> new file mode 100644
> index 0000000..483d991
> --- /dev/null
> +++ b/include/faraday/ftsdmc021.h
> @@ -0,0 +1,151 @@
> +/*
> + * (C) Copyright 2009 Faraday Technology
> + * Po-Yu Chuang <ratbert@faraday-tech.com>
> + *
> + * (C) Copyright 2011 Andes Technology Corp
> + * Macpaul Lin <macpaul@andestech.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> + */
> +
> +/*
> + * FTSDMC021 - SDRAM Controller
> + */
> +#ifndef __FTSDMC021_H
> +#define __FTSDMC021_H
> +
> +#ifndef __ASSEMBLY__
> +struct ftsdmc021 {
> +	unsigned int	tp1;		/* 0x00 - SDRAM Timing Parameter 1 */
> +	unsigned int	tp2;		/* 0x04 - SDRAM Timing Parameter 2 */
> +	unsigned int	cr1;		/* 0x08 - SDRAM Configuration Reg 1 */
> +	unsigned int	cr2;		/* 0x0c - SDRAM Configuration Reg 2 */
> +	unsigned int	bank0_bsr;	/* 0x10 - Ext. Bank Base/Size Reg 0 */
> +	unsigned int	bank1_bsr;	/* 0x14 - Ext. Bank Base/Size Reg 1 */
> +	unsigned int	bank2_bsr;	/* 0x18 - Ext. Bank Base/Size Reg 2 */
> +	unsigned int	bank3_bsr;	/* 0x1c - Ext. Bank Base/Size Reg 3 */
> +	unsigned int	bank4_bsr;	/* 0x20 - Ext. Bank Base/Size Reg 4 */
> +	unsigned int	bank5_bsr;	/* 0x24 - Ext. Bank Base/Size Reg 5 */
> +	unsigned int	bank6_bsr;	/* 0x28 - Ext. Bank Base/Size Reg 6 */
> +	unsigned int	bank7_bsr;	/* 0x2c - Ext. Bank Base/Size Reg 7 */
> +	unsigned int	ragr;		/* 0x30 - Read Arbitration Group Reg */
> +	unsigned int	frr;		/* 0x34 - Flush Request Register */
> +	unsigned int	ebisr;		/* 0x38 - EBI Support Register	*/
> +	unsigned int	RSVED[25];	/* 0x3c-0x9c - Reserved		*/

Sorry, but only macro definitions use ALL CAPS names.  Variables names
are all lower case.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Pray to God, but keep rowing to shore. - Russian Proverb