Re: [patch] ehci-omap: (mostly) cosmetic cleanups

[Tony Lindgren <tony@atomide.com>]

* Felipe Balbi <me@felipebalbi.com> [081003 01:02]:
> On Fri, Oct 03, 2008 at 12:03:33AM -0700, David Brownell wrote:
> > From: David Brownell <dbrownell@users.sourceforge.net>
> > 
> > Various cleanups to the OMAP EHCI bus glue:
> > 
> >  - Whitespace fixes.
> > 
> >  - Busy-wait loops should really have timeouts, but lacking
> >    that ... at least cpu_relax() in the body.
> > 
> >  - Debug messages should end with newlines, so they don't
> >    break the next message; should not embed newlines; and
> >    should otherwise be clean and correctly spelled.
> > 
> >  - The probe() method should have initialized the SBRN value,
> >    so the driver didn't report "USB 0.0 started".
> > 
> >  - The MODULE_ALIAS() was missing its "platform:" prefix;
> >    doing it right means the module could hotplug/coldplug.
> > 
> > No real functional change except for the hotplug/coldplug thing.
> 
> I actually sent this patch (besides SBRN init) way back in time but
> Vikram and Anand (??) asked to hold on that patch since some patches
> would come from TI for ehci and ohci.
> 
> Never saw those patches till now.

Pushing this today anyways, seems like some good fixes.

Tony