From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:47790 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751449AbeEPJXA (ORCPT ); Wed, 16 May 2018 05:23:00 -0400 From: Kalle Valo To: Arend van Spriel Cc: linux-wireless@vger.kernel.org Subject: Re: [PATCH 4/8] mwifiex: support sysfs initiated device coredump References: <1526375691-31789-5-git-send-email-arend.vanspriel@broadcom.com> <201805160424.9rVs4AKw%fengguang.wu@intel.com> <5AFBF1E5.7000505@broadcom.com> Date: Wed, 16 May 2018 12:22:55 +0300 In-Reply-To: <5AFBF1E5.7000505@broadcom.com> (Arend van Spriel's message of "Wed, 16 May 2018 10:55:01 +0200") Message-ID: <87eficlztc.fsf@kamboji.qca.qualcomm.com> (sfid-20180516_112304_848891_7A41E724) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-wireless-owner@vger.kernel.org List-ID: Arend van Spriel writes: > On 5/15/2018 10:22 PM, kbuild test robot wrote: >> Hi Arend, >> >> I love your patch! Yet something to improve: >> >> [auto build test ERROR on next-20180515] >> [cannot apply to wireless-drivers-next/master wireless-drivers/master bluetooth-next/master v4.17-rc5 v4.17-rc4 v4.17-rc3 v4.17-rc5] >> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] >> >> url: https://github.com/0day-ci/linux/commits/Arend-van-Spriel/brcmfmac-coredump-functionality-and-fixes/20180515-233022 >> config: i386-allmodconfig (attached as .config) >> compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 >> reproduce: >> # save the attached .config to linux build tree >> make ARCH=i386 >> >> All errors (new ones prefixed by >>): >> >> WARNING: modpost: missing MODULE_LICENSE() in sound/soc/omap/snd-soc-sdma.o >> see include/linux/module.h for more information >>>> ERROR: "mwifiex_send_cmd" [drivers/net/wireless/marvell/mwifiex/mwifiex_usb.ko] undefined! > > Yikes. Missed that. In my build this is a warning which I overlooked. > Is there some Kconfig to make it an ERROR like above? No idea. > You applied the first patch so should I resubmit all other patches? Yes, please. -- Kalle Valo