On Thu, Apr 29, 2010 at 12:55 PM, Steve Sakoman <sako...@gmail.com> wrote: > On Thu, Apr 29, 2010 at 11:15 AM, Scott Wood <scottw...@freescale.com> wrote: >> The mandatory use of structs even with large reserved areas seems to invite >> and obscure such issues (and the comments don't help much, since nothing >> ensures they're accurate). > > Good catch! > > Agreed -- I really hate sparse structs like this since they just > invite this type of error when you fill in missing registers. > > I'll fix and resubmit the patch for further comment.
BTW, the reason I didn't catch this in testing is that the error would only show up for mmc channel 2, which is not available on Overo (it is used for the SDIO wifi interface). My testing was done on mmc1 and mmc2. Steve _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
