Hi Balbi, I saw you applied "e9c07984979d usb: dwc2: Force mode optimizations" patch to your testing/next. Please re-apply this commit. On 2/15/2018 6:37 PM, Grigor Tovmasyan wrote: > Hi Balbi > > On 2/15/2018 17:34, Felipe Balbi wrote: >> Hi, >> >> kbuild test robot <fengguang.wu@xxxxxxxxx> writes: >>> tree: https://urldefense.proofpoint.com/v2/url?u=https-3A__git.kernel.org_pub_scm_linux_kernel_git_balbi_usb.git&d=DwIFAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=K1ULVL1slpLXpMJJlAXSOxws4tRq0IkTBqxDkyW2hUQ&m=2Hq2CZMELUidsi9x-Kqf4yuAh4IoxCX3Th1gZtmyZmc&s=Kw25jbj17VVP-cEEsCvpsfmS6NIuM8JAI8vfOtC2Nt4&e= testing/next >>> head: 65223a0ed97c8c4b18c4899653745058f87d67e3 >>> commit: 4caf1fe8fb537388810a4c8fecfa5324b26f80ea [45/57] usb: dwc2: Make dwc2_force_mode() static >>> config: i386-randconfig-x072-201806 (attached as .config) >>> compiler: gcc-7 (Debian 7.3.0-1) 7.3.0 >>> reproduce: >>> git checkout 4caf1fe8fb537388810a4c8fecfa5324b26f80ea >>> # save the attached .config to linux build tree >>> make ARCH=i386 >>> >>> All errors (new ones prefixed by >>): >>> >>>>> drivers/usb/dwc2/core.c:438:13: error: static declaration of 'dwc2_clear_force_mode' follows non-static declaration >>> static void dwc2_clear_force_mode(struct dwc2_hsotg *hsotg) >>> ^~~~~~~~~~~~~~~~~~~~~ >>> In file included from drivers/usb/dwc2/core.c:57:0: >>> drivers/usb/dwc2/core.h:1101:6: note: previous declaration of 'dwc2_clear_force_mode' was here >>> void dwc2_clear_force_mode(struct dwc2_hsotg *hsotg); >>> ^~~~~~~~~~~~~~~~~~~~~ >>> >>> vim +/dwc2_clear_force_mode +438 drivers/usb/dwc2/core.c >>> >>> 428 >>> 429 /** >>> 430 * dwc2_clear_force_mode() - Clears the force mode bits. >>> 431 * >>> 432 * After clearing the bits, wait up to 100 ms to account for any >>> 433 * potential IDDIG filter delay. We can't know if we expect this delay >>> 434 * or not because the value of the connector ID status is affected by >>> 435 * the force mode. We only need to call this once during probe if >>> 436 * dr_mode == OTG. >>> 437 */ >>> > 438 static void dwc2_clear_force_mode(struct dwc2_hsotg *hsotg) >> patch dropped. >> > This patch have a dependency from "[PATCH 6/6] usb: dwc2: Force mode" > ("[PATCH 0/6] usb: dwc2: minor fixes" patches) which I missed. > > I will rebase > [PATCH 5/6] usb: dwc2: eliminate irq parameter from dwc2_gadget_init > [PATCH 6/6] usb: dwc2: Force mode > > Please apply this patch after applying that two patches. > > Thanks, Grigor. > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
