tree:   https://git.kernel.org/pub/scm/linux/kernel/git/peter.chen/usb 
add_pwrseq_for_usb
head:   236dd95c9a8872af797cc504ea9d0fe74b8c55d3
commit: d178023ba59940c022fcd0be5b65a0b328fa368a [13/16] usb: core: add power 
sequence handling for USB devices
config: blackfin-BF526-EZBRD_defconfig (attached as .config)
compiler: bfin-uclinux-gcc (GCC) 4.6.3
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
        git checkout d178023ba59940c022fcd0be5b65a0b328fa368a
        # save the attached .config to linux build tree
        make.cross ARCH=blackfin 

All warnings (new ones prefixed by >>):

   In file included from drivers/usb/core/hub.c:29:0:
   include/linux/pwrseq.h: In function 'pwrseq_pre_power_on':
   include/linux/pwrseq.h:49:46: warning: no return statement in function 
returning non-void [-Wreturn-type]
   include/linux/pwrseq.h: In function 'pwrseq_alloc':
   include/linux/pwrseq.h:53:3: warning: no return statement in function 
returning non-void [-Wreturn-type]
   include/linux/pwrseq.h: At top level:
   include/linux/pwrseq.h:54:39: warning: 'struct mmc_host' declared inside 
parameter list [enabled by default]
   include/linux/pwrseq.h:54:39: warning: its scope is only this definition or 
declaration, which is probably not what you want [enabled by default]
   drivers/usb/core/hub.c: In function 'hub_of_pwrseq_off':
>> drivers/usb/core/hub.c:1698:3: warning: passing argument 1 of 'pwrseq_free' 
>> from incompatible pointer type [enabled by default]
   include/linux/pwrseq.h:54:20: note: expected 'struct mmc_host *' but 
argument is of type 'struct pwrseq *'
   drivers/usb/core/hub.c: In function 'hub_of_pwrseq_on':
   drivers/usb/core/hub.c:1742:2: warning: passing argument 1 of 'pwrseq_free' 
from incompatible pointer type [enabled by default]
   include/linux/pwrseq.h:54:20: note: expected 'struct mmc_host *' but 
argument is of type 'struct pwrseq *'

vim +/pwrseq_free +1698 drivers/usb/core/hub.c

  1682          usb_put_intf(to_usb_interface(hub->intfdev));
  1683          kfree(hub);
  1684  }
  1685  
  1686  static unsigned highspeed_hubs;
  1687  
  1688  static void hub_of_pwrseq_off(struct usb_hub *hub)
  1689  {
  1690          struct pwrseq *hdev_pwrseq;
  1691          struct pwrseq_node_powered_on *pwrseq_node, *tmp_node;
  1692  
  1693          list_for_each_entry_safe(pwrseq_node, tmp_node,
  1694                          &hub->pwrseq_list, list) {
  1695                  hdev_pwrseq = pwrseq_node->pwrseq_on;
  1696                  pwrseq_power_off(hdev_pwrseq);
  1697                  list_del(&pwrseq_node->list);
> 1698                  pwrseq_free(hdev_pwrseq);
  1699                  kfree(pwrseq_node);
  1700          }
  1701  }
  1702  
  1703  static int hub_of_pwrseq_on(struct usb_hub *hub)
  1704  {
  1705          struct device *parent;
  1706          struct device_node *node;

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

Attachment: .config.gz
Description: Binary data

Reply via email to