Re: [U-Boot] [PATCH 01/10] arm: vf610: fix anadig register struct

2013-09-30 Thread Marcel Ziswiler
On 09/17/2013 03:27 PM, Otavio Salvador wrote: I agree with the way to fix it but it is a little bit hard to get it is a 'reserved'; we used reserved_ to make it more explicit. Take a look at http://git.denx.de/?p=u-boot.git;a=blob;f=arch/arm/include/asm/arch-mxs/regs-power-mx28.h;h=9528e3ce9ad

Re: [U-Boot] [PATCH 01/10] arm: vf610: fix anadig register struct

2013-09-17 Thread Otavio Salvador
On Tue, Sep 17, 2013 at 7:45 AM, Marcel Ziswiler wrote: > The anadig_reg structure started at the wrong offset (fixed by adding > resvA[4]), was missing some reserved field required for alignment > purpose (resvB[3] between pll4_denom and pll6_ctrl) and further > contained too short a reserved fie

[U-Boot] [PATCH 01/10] arm: vf610: fix anadig register struct

2013-09-17 Thread Marcel Ziswiler
The anadig_reg structure started at the wrong offset (fixed by adding resvA[4]), was missing some reserved field required for alignment purpose (resvB[3] between pll4_denom and pll6_ctrl) and further contained too short a reserved field causing further miss-alignment (resv10[7]). Discovered and te