Re: [PATCH 14/27] HID: wacom: EKR: add a worker to add/remove resources on addition/removal

From: kbuild test robot <lkp@intel.com>
To: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Cc: kbuild-all@01.org, Jiri Kosina <jikos@kernel.org>,
	Ping Cheng <pinglinux@gmail.com>,
	Jason Gerecke <killertofu@gmail.com>,
	Aaron Skomra <skomra@gmail.com>,
	Peter Hutterer <peter.hutterer@who-t.net>,
	linux-kernel@vger.kernel.org, linux-input@vger.kernel.org
Subject: Re: [PATCH 14/27] HID: wacom: EKR: add a worker to add/remove resources on addition/removal
Date: Tue, 5 Jul 2016 23:21:21 +0800	[thread overview]
Message-ID: <201607052339.G1cyGzKQ%fengguang.wu@intel.com> (raw)
In-Reply-To: <1467729563-23318-15-git-send-email-benjamin.tissoires@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 2135 bytes --]

Hi,

[auto build test WARNING on hid/for-next]
[also build test WARNING on v4.7-rc6 next-20160705]
[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/Benjamin-Tissoires/HID-wacom-cleanup-EKR-LED/20160705-225431
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jikos/hid.git for-next
config: m68k-allyesconfig (attached as .config)
compiler: m68k-linux-gcc (GCC) 4.9.0
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=m68k 

All warnings (new ones prefixed by >>):

   drivers/hid/wacom_wac.c: In function 'wacom_remote_status_irq':
>> drivers/hid/wacom_wac.c:830:9: warning: missing braces around initializer [-Wmissing-braces]
     struct wacom_remote_data remote_data = {0};
            ^
   drivers/hid/wacom_wac.c:830:9: warning: (near initialization for 'remote_data.remote') [-Wmissing-braces]

vim +830 drivers/hid/wacom_wac.c

   814		if (!wacom->battery &&
   815		    !(features->quirks & WACOM_QUIRK_BATTERY)) {
   816			features->quirks |= WACOM_QUIRK_BATTERY;
   817			wacom_schedule_work(wacom_wac, WACOM_WORKER_BATTERY);
   818		}
   819	
   820		wacom_notify_battery(wacom_wac, bat_percent, bat_charging, 1,
   821				     bat_charging);
   822	
   823		return 1;
   824	}
   825	
   826	static int wacom_remote_status_irq(struct wacom_wac *wacom_wac, size_t len)
   827	{
   828		struct wacom *wacom = container_of(wacom_wac, struct wacom, wacom_wac);
   829		unsigned char *data = wacom_wac->data;
 > 830		struct wacom_remote_data remote_data = {0};
   831		unsigned long flags;
   832		int i, ret;
   833	
   834		if (data[0] != WACOM_REPORT_DEVICE_LIST)
   835			return 0;
   836	
   837		for (i = 0; i < WACOM_MAX_REMOTES; i++) {
   838			int j = i * 6;

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 37195 bytes --]