linux-kernel - Re: [PATCH v2 3/4] usb: dwc3: of-simple: Add support to get resets for the device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 4 Apr 2017 13:34:21 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Vivek Gautam <vivek.gautam@...eaurora.org>
Cc:     kbuild-all@...org, linux-kernel@...r.kernel.org,
        linux-tegra@...r.kernel.org, linux-usb@...r.kernel.org,
        p.zabel@...gutronix.de, swarren@...dotorg.org,
        thierry.reding@...il.com, balbi@...nel.org,
        Vivek Gautam <vivek.gautam@...eaurora.org>
Subject: Re: [PATCH v2 3/4] usb: dwc3: of-simple: Add support to get resets
 for the device

Hi Vivek,

[auto build test WARNING on balbi-usb/next]
[also build test WARNING on v4.11-rc5 next-20170403]
[cannot apply to pza/reset/next]
[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/Vivek-Gautam/reset-APIs-to-manage-a-list-of-resets/20170404-111639
base:   https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git next
config: i386-randconfig-r0-201714 (attached as .config)
compiler: gcc-5 (Debian 5.4.1-2) 5.4.1 20160904
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   In file included from drivers/usb/dwc3/dwc3-of-simple.c:32:0:
   include/linux/reset.h:106:1: error: expected identifier or '(' before '{' token
    {
    ^
   include/linux/reset.h: In function 'devm_reset_control_array_get':
   include/linux/reset.h:127:9: warning: return makes integer from pointer without a cast [-Wint-conversion]
     return optional ? NULL : ERR_PTR(-ENOTSUPP);
            ^
   include/linux/reset.h: In function 'of_reset_control_array_get':
   include/linux/reset.h:135:9: warning: return makes integer from pointer without a cast [-Wint-conversion]
     return optional ? NULL : ERR_PTR(-ENOTSUPP);
            ^
   drivers/usb/dwc3/dwc3-of-simple.c: At top level:
>> include/linux/reset.h:105:19: warning: 'of_reset_control_get_count' used but never defined
    static inline int of_reset_control_get_count(struct device_node *node);
                      ^

vim +/of_reset_control_get_count +105 include/linux/reset.h

bb475230 Ramiro Oliveira 2017-01-13   99  					struct device *dev, const char *id,
bb475230 Ramiro Oliveira 2017-01-13  100  					int index, bool shared, bool optional)
5bcd0b7f Axel Lin        2015-09-01  101  {
0ca10b60 Philipp Zabel   2017-03-20  102  	return optional ? NULL : ERR_PTR(-ENOTSUPP);
5bcd0b7f Axel Lin        2015-09-01  103  }
5bcd0b7f Axel Lin        2015-09-01  104  
045cc544 Vivek Gautam    2017-04-03 @105  static inline int of_reset_control_get_count(struct device_node *node);
045cc544 Vivek Gautam    2017-04-03  106  {
045cc544 Vivek Gautam    2017-04-03  107  	return -ENOTSUPP;
045cc544 Vivek Gautam    2017-04-03  108  }
045cc544 Vivek Gautam    2017-04-03  109  
d5652c65 Vivek Gautam    2017-04-03  110  static inline int reset_control_array_assert(int num_rsts,
d5652c65 Vivek Gautam    2017-04-03  111  				struct reset_control_array *resets)
d5652c65 Vivek Gautam    2017-04-03  112  {
d5652c65 Vivek Gautam    2017-04-03  113  	return 0;
d5652c65 Vivek Gautam    2017-04-03  114  }
d5652c65 Vivek Gautam    2017-04-03  115  
d5652c65 Vivek Gautam    2017-04-03  116  static inline int reset_control_array_deassert(int num_rsts,
d5652c65 Vivek Gautam    2017-04-03  117  				struct reset_control_array *resets)
d5652c65 Vivek Gautam    2017-04-03  118  {
d5652c65 Vivek Gautam    2017-04-03  119  	return 0;
d5652c65 Vivek Gautam    2017-04-03  120  }
d5652c65 Vivek Gautam    2017-04-03  121  
d5652c65 Vivek Gautam    2017-04-03  122  static inline
d5652c65 Vivek Gautam    2017-04-03  123  int devm_reset_control_array_get(struct device *dev, int num_rsts,
d5652c65 Vivek Gautam    2017-04-03  124  				struct reset_control_array *resets,
d5652c65 Vivek Gautam    2017-04-03  125  				bool shared, bool optional)
d5652c65 Vivek Gautam    2017-04-03  126  {
d5652c65 Vivek Gautam    2017-04-03 @127  	return optional ? NULL : ERR_PTR(-ENOTSUPP);
d5652c65 Vivek Gautam    2017-04-03  128  }
d5652c65 Vivek Gautam    2017-04-03  129  
d5652c65 Vivek Gautam    2017-04-03  130  static inline

:::::: The code at line 105 was first introduced by commit
:::::: 045cc544119a744cda92da0644066f18cb3760b4 reset: Add API to count number of reset available with device

:::::: TO: Vivek Gautam <vivek.gautam@...eaurora.org>
:::::: CC: 0day robot <fengguang.wu@...el.com>

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

Download attachment ".config.gz" of type "application/gzip" (29765 bytes)