[peter.chen-usb:add_pwrseq_for_usb 13/16] include/linux/pwrseq.h:54:39: warning: 'struct mmc_host' declared inside parameter list

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

 



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: sh-titan_defconfig (attached as .config)
compiler: sh4-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
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=sh 

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]
    static inline int pwrseq_pre_power_on(struct pwrseq *pwrseq) {}
                                                 ^
   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]
      const char *dev_name) {}
      ^
   include/linux/pwrseq.h: At top level:
>> include/linux/pwrseq.h:54:39: warning: 'struct mmc_host' declared inside parameter list
    static inline void pwrseq_free(struct mmc_host *host) {}
                                          ^
>> include/linux/pwrseq.h:54:39: warning: its scope is only this definition or declaration, which is probably not what you want
   drivers/usb/core/hub.c: In function 'hub_of_pwrseq_off':
   drivers/usb/core/hub.c:1698:15: error: passing argument 1 of 'pwrseq_free' from incompatible pointer type [-Werror=incompatible-pointer-types]
      pwrseq_free(hdev_pwrseq);
                  ^
   In file included from drivers/usb/core/hub.c:29:0:
   include/linux/pwrseq.h:54:20: note: expected 'struct mmc_host *' but argument is of type 'struct pwrseq *'
    static inline void pwrseq_free(struct mmc_host *host) {}
                       ^
   drivers/usb/core/hub.c: In function 'hub_of_pwrseq_on':
   drivers/usb/core/hub.c:1742:14: error: passing argument 1 of 'pwrseq_free' from incompatible pointer type [-Werror=incompatible-pointer-types]
     pwrseq_free(hdev_pwrseq);
                 ^
   In file included from drivers/usb/core/hub.c:29:0:
   include/linux/pwrseq.h:54:20: note: expected 'struct mmc_host *' but argument is of type 'struct pwrseq *'
    static inline void pwrseq_free(struct mmc_host *host) {}
                       ^
   cc1: some warnings being treated as errors

vim +54 include/linux/pwrseq.h

3aa8793f drivers/mmc/core/pwrseq.h Ulf Hansson         2014-11-28  43  
04001411 include/linux/pwrseq.h    Krzysztof Kozlowski 2016-06-09  44  static inline int pwrseq_register(struct pwrseq *pwrseq)
d97a1e5d drivers/mmc/core/pwrseq.h Srinivas Kandagatla 2016-04-14  45  {
d97a1e5d drivers/mmc/core/pwrseq.h Srinivas Kandagatla 2016-04-14  46  	return -ENOSYS;
d97a1e5d drivers/mmc/core/pwrseq.h Srinivas Kandagatla 2016-04-14  47  }
04001411 include/linux/pwrseq.h    Krzysztof Kozlowski 2016-06-09  48  static inline void pwrseq_unregister(struct pwrseq *pwrseq) {}
210f01d9 include/linux/pwrseq.h    Peter Chen          2016-06-15 @49  static inline int pwrseq_pre_power_on(struct pwrseq *pwrseq) {}
c2f5b6a4 include/linux/pwrseq.h    Krzysztof Kozlowski 2016-06-09  50  static inline void pwrseq_post_power_on(struct pwrseq *pwrseq) {}
c2f5b6a4 include/linux/pwrseq.h    Krzysztof Kozlowski 2016-06-09  51  static inline void pwrseq_power_off(struct pwrseq *pwrseq) {}
210f01d9 include/linux/pwrseq.h    Peter Chen          2016-06-15  52  static inline struct pwrseq *pwrseq_alloc(struct device_node *np,
210f01d9 include/linux/pwrseq.h    Peter Chen          2016-06-15 @53  		const char *dev_name) {}
210f01d9 include/linux/pwrseq.h    Peter Chen          2016-06-15 @54  static inline void pwrseq_free(struct mmc_host *host) {}
3aa8793f drivers/mmc/core/pwrseq.h Ulf Hansson         2014-11-28  55  
57402ee8 include/linux/pwrseq.h    Krzysztof Kozlowski 2016-06-09  56  #endif /* CONFIG_POWER_SEQ */
3aa8793f drivers/mmc/core/pwrseq.h Ulf Hansson         2014-11-28  57  

:::::: The code at line 54 was first introduced by commit
:::::: 210f01d924a442bc83b11f8bb2c0ee6845fa2e20 power: pwrseq: change common helpers as generic

:::::: TO: Peter Chen <peter.chen@xxxxxxx>
:::::: CC: Peter Chen <peter.chen@xxxxxxx>

---
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