From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] usb: musb: OMAP4430: Remove a redundant omap4430_phy_init call in usb_musb_init Date: Mon, 3 Oct 2011 11:46:42 -0700 Message-ID: <20111003184642.GH6324@atomide.com> References: <1316508629.14514.2.camel@phoenix> <20110929070849.GA27987@legolas.emea.dhcp.ti.com> <20110930180121.GK6324@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20110930180121.GK6324@atomide.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Felipe Balbi , Bjarne Steinsbo Cc: Russell King , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Axel Lin , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: linux-omap@vger.kernel.org * Tony Lindgren [110930 10:28]: > * Felipe Balbi [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 > > > > Acked-by: Felipe Balbi > > 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@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Mon, 3 Oct 2011 11:46:42 -0700 Subject: [PATCH] usb: musb: OMAP4430: Remove a redundant omap4430_phy_init call in usb_musb_init In-Reply-To: <20110930180121.GK6324@atomide.com> References: <1316508629.14514.2.camel@phoenix> <20110929070849.GA27987@legolas.emea.dhcp.ti.com> <20110930180121.GK6324@atomide.com> Message-ID: <20111003184642.GH6324@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Tony Lindgren [110930 10:28]: > * Felipe Balbi [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 > > > > Acked-by: Felipe Balbi > > 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