[PATCH] usb: musb: OMAP4430: Remove a redundant omap4430_phy_init call in usb_musb_init

Tony Lindgren tony at atomide.com
Mon Oct 3 14:46:42 EDT 2011


* Tony Lindgren <tony at atomide.com> [110930 10:28]:
> * Felipe Balbi <balbi at ti.com> [110928 23:35]:
> > On Tue, Sep 20, 2011 at 04:50:29PM +0800, Axel Lin wrote:
> > > Current code calls omap4430_phy_init() twice in usb_musb_init().
> > > Calling omap4430_phy_init() once is enough.
> > > This patch removes the first omap4430_phy_init() call, which using an
> > > uninitialized pointer as parameter.
> > > 
> > > This patch elimates below build warning:
> > > arch/arm/mach-omap2/usb-musb.c: In function 'usb_musb_init':
> > > arch/arm/mach-omap2/usb-musb.c:141: warning: 'dev' may be used uninitialized in this function
> > > 
> > > Signed-off-by: Axel Lin <axel.lin at gmail.com>
> > 
> > Acked-by: Felipe Balbi <balbi at ti.com>
> 
> Thanks, applying into fixes.

FYI, I'll update this patch to have also Bjarne's SOB to
this patch because of the earlier reference. Will still
use Axel's patch as it shows the compile warning.

Regards,

Tony

 
> Tony
> 
> > 
> > > ---
> > >  arch/arm/mach-omap2/usb-musb.c |    3 ---
> > >  1 files changed, 0 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/arch/arm/mach-omap2/usb-musb.c b/arch/arm/mach-omap2/usb-musb.c
> > > index a65145b..19e4dac 100644
> > > --- a/arch/arm/mach-omap2/usb-musb.c
> > > +++ b/arch/arm/mach-omap2/usb-musb.c
> > > @@ -137,9 +137,6 @@ void __init usb_musb_init(struct omap_musb_board_data *musb_board_data)
> > >  	musb_plat.mode = board_data->mode;
> > >  	musb_plat.extvbus = board_data->extvbus;
> > >  
> > > -	if (cpu_is_omap44xx())
> > > -		omap4430_phy_init(dev);
> > > -
> > >  	if (cpu_is_omap3517() || cpu_is_omap3505()) {
> > >  		oh_name = "am35x_otg_hs";
> > >  		name = "musb-am35x";
> > > -- 
> > > 1.7.4.1
> > > 
> > > 
> > > 
> > 
> > -- 
> > balbi
> 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



More information about the linux-arm-kernel mailing list