Re: [PATCH 1/2] usb: gadget: f_midi: use flexible array member for

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michal,

[auto build test WARNING on v4.4-rc8]
[cannot apply to balbi-usb/next next-20160105]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Michal-Nazarewicz/usb-gadget-f_midi-use-flexible-array-member-for/20160105-215816
config: x86_64-randconfig-x016-01041832 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/usb/gadget/function/f_midi.c: In function 'f_midi_free':
>> drivers/usb/gadget/function/f_midi.c:1070:6: warning: unused variable 'i' [-Wunused-variable]
     int i;
         ^

vim +/i +1070 drivers/usb/gadget/function/f_midi.c

6f1de344 Andrzej Pietrasiewicz 2014-10-16  1054  	opts->id = SNDRV_DEFAULT_STR1;
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1055  	opts->buflen = 256;
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1056  	opts->qlen = 32;
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1057  	opts->in_ports = 1;
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1058  	opts->out_ports = 1;
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1059  
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1060  	config_group_init_type_name(&opts->func_inst.group, "",
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1061  				    &midi_func_type);
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1062  
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1063  	return &opts->func_inst;
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1064  }
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1065  
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1066  static void f_midi_free(struct usb_function *f)
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1067  {
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1068  	struct f_midi *midi;
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1069  	struct f_midi_opts *opts;
b85e9de9 Andrzej Pietrasiewicz 2014-10-16 @1070  	int i;
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1071  
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1072  	midi = func_to_midi(f);
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1073  	opts = container_of(f->fi, struct f_midi_opts, func_inst);
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1074  	kfree(midi->id);
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1075  	mutex_lock(&opts->lock);
b85e9de9 Andrzej Pietrasiewicz 2014-10-16  1076  	kfree(midi);
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1077  	--opts->refcnt;
6f1de344 Andrzej Pietrasiewicz 2014-10-16  1078  	mutex_unlock(&opts->lock);

:::::: The code at line 1070 was first introduced by commit
:::::: b85e9de9e818de0dcbc50b7b4242192eb6194855 usb: gadget: f_midi: convert to new function interface with backward compatibility

:::::: TO: Andrzej Pietrasiewicz <andrzej.p@xxxxxxxxxxx>
:::::: CC: Felipe Balbi <balbi@xxxxxx>

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

Attachment: .config.gz
Description: Binary data


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux