> > + /* Set LUPWAIT to be active low > and enabled */ > > + *mxmr = 0x08000000 | > MxMR_GPL_x4DIS; > > ...because we should now be using i/o accessors. Not entirely sure what you mean here. Should I be using __raw_writel, or writel, or ...? Thanks. Ron