Hi Joel, >> /* Lot of common early inits here */ >> astbmc_early_init(); >> + prd_init(); > > Is there any reason not to put it in astbmc_early_init? I was thinking about doing this, but it may be possible that we have an astbmc machine without PRD support; the two aren't necessarily linked. Cheers, Jeremy