From mboxrd@z Thu Jan  1 00:00:00 1970
From: Pavel Machek <pavel@denx.de>
Date: Mon, 23 Nov 2015 13:25:39 +0100
Subject: [U-Boot] [PATCH 04/10] ARM: socfpga: arria10: add stub sdram
 init for Arria10
In-Reply-To: <1447968947-8395-5-git-send-email-dinguyen@opensource.altera.com>
References: <1447968947-8395-1-git-send-email-dinguyen@opensource.altera.com>
 <1447968947-8395-5-git-send-email-dinguyen@opensource.altera.com>
Message-ID: <20151123122539.GB5361@amd>
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!

> +union dramaddrw_reg {
> +	struct {
> +		u32 cfg_col_addr_width:5;
> +		u32 cfg_row_addr_width:5;
> +		u32 cfg_bank_addr_width:4;
> +		u32 cfg_bank_group_addr_width:2;
> +		u32 cfg_cs_addr_width:3;
> +		u32 reserved:13;
> +	};
> +	u32 word;
> +};

Umm. Are you sure this is good idea? Is gcc allowed  to do something
funny and reorder bitfields in unexpected way?
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html