Hi Claudiu, I love your patch! Yet something to improve: [auto build test ERROR on regulator/for-next] [also build test ERROR on linus/master linux/master v5.10-rc3 next-20201112] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Claudiu-Beznea/regulator-mcp1650... base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next config: nios2-randconfig-r034-20201111 (attached as .config) compiler: nios2-linux-gcc (GCC) 9.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/d97d6b70286fa3a28e744a55da1b7eb6a... git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Claudiu-Beznea/regulator-mcp16502-add-support-for-ramp-delay/20201111-170928 git checkout d97d6b70286fa3a28e744a55da1b7eb6a026cee3 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=nios2 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All errors (new ones prefixed by >>): In file included from include/asm-generic/bug.h:20, from ./arch/nios2/include/generated/asm/bug.h:1, from include/linux/bug.h:5, from include/linux/thread_info.h:12, from include/asm-generic/current.h:5, from ./arch/nios2/include/generated/asm/current.h:1, from include/linux/sched.h:12, from include/linux/ratelimit.h:6, from include/linux/dev_printk.h:16, from include/linux/device.h:15, from include/linux/gpio/driver.h:5, from include/asm-generic/gpio.h:11, from include/linux/gpio.h:62, from drivers/regulator/mcp16502.c:11: drivers/regulator/mcp16502.c: In function 'mcp16502_set_voltage_time_sel': 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:287:38: note: in definition of macro '__abs_choose_expr' 287 | __builtin_types_compatible_p(typeof(x), signed type) || \ | ^ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:287:38: note: in definition of macro '__abs_choose_expr' 287 | __builtin_types_compatible_p(typeof(x), signed type) || \ | ^ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:288:38: note: in definition of macro '__abs_choose_expr' 288 | __builtin_types_compatible_p(typeof(x), unsigned type), \ | ^ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:288:38: note: in definition of macro '__abs_choose_expr' 288 | __builtin_types_compatible_p(typeof(x), unsigned type), \ | ^ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:24: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:24: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ drivers/regulator/mcp16502.c:296:47: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'? 296 | old_sel * rdev->desc->linear_ranges->uV_step); | ^~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ 281 | __builtin_choose_expr( \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ include/linux/kernel.h:286:43: error: first argument to '__builtin_choose_expr' not a constant 286 | #define __abs_choose_expr(x, type, other) __builtin_choose_expr( \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:280:3: note: in expansion of macro '__abs_choose_expr' 280 | __abs_choose_expr(x, char, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ include/linux/kernel.h:286:43: error: first argument to '__builtin_choose_expr' not a constant 286 | #define __abs_choose_expr(x, type, other) __builtin_choose_expr( \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:279:3: note: in expansion of macro '__abs_choose_expr' 279 | __abs_choose_expr(x, short, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ include/linux/kernel.h:286:43: error: first argument to '__builtin_choose_expr' not a constant 286 | #define __abs_choose_expr(x, type, other) __builtin_choose_expr( \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ include/linux/kernel.h:278:3: note: in expansion of macro '__abs_choose_expr' 278 | __abs_choose_expr(x, int, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ include/linux/kernel.h:286:43: error: first argument to '__builtin_choose_expr' not a constant 286 | #define __abs_choose_expr(x, type, other) __builtin_choose_expr( \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/kernel.h:289:54: note: in definition of macro '__abs_choose_expr' 289 | ({ signed type __x = (x); __x < 0 ? -__x : __x; }), other) | ^~~~~ include/linux/kernel.h:277:3: note: in expansion of macro '__abs_choose_expr' 277 | __abs_choose_expr(x, long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ include/linux/kernel.h:286:43: error: first argument to '__builtin_choose_expr' not a constant 286 | #define __abs_choose_expr(x, type, other) __builtin_choose_expr( \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/kernel.h:276:16: note: in expansion of macro '__abs_choose_expr' 276 | #define abs(x) __abs_choose_expr(x, long long, \ | ^~~~~~~~~~~~~~~~~ drivers/regulator/mcp16502.c:295:13: note: in expansion of macro 'abs' 295 | uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step - | ^~~ vim +295 drivers/regulator/mcp16502.c 280 281 static int mcp16502_set_voltage_time_sel(struct regulator_dev *rdev, 282 unsigned int old_sel, 283 unsigned int new_sel) 284 { 285 static const u8 us_ramp[] = { 8, 16, 24, 32 }; 286 int id = rdev_get_id(rdev); 287 unsigned int uV_delta, val; 288 int ret; 289 290 ret = regmap_read(rdev->regmap, MCP16502_REG_BASE(id, CFG), &val); 291 if (ret) 292 return ret; 293 294 val = (val & MCP16502_DVSR) >> 2; 296 old_sel * rdev->desc->linear_ranges->uV_step); 297 switch (id) { 298 case BUCK1: 299 case LDO1: 300 case LDO2: 301 ret = DIV_ROUND_CLOSEST(uV_delta * us_ramp[val], 302 mcp16502_ramp_b1l12[val]); 303 break; 304 305 case BUCK2: 306 case BUCK3: 307 case BUCK4: 308 ret = DIV_ROUND_CLOSEST(uV_delta * us_ramp[val], 309 mcp16502_ramp_b234[val]); 310 break; 311 312 default: 313 return -EINVAL; 314 } 315 316 return ret; 317 } 318 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org