Jeff Garzik wrote: > Albert Lee wrote: > >> The first rmb() is to make sure bccrl is read before bccrlv for later >> (bccrl >= bccrlv) check since both reading the same memory address. > > > That's already guaranteed without the rmb(), AFAICS. > Hmm, thanks for the advice. Will remove both rmb()s. Will also remove the now unused (pll_clock < 0) check from pdc_hardware_init() and test/verify Mikael's patch when I go back to office tomorrow. -- albert - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html