From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752923AbbDAJiF (ORCPT ); Wed, 1 Apr 2015 05:38:05 -0400 Received: from youngberry.canonical.com ([91.189.89.112]:43253 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752705AbbDAJh5 (ORCPT ); Wed, 1 Apr 2015 05:37:57 -0400 Date: Wed, 1 Apr 2015 10:37:52 +0100 From: Luis Henriques To: Kamal Mostafa Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, kernel-team@lists.ubuntu.com, Felipe Balbi , Arnd Bergmann Subject: Re: [PATCH 3.13.y-ckt 056/143] usb: musb: add omap-control dependency Message-ID: <20150401093752.GB20394@charon.olymp> References: <1427831308-1854-1-git-send-email-kamal@canonical.com> <1427831308-1854-57-git-send-email-kamal@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1427831308-1854-57-git-send-email-kamal@canonical.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 31, 2015 at 12:47:01PM -0700, Kamal Mostafa wrote: > 3.13.11-ckt18 -stable review patch. If anyone has any objections, please let me know. > Arnd reported a regression caused by this patch. It seems to be fixed by commit 1c390eb360c3 ("usb: musb: fix Kconfig regression"), also tagged for stable. However, it looks like this patch was actually dropped from all the stable trees as Arnd also pointed out that the original problem was relatively harmless. Cheers, -- Luís > ------------------ > > From: Arnd Bergmann > > commit fbba7db3990cb707ff91cd6507d53a0a730afe97 upstream. > > The omap musb front-end calls into the phy driver directly > instead of using a generic phy interface, which causes a link > error when the specific driver is not built-in: > > drivers/built-in.o: In function `omap2430_musb_disable': > usb/musb/omap2430.c:480: undefined reference to `omap_control_usb_set_mode' > drivers/built-in.o: In function `omap2430_musb_enable': > usb/musb/omap2430.c:466: undefined reference to `omap_control_usb_set_mode' > usb/musb/omap2430.c:447: undefined reference to `omap_control_usb_set_mode' > drivers/built-in.o: In function `omap_musb_set_mailbox': > usb/musb/omap2430.c:273: undefined reference to `omap_control_usb_set_mode' > usb/musb/omap2430.c:304: undefined reference to `omap_control_usb_set_mode' > drivers/built-in.o:(.debug_addr+0xbd9e0): more undefined references to `omap_control_usb_set_mode' follow > > This adds an explicit dependency. > > Signed-off-by: Arnd Bergmann > Fixes: ca784be36cc725 ("usb: start using the control module driver") > Signed-off-by: Felipe Balbi > Signed-off-by: Kamal Mostafa > --- > drivers/usb/musb/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig > index a70f46f..ddcbcdd 100644 > --- a/drivers/usb/musb/Kconfig > +++ b/drivers/usb/musb/Kconfig > @@ -74,7 +74,7 @@ config USB_MUSB_TUSB6010 > > config USB_MUSB_OMAP2PLUS > tristate "OMAP2430 and onwards" > - depends on ARCH_OMAP2PLUS && USB > + depends on ARCH_OMAP2PLUS && USB && OMAP_CONTROL_PHY > select GENERIC_PHY > > config USB_MUSB_AM35X > -- > 1.9.1 > > > -- > kernel-team mailing list > kernel-team@lists.ubuntu.com > https://lists.ubuntu.com/mailman/listinfo/kernel-team