Hi Jerry,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on balbi-usb/next]
[also build test ERROR on v4.16 next-20180409]
[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/Jerry-Zhang/usb-gadget-f_midi-Add-configfs-attrs-for-card-and-device/20180410-061518
base:   https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git next
config: x86_64-randconfig-x011-201814 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   drivers/usb/gadget/function/f_midi.c: In function 'f_midi_opts_card_show':
>> drivers/usb/gadget/function/f_midi.c:1188:22: error: 'struct 
>> usb_function_instance' has no member named 'f'; did you mean 'fd'?
     if (opts->func_inst.f) {
                         ^
                         fd
   drivers/usb/gadget/function/f_midi.c:1189:39: error: 'struct 
usb_function_instance' has no member named 'f'; did you mean 'fd'?
      midi = func_to_midi(opts->func_inst.f);
                                          ^
                                          fd
   drivers/usb/gadget/function/f_midi.c: In function 'f_midi_opts_device_show':
   drivers/usb/gadget/function/f_midi.c:1206:22: error: 'struct 
usb_function_instance' has no member named 'f'; did you mean 'fd'?
     if (opts->func_inst.f) {
                         ^
                         fd
   drivers/usb/gadget/function/f_midi.c:1207:39: error: 'struct 
usb_function_instance' has no member named 'f'; did you mean 'fd'?
      midi = func_to_midi(opts->func_inst.f);
                                          ^
                                          fd

vim +1188 drivers/usb/gadget/function/f_midi.c

  1181  
  1182  static ssize_t f_midi_opts_card_show(struct config_item *item, char 
*page)
  1183  {
  1184          struct f_midi_opts *opts = to_f_midi_opts(item);
  1185          int card = -1;
  1186          struct f_midi *midi;
  1187  
> 1188          if (opts->func_inst.f) {
  1189                  midi = func_to_midi(opts->func_inst.f);
  1190                  if (midi->rmidi && midi->rmidi->card) {
  1191                          card = midi->rmidi->card->number;
  1192                  }
  1193          }
  1194  
  1195          return sprintf(page, "%d\n", card);
  1196  }
  1197  

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

Attachment: .config.gz
Description: application/gzip

Reply via email to