Hi Ognjen,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on platform-drivers-x86/for-next]
[also build test ERROR on v4.15-rc4 next-20171222]
[cannot apply to pm/linux-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/Ognjen-Galic/battery-Add-the-battery-hooking-API/20171223-144855
base: git://git.infradead.org/users/dvhart/linux-platform-drivers-x86.git for-next
config: x86_64-acpi-redef (attached as .config)
compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All errors (new ones prefixed by >>):
In file included from include/linux/list.h:9:0,
from include/linux/module.h:9,
from drivers/power/supply/ds2780_battery.c:16:
>> include/linux/kernel.h:928:42: error: expected identifier or '(' before '{' token
#define container_of(ptr, type, member) ({ \
^
include/linux/power_supply.h:374:33: note: in expansion of macro 'container_of'
#define to_power_supply(device) container_of(device, struct power_supply, dev)
^~~~~~~~~~~~
drivers/power/supply/ds2780_battery.c:59:36: note: in expansion of macro 'to_power_supply'
static inline struct power_supply *to_power_supply(struct device *dev)
^~~~~~~~~~~~~~~
--
In file included from include/linux/list.h:9:0,
from include/linux/module.h:9,
from drivers/power/supply/ds2781_battery.c:14:
>> include/linux/kernel.h:928:42: error: expected identifier or '(' before '{' token
#define container_of(ptr, type, member) ({ \
^
include/linux/power_supply.h:374:33: note: in expansion of macro 'container_of'
#define to_power_supply(device) container_of(device, struct power_supply, dev)
^~~~~~~~~~~~
drivers/power/supply/ds2781_battery.c:57:36: note: in expansion of macro 'to_power_supply'
static inline struct power_supply *to_power_supply(struct device *dev)
^~~~~~~~~~~~~~~
--
In file included from include/linux/list.h:9:0,
from include/linux/module.h:9,
from drivers/power//supply/ds2780_battery.c:16:
>> include/linux/kernel.h:928:42: error: expected identifier or '(' before '{' token
#define container_of(ptr, type, member) ({ \
^
include/linux/power_supply.h:374:33: note: in expansion of macro 'container_of'
#define to_power_supply(device) container_of(device, struct power_supply, dev)
^~~~~~~~~~~~
drivers/power//supply/ds2780_battery.c:59:36: note: in expansion of macro 'to_power_supply'
static inline struct power_supply *to_power_supply(struct device *dev)
^~~~~~~~~~~~~~~
--
In file included from include/linux/list.h:9:0,
from include/linux/module.h:9,
from drivers/power//supply/ds2781_battery.c:14:
>> include/linux/kernel.h:928:42: error: expected identifier or '(' before '{' token
#define container_of(ptr, type, member) ({ \
^
include/linux/power_supply.h:374:33: note: in expansion of macro 'container_of'
#define to_power_supply(device) container_of(device, struct power_supply, dev)
^~~~~~~~~~~~
drivers/power//supply/ds2781_battery.c:57:36: note: in expansion of macro 'to_power_supply'
static inline struct power_supply *to_power_supply(struct device *dev)
^~~~~~~~~~~~~~~
vim +928 include/linux/kernel.h
^1da177e4 Linus Torvalds 2005-04-16 911
91f68b735 Wu Fengguang 2009-01-07 912
e8c97af0c Randy Dunlap 2017-10-13 913 /**
e8c97af0c Randy Dunlap 2017-10-13 914 * swap - swap values of @a and @b
e8c97af0c Randy Dunlap 2017-10-13 915 * @a: first value
e8c97af0c Randy Dunlap 2017-10-13 916 * @b: second value
91f68b735 Wu Fengguang 2009-01-07 917 */
ac7b90049 Peter Zijlstra 2009-02-04 918 #define swap(a, b) \
ac7b90049 Peter Zijlstra 2009-02-04 919 do { typeof(a) __tmp = (a); (a) = (b); (b) = __tmp; } while (0)
91f68b735 Wu Fengguang 2009-01-07 920
^1da177e4 Linus Torvalds 2005-04-16 921 /**
^1da177e4 Linus Torvalds 2005-04-16 922 * container_of - cast a member of a structure out to the containing structure
^1da177e4 Linus Torvalds 2005-04-16 923 * @ptr: the pointer to the member.
^1da177e4 Linus Torvalds 2005-04-16 924 * @type: the type of the container struct this is embedded in.
^1da177e4 Linus Torvalds 2005-04-16 925 * @member: the name of the member within the struct.
^1da177e4 Linus Torvalds 2005-04-16 926 *
^1da177e4 Linus Torvalds 2005-04-16 927 */
^1da177e4 Linus Torvalds 2005-04-16 @928 #define container_of(ptr, type, member) ({ \
c7acec713 Ian Abbott 2017-07-12 929 void *__mptr = (void *)(ptr); \
c7acec713 Ian Abbott 2017-07-12 930 BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
c7acec713 Ian Abbott 2017-07-12 931 !__same_type(*(ptr), void), \
c7acec713 Ian Abbott 2017-07-12 932 "pointer type mismatch in container_of()"); \
c7acec713 Ian Abbott 2017-07-12 933 ((type *)(__mptr - offsetof(type, member))); })
^1da177e4 Linus Torvalds 2005-04-16 934
:::::: The code at line 928 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
:::::: TO: Linus Torvalds <tor...@pp...>
:::::: CC: Linus Torvalds <tor...@pp...>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
|