mirror of https://lore.kernel.org/lkml/
 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: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-29 16:55 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

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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®