All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Mark Brown <broonie@kernel.org>
Cc: kbuild-all@01.org, Chen-Yu Tsai <wens@csie.org>,
	Mark Brown <broonie@kernel.org>,
	linux-kernel@vger.kernel.org
Subject: Re: Applied "regulator: axp20x: support AXP809 variant" to the regulator tree
Date: Wed, 30 Mar 2016 03:09:10 +0800	[thread overview]
Message-ID: <201603300314.34PbnWqZ%fengguang.wu@intel.com> (raw)
In-Reply-To: <E1akwvL-0000qf-Ve@finisterre>

[-- Attachment #1: Type: text/plain, Size: 13383 bytes --]

Hi Mark,

[auto build test WARNING on regulator/for-next]
[also build test WARNING on v4.6-rc1 next-20160329]
[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/Mark-Brown/Applied-regulator-axp20x-support-AXP809-variant-to-the-regulator-tree/20160330-005751
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
config: x86_64-randconfig-n0-03300206 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

      ^
   drivers/regulator/axp20x-regulator.c:272:14: error: 'AXP809_LDO_IO0' undeclared here (not in a function)
     AXP_DESC_IO(AXP809, LDO_IO0, "ldo_io0", "ips", 700, 3300, 100,
                 ^
   drivers/regulator/axp20x-regulator.c:41:3: note: in definition of macro 'AXP_DESC_IO'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:272:14: error: array index in initializer not of integer type
     AXP_DESC_IO(AXP809, LDO_IO0, "ldo_io0", "ips", 700, 3300, 100,
                 ^
   drivers/regulator/axp20x-regulator.c:41:3: note: in definition of macro 'AXP_DESC_IO'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:272:14: note: (near initialization for 'axp809_regulators')
     AXP_DESC_IO(AXP809, LDO_IO0, "ldo_io0", "ips", 700, 3300, 100,
                 ^
   drivers/regulator/axp20x-regulator.c:41:3: note: in definition of macro 'AXP_DESC_IO'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:275:14: error: 'AXP809_LDO_IO1' undeclared here (not in a function)
     AXP_DESC_IO(AXP809, LDO_IO1, "ldo_io1", "ips", 700, 3300, 100,
                 ^
   drivers/regulator/axp20x-regulator.c:41:3: note: in definition of macro 'AXP_DESC_IO'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:275:14: error: array index in initializer not of integer type
     AXP_DESC_IO(AXP809, LDO_IO1, "ldo_io1", "ips", 700, 3300, 100,
                 ^
   drivers/regulator/axp20x-regulator.c:41:3: note: in definition of macro 'AXP_DESC_IO'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:275:14: note: (near initialization for 'axp809_regulators')
     AXP_DESC_IO(AXP809, LDO_IO1, "ldo_io1", "ips", 700, 3300, 100,
                 ^
   drivers/regulator/axp20x-regulator.c:41:3: note: in definition of macro 'AXP_DESC_IO'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:278:17: error: 'AXP809_RTC_LDO' undeclared here (not in a function)
     AXP_DESC_FIXED(AXP809, RTC_LDO, "rtc_ldo", "ips", 1800),
                    ^
   drivers/regulator/axp20x-regulator.c:96:3: note: in definition of macro 'AXP_DESC_FIXED'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:278:17: error: array index in initializer not of integer type
     AXP_DESC_FIXED(AXP809, RTC_LDO, "rtc_ldo", "ips", 1800),
                    ^
   drivers/regulator/axp20x-regulator.c:96:3: note: in definition of macro 'AXP_DESC_FIXED'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:278:17: note: (near initialization for 'axp809_regulators')
     AXP_DESC_FIXED(AXP809, RTC_LDO, "rtc_ldo", "ips", 1800),
                    ^
   drivers/regulator/axp20x-regulator.c:96:3: note: in definition of macro 'AXP_DESC_FIXED'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:279:14: error: 'AXP809_SW' undeclared here (not in a function)
     AXP_DESC_SW(AXP809, SW, "sw", "swin", AXP22X_PWR_OUT_CTRL2, BIT(6)),
                 ^
   drivers/regulator/axp20x-regulator.c:82:3: note: in definition of macro 'AXP_DESC_SW'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:279:14: error: array index in initializer not of integer type
     AXP_DESC_SW(AXP809, SW, "sw", "swin", AXP22X_PWR_OUT_CTRL2, BIT(6)),
                 ^
   drivers/regulator/axp20x-regulator.c:82:3: note: in definition of macro 'AXP_DESC_SW'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c:279:14: note: (near initialization for 'axp809_regulators')
     AXP_DESC_SW(AXP809, SW, "sw", "swin", AXP22X_PWR_OUT_CTRL2, BIT(6)),
                 ^
   drivers/regulator/axp20x-regulator.c:82:3: note: in definition of macro 'AXP_DESC_SW'
     [_family##_##_id] = {       \
      ^
   drivers/regulator/axp20x-regulator.c: In function 'axp20x_set_dcdc_freq':
   drivers/regulator/axp20x-regulator.c:297:7: error: 'AXP809_ID' undeclared (first use in this function)
     case AXP809_ID:
          ^
   drivers/regulator/axp20x-regulator.c:297:7: note: each undeclared identifier is reported only once for each function it appears in
   drivers/regulator/axp20x-regulator.c: In function 'axp20x_set_dcdc_workmode':
   drivers/regulator/axp20x-regulator.c:377:7: error: 'AXP809_ID' undeclared (first use in this function)
     case AXP809_ID:
          ^
   drivers/regulator/axp20x-regulator.c: In function 'axp20x_regulator_probe':
   drivers/regulator/axp20x-regulator.c:420:7: error: 'AXP809_ID' undeclared (first use in this function)
     case AXP809_ID:
          ^
   drivers/regulator/axp20x-regulator.c:422:17: error: 'AXP809_REG_ID_MAX' undeclared (first use in this function)
      nregulators = AXP809_REG_ID_MAX;
                    ^
   drivers/regulator/axp20x-regulator.c:422:15: warning: assignment makes integer from pointer without a cast [-Wint-conversion]
      nregulators = AXP809_REG_ID_MAX;
                  ^
   In file included from include/linux/err.h:4:0,
                    from drivers/regulator/axp20x-regulator.c:16:
   drivers/regulator/axp20x-regulator.c:447:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DC1SW)) {
                                                ^
   include/linux/compiler.h:151:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^
>> drivers/regulator/axp20x-regulator.c:446:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DC1SW) ||
      ^
   drivers/regulator/axp20x-regulator.c:447:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DC1SW)) {
                                                ^
   include/linux/compiler.h:151:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^
>> drivers/regulator/axp20x-regulator.c:446:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DC1SW) ||
      ^
   drivers/regulator/axp20x-regulator.c:447:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DC1SW)) {
                                                ^
   include/linux/compiler.h:162:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^
>> drivers/regulator/axp20x-regulator.c:446:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DC1SW) ||
      ^
   drivers/regulator/axp20x-regulator.c:456:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DC5LDO)) {
                                                ^
   include/linux/compiler.h:151:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^
   drivers/regulator/axp20x-regulator.c:455:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DC5LDO) ||
      ^
   drivers/regulator/axp20x-regulator.c:456:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DC5LDO)) {
                                                ^
   include/linux/compiler.h:151:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^
   drivers/regulator/axp20x-regulator.c:455:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DC5LDO) ||
      ^
   drivers/regulator/axp20x-regulator.c:456:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DC5LDO)) {
                                                ^
   include/linux/compiler.h:162:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^
   drivers/regulator/axp20x-regulator.c:455:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DC5LDO) ||
      ^
   drivers/regulator/axp20x-regulator.c:485:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DCDC1))
                                                ^
   include/linux/compiler.h:151:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^
   drivers/regulator/axp20x-regulator.c:484:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DCDC1) ||
      ^
   drivers/regulator/axp20x-regulator.c:485:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DCDC1))
                                                ^
   include/linux/compiler.h:151:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^
   drivers/regulator/axp20x-regulator.c:484:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DCDC1) ||
      ^
   drivers/regulator/axp20x-regulator.c:485:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DCDC1))
                                                ^
   include/linux/compiler.h:162:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^
   drivers/regulator/axp20x-regulator.c:484:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DCDC1) ||
      ^
   drivers/regulator/axp20x-regulator.c:491:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DCDC5))
                                                ^
   include/linux/compiler.h:151:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^
   drivers/regulator/axp20x-regulator.c:490:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DCDC5) ||
      ^
   drivers/regulator/axp20x-regulator.c:491:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DCDC5))
                                                ^
   include/linux/compiler.h:151:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^
   drivers/regulator/axp20x-regulator.c:490:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DCDC5) ||
      ^
   drivers/regulator/axp20x-regulator.c:491:45: warning: comparison between pointer and integer
          (regulators == axp809_regulators && i == AXP809_DCDC5))
                                                ^
   include/linux/compiler.h:162:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^
   drivers/regulator/axp20x-regulator.c:490:3: note: in expansion of macro 'if'
      if ((regulators == axp22x_regulators && i == AXP22X_DCDC5) ||
      ^

vim +/if +446 drivers/regulator/axp20x-regulator.c

   430		/* This only sets the dcdc freq. Ignore any errors */
   431		axp20x_regulator_parse_dt(pdev);
   432	
   433		for (i = 0; i < nregulators; i++) {
   434			const struct regulator_desc *desc = &regulators[i];
   435			struct regulator_desc *new_desc;
   436	
   437			/*
   438			 * Regulators DC1SW and DC5LDO are connected internally,
   439			 * so we have to handle their supply names separately.
   440			 *
   441			 * We always register the regulators in proper sequence,
   442			 * so the supply names are correctly read. See the last
   443			 * part of this loop to see where we save the DT defined
   444			 * name.
   445			 */
 > 446			if ((regulators == axp22x_regulators && i == AXP22X_DC1SW) ||
   447			    (regulators == axp809_regulators && i == AXP809_DC1SW)) {
   448				new_desc = devm_kzalloc(&pdev->dev, sizeof(*desc),
   449							GFP_KERNEL);
   450				*new_desc = regulators[i];
   451				new_desc->supply_name = dcdc1_name;
   452				desc = new_desc;
   453			}
   454	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 21692 bytes --]

  parent reply	other threads:[~2016-03-29 19:10 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-29 16:55 Applied "regulator: axp20x: support AXP809 variant" to the regulator tree Mark Brown
2016-03-29 17:19 ` kbuild test robot
2016-03-29 17:24   ` Mark Brown
2016-03-29 19:09 ` kbuild test robot [this message]
  -- strict thread matches above, loose matches on Subject: below --
2016-02-02 10:27 [PATCH 06/11] regulator: axp20x: support AXP809 variant Chen-Yu Tsai
2016-05-31 17:46 ` Applied "regulator: axp20x: support AXP809 variant" to the regulator tree Mark Brown
2016-05-31 17:46   ` Mark Brown
2016-05-31 17:46   ` Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201603300314.34PbnWqZ%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=broonie@kernel.org \
    --cc=kbuild-all@01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=wens@csie.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.