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_where to make it more explicit. Take a look at

[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

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 mar...@ziswiler.com 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