RE: [PATCH 0/2] usb: dwc2: Revert patches causing problems

From: Paul Zimmerman
Date: Thu Sep 11 2014 - 16:11:28 EST


> From: Paul Zimmerman
> Sent: Thursday, September 11, 2014 12:11 PM
>
> > From: Robert Baldyga [mailto:r.baldyga@xxxxxxxxxxx]
> > Sent: Thursday, September 11, 2014 5:53 AM
> >
> > These two patches breaks dwc2 driver. The first one causes build break,
> > the second breaks gadget at Samsung platforms.
> >
> > Best regards
> > Robert Baldyga
> >
> > Robert Baldyga (2):
> > Revert "usb: dwc2: Update Kconfig to support dual-role"
> > Revert "usb: dwc2: move "samsung,s3c6400-hsotg" into common platform"
> >
> > drivers/usb/dwc2/Kconfig | 63 +++++++++++++++++++--------------------------
> > drivers/usb/dwc2/Makefile | 21 +++++++--------
> > drivers/usb/dwc2/gadget.c | 1 +
> > drivers/usb/dwc2/platform.c | 1 -
> > 4 files changed, 38 insertions(+), 48 deletions(-)
>
> Yep, I can confirm the build breakage. Greg, we should revert these two
> before you send your update for the next -rc, if it's not too late.
>
> Dinh, before resending the series, please test that each individual
> patch does not break the build.

FYI, I see the build breakage when using ARM multi_v7_defconfig,
with the mass-storage gadget enabled and dwc2 enabled for
peripheral-only mode:

CC init/version.o
LD init/built-in.o
drivers/built-in.o: In function `dwc2_hc_set_even_odd_frame':
:(.text+0x21ca0c): undefined reference to `dwc2_hcd_get_frame_number'
drivers/built-in.o: In function `dwc2_handle_common_intr':
:(.text+0x21f5dc): undefined reference to `dwc2_hcd_start'
:(.text+0x21f660): undefined reference to `dwc2_hcd_start'
:(.text+0x21f7c4): undefined reference to `dwc2_hcd_start'
:(.text+0x21f974): undefined reference to `dwc2_hcd_disconnect'

--
Paul

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/