[PATCH v1] ppc44x:PHY fixup for USB on canyonlands board
Benjamin Herrenschmidt
benh at kernel.crashing.org
Wed Nov 24 08:10:21 EST 2010
> > + setbits8(&bcsr[7], BCSR_USB_EN);
> > + udelay(100000);
> > +
> > + clrbits8(&bcsr[7], BCSR_USB_EN);
> > + udelay(100000);
>
> Thats a total bootup delay of 200ms. Is this really needed?
In addition, so large delays should use msleep() if possible (depends
how early we are here).
Cheers,
Ben,
> And I suggest to add a few comments to the code explaining why exactly you are
> setting/clearing the bits in the BCSR and the GPIO registers.
Seconded,
Cheers,
Ben.
More information about the Linuxppc-dev
mailing list