Re: [PATCH v2 5/6] regulator: mcp16502: add support for ramp delay

From: kernel test robot
Date: Fri Nov 13 2020 - 02:42:10 EST

Next message: Benjamin Tissoires: "Re: [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'"
Previous message: Dmitry Torokhov: "Re: [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables"
In reply to: Claudiu Beznea: "[PATCH v2 5/6] regulator: mcp16502: add support for ramp delay"
Next in thread: Mark Brown: "Re: [PATCH v2 0/6] regulator: mcp16502: add support for ramp delay"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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-mcp16502-add-support-for-ramp-delay/20201111-170928
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/d97d6b70286fa3a28e744a55da1b7eb6a026cee3
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@xxxxxxxxx>

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':
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean 'step'?
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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> drivers/regulator/mcp16502.c:295:54: error: 'const struct linear_range' has no member named 'uV_step'; did you mean '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 -
| ^~~
>> include/linux/kernel.h:281:3: error: first argument to '__builtin_choose_expr' not a constant
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;
> 295 uV_delta = abs(new_sel * rdev->desc->linear_ranges->uV_step -
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@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

Next message: Benjamin Tissoires: "Re: [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'"
Previous message: Dmitry Torokhov: "Re: [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables"
In reply to: Claudiu Beznea: "[PATCH v2 5/6] regulator: mcp16502: add support for ramp delay"
Next in thread: Mark Brown: "Re: [PATCH v2 0/6] regulator: mcp16502: add support for ramp delay"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]