All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Stefan Popa <stefan.popa@analog.com>
Cc: kbuild-all@01.org, sre@kernel.org, davem@davemloft.net,
	mchehab@kernel.org, gregkh@linuxfoundation.org,
	linus.walleij@linaro.org, akpm@linux-foundation.org,
	rdunlap@infradead.org, linux-pm@vger.kernel.org,
	linux-kernel@vger.kernel.org, stefan.popa@analog.com
Subject: Re: [PATCH 2/3] adp5061: Add support for battery charging enable
Date: Thu, 5 Apr 2018 01:01:46 +0800	[thread overview]
Message-ID: <201804050003.KmxmZBka%fengguang.wu@intel.com> (raw)
In-Reply-To: <1522829877-7977-1-git-send-email-stefan.popa@analog.com>

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

Hi Stefan,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.16 next-20180404]
[cannot apply to battery/master]
[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/Stefan-Popa/adp5061-New-driver-for-ADP5061-I2C-battery-charger/20180405-000342
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/kobject.h:20:0,
                    from include/linux/module.h:17,
                    from drivers/power/supply/adp5061.c:10:
>> drivers/power/supply/adp5061.c:739:23: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:104:10: note: in definition of macro '__ATTR'
     .show = _show,      \
             ^~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
   drivers/power/supply/adp5061.c:739:23: note: (near initialization for 'dev_attr_charging_enabled.show')
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:104:10: note: in definition of macro '__ATTR'
     .show = _show,      \
             ^~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
>> drivers/power/supply/adp5061.c:739:23: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:105:11: note: in definition of macro '__ATTR'
     .store = _store,      \
              ^~~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
   drivers/power/supply/adp5061.c:739:23: note: (near initialization for 'dev_attr_charging_enabled.store')
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:105:11: note: in definition of macro '__ATTR'
     .store = _store,      \
              ^~~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +739 drivers/power/supply/adp5061.c

   738	
 > 739	static DEVICE_ATTR_RW(charging_enabled);
   740	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 49774 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, sre@kernel.org, davem@davemloft.net,
	mchehab@kernel.org, gregkh@linuxfoundation.org,
	linus.walleij@linaro.org, akpm@linux-foundation.org,
	rdunlap@infradead.org, linux-pm@vger.kernel.org,
	linux-kernel@vger.kernel.org, stefan.popa@analog.com
Subject: Re: [PATCH 2/3] adp5061: Add support for battery charging enable
Date: Thu, 5 Apr 2018 01:01:46 +0800	[thread overview]
Message-ID: <201804050003.KmxmZBka%fengguang.wu@intel.com> (raw)
In-Reply-To: <1522829877-7977-1-git-send-email-stefan.popa@analog.com>

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

Hi Stefan,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.16 next-20180404]
[cannot apply to battery/master]
[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/Stefan-Popa/adp5061-New-driver-for-ADP5061-I2C-battery-charger/20180405-000342
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/kobject.h:20:0,
                    from include/linux/module.h:17,
                    from drivers/power/supply/adp5061.c:10:
>> drivers/power/supply/adp5061.c:739:23: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:104:10: note: in definition of macro '__ATTR'
     .show = _show,      \
             ^~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
   drivers/power/supply/adp5061.c:739:23: note: (near initialization for 'dev_attr_charging_enabled.show')
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:104:10: note: in definition of macro '__ATTR'
     .show = _show,      \
             ^~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
>> drivers/power/supply/adp5061.c:739:23: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:105:11: note: in definition of macro '__ATTR'
     .store = _store,      \
              ^~~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
   drivers/power/supply/adp5061.c:739:23: note: (near initialization for 'dev_attr_charging_enabled.store')
    static DEVICE_ATTR_RW(charging_enabled);
                          ^
   include/linux/sysfs.h:105:11: note: in definition of macro '__ATTR'
     .store = _store,      \
              ^~~~~~
   include/linux/device.h:581:45: note: in expansion of macro '__ATTR_RW'
     struct device_attribute dev_attr_##_name = __ATTR_RW(_name)
                                                ^~~~~~~~~
>> drivers/power/supply/adp5061.c:739:8: note: in expansion of macro 'DEVICE_ATTR_RW'
    static DEVICE_ATTR_RW(charging_enabled);
           ^~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +739 drivers/power/supply/adp5061.c

   738	
 > 739	static DEVICE_ATTR_RW(charging_enabled);
   740	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 49774 bytes --]

  reply	other threads:[~2018-04-04 17:01 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-04  8:17 [PATCH 2/3] adp5061: Add support for battery charging enable Stefan Popa
2018-04-04  8:17 ` Stefan Popa
2018-04-04 17:01 ` kbuild test robot [this message]
2018-04-04 17:01   ` kbuild test robot
2018-04-10  9:51 ` [PATCH v2 " Stefan Popa
2018-04-10  9:51   ` Stefan Popa
2018-04-11 15:10   ` [PATCH v3 3/4] " Stefan Popa
2018-04-11 15:10     ` Stefan Popa
2019-10-11 10:56     ` [PATCH v3][RESEND] " Alexandru Ardelean
2019-10-13 23:10       ` kbuild test robot
2019-10-13 23:10         ` kbuild test robot
2019-11-01 13:52         ` Ardelean, Alexandru
2019-11-01 13:52           ` Ardelean, Alexandru

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=201804050003.KmxmZBka%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=davem@davemloft.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=kbuild-all@01.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=sre@kernel.org \
    --cc=stefan.popa@analog.com \
    /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.