[rcar:gmsl/dev 23/24] drivers/regulator/gpio-regulator.c:245:54: warning: format specifies type 'int' but the argument has type 'long'
2:51 a.m.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/kbingham/rcar.git gmsl/dev
head: 73e3ee66deb28072d44a06d4996854442f049d23
commit: ad66c71305dbc51e01c94cf22fd62d179c34a52f [23/24] DNI: Regulator: Debug
config: riscv-randconfig-r015-20200715 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project
02946de3802d3bc65bc9f2eb9b8d4969b5a7add8)
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
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
git checkout ad66c71305dbc51e01c94cf22fd62d179c34a52f
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:148:
include/asm-generic/io.h:572:9: warning: performing pointer arithmetic on a null
pointer has undefined behavior [-Wnull-pointer-arithmetic]
return inl(addr);
^~~~~~~~~
arch/riscv/include/asm/io.h:56:76: note: expanded from macro 'inl'
#define inl(c) ({ u32 __v; __io_pbr(); __v = readl_cpu((void*)(PCI_IOBASE +
(c))); __io_par(__v); __v; })
~~~~~~~~~~ ^
arch/riscv/include/asm/mmio.h:95:76: note: expanded from macro 'readl_cpu'
#define readl_cpu(c) ({ u32 __r = le32_to_cpu((__force
__le32)__raw_readl(c)); __r; })
^
include/uapi/linux/byteorder/little_endian.h:34:51: note: expanded from macro
'__le32_to_cpu'
#define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
^
In file included from drivers/regulator/gpio-regulator.c:26:
In file included from include/linux/regulator/machine.h:16:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:10:
In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:13:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:148:
include/asm-generic/io.h:580:2: warning: performing pointer arithmetic on a null
pointer has undefined behavior [-Wnull-pointer-arithmetic]
outb(value, addr);
^~~~~~~~~~~~~~~~~
arch/riscv/include/asm/io.h:58:68: note: expanded from macro 'outb'
#define outb(v,c) ({ __io_pbw(); writeb_cpu((v),(void*)(PCI_IOBASE + (c)));
__io_paw(); })
~~~~~~~~~~ ^
arch/riscv/include/asm/mmio.h:97:52: note: expanded from macro 'writeb_cpu'
#define writeb_cpu(v, c) ((void)__raw_writeb((v), (c)))
^
In file included from drivers/regulator/gpio-regulator.c:26:
In file included from include/linux/regulator/machine.h:16:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:10:
In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:13:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:148:
include/asm-generic/io.h:588:2: warning: performing pointer arithmetic on a null
pointer has undefined behavior [-Wnull-pointer-arithmetic]
outw(value, addr);
^~~~~~~~~~~~~~~~~
arch/riscv/include/asm/io.h:59:68: note: expanded from macro 'outw'
#define outw(v,c) ({ __io_pbw(); writew_cpu((v),(void*)(PCI_IOBASE + (c)));
__io_paw(); })
~~~~~~~~~~ ^
arch/riscv/include/asm/mmio.h:98:76: note: expanded from macro 'writew_cpu'
#define writew_cpu(v, c) ((void)__raw_writew((__force u16)cpu_to_le16(v), (c)))
^
In file included from drivers/regulator/gpio-regulator.c:26:
In file included from include/linux/regulator/machine.h:16:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:10:
In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:13:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:148:
include/asm-generic/io.h:596:2: warning: performing pointer arithmetic on a null
pointer has undefined behavior [-Wnull-pointer-arithmetic]
outl(value, addr);
^~~~~~~~~~~~~~~~~
arch/riscv/include/asm/io.h:60:68: note: expanded from macro 'outl'
#define outl(v,c) ({ __io_pbw(); writel_cpu((v),(void*)(PCI_IOBASE + (c)));
__io_paw(); })
~~~~~~~~~~ ^
arch/riscv/include/asm/mmio.h:99:76: note: expanded from macro 'writel_cpu'
#define writel_cpu(v, c) ((void)__raw_writel((__force u32)cpu_to_le32(v), (c)))
^
In file included from drivers/regulator/gpio-regulator.c:26:
In file included from include/linux/regulator/machine.h:16:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:10:
In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:13:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:148:
include/asm-generic/io.h:1017:55: warning: performing pointer arithmetic on a null
pointer has undefined behavior [-Wnull-pointer-arithmetic]
return (port > MMIO_UPPER_LIMIT) ? NULL : PCI_IOBASE + port;
~~~~~~~~~~ ^
> drivers/regulator/gpio-regulator.c:245:54: warning: format
specifies type 'int' but the argument has type 'long' [-Wformat]
dev_err(dev, "OF_GET_GPIO_REGULATOR_CONFIG %d\n",
PTR_ERR(config));
~~
^~~~~~~~~~~~~~~
%ld
include/linux/dev_printk.h:104:32: note: expanded from macro 'dev_err'
_dev_err(dev, dev_fmt(fmt), ##__VA_ARGS__)
~~~ ^~~~~~~~~~~
8 warnings generated.
vim +245 drivers/regulator/gpio-regulator.c
223
224 static int gpio_regulator_probe(struct platform_device *pdev)
225 {
226 struct device *dev = &pdev->dev;
227 struct gpio_regulator_config *config = dev_get_platdata(dev);
228 struct device_node *np = dev->of_node;
229 struct gpio_regulator_data *drvdata;
230 struct regulator_config cfg = { };
231 struct regulator_dev *rdev;
232 enum gpiod_flags gflags;
233 int ptr, ret, state, i;
234
235 drvdata = devm_kzalloc(dev, sizeof(struct gpio_regulator_data),
236 GFP_KERNEL);
237 if (drvdata == NULL)
238 return -ENOMEM;
239
240 if (np) {
241 config = of_get_gpio_regulator_config(dev, np,
242 &drvdata->desc);
243
244 if (IS_ERR(config)) {
245 dev_err(dev, "OF_GET_GPIO_REGULATOR_CONFIG %d\n",
PTR_ERR(config));
246 return PTR_ERR(config);
247 }
248 }
249
250 dev_err(dev, "Probing GPIO Regulator\n");
251
252 drvdata->desc.name = devm_kstrdup(dev, config->supply_name, GFP_KERNEL);
253 if (drvdata->desc.name == NULL) {
254 dev_err(dev, "Failed to allocate supply name\n");
255 return -ENOMEM;
256 }
257
258 drvdata->gpiods = devm_kzalloc(dev, sizeof(struct gpio_desc *),
259 GFP_KERNEL);
260 if (!drvdata->gpiods)
261 return -ENOMEM;
262
263 for (i = 0; i < config->ngpios; i++) {
264 drvdata->gpiods[i] = devm_gpiod_get_index(dev,
265 NULL,
266 i,
267 config->gflags[i]);
268 if (IS_ERR(drvdata->gpiods[i]))
269 return PTR_ERR(drvdata->gpiods[i]);
270 /* This is good to know */
271 gpiod_set_consumer_name(drvdata->gpiods[i], drvdata->desc.name);
272 }
273 drvdata->nr_gpios = config->ngpios;
274
275 drvdata->states = devm_kmemdup(dev,
276 config->states,
277 config->nr_states *
278 sizeof(struct gpio_regulator_state),
279 GFP_KERNEL);
280 if (drvdata->states == NULL) {
281 dev_err(dev, "Failed to allocate state data\n");
282 return -ENOMEM;
283 }
284 drvdata->nr_states = config->nr_states;
285
286 drvdata->desc.owner = THIS_MODULE;
287 drvdata->desc.enable_time = config->startup_delay;
288
289 /* handle regulator type*/
290 switch (config->type) {
291 case REGULATOR_VOLTAGE:
292 drvdata->desc.type = REGULATOR_VOLTAGE;
293 drvdata->desc.ops = &gpio_regulator_voltage_ops;
294 drvdata->desc.n_voltages = config->nr_states;
295 break;
296 case REGULATOR_CURRENT:
297 drvdata->desc.type = REGULATOR_CURRENT;
298 drvdata->desc.ops = &gpio_regulator_current_ops;
299 break;
300 default:
301 dev_err(dev, "No regulator type set\n");
302 return -EINVAL;
303 }
304
305 /* build initial state from gpio init data. */
306 state = 0;
307 for (ptr = 0; ptr < drvdata->nr_gpios; ptr++) {
308 if (config->gflags[ptr] == GPIOD_OUT_HIGH)
309 state |= (1 << ptr);
310 }
311 drvdata->state = state;
312
313 cfg.dev = dev;
314 cfg.init_data = config->init_data;
315 cfg.driver_data = drvdata;
316 cfg.of_node = np;
317
318 /*
319 * The signal will be inverted by the GPIO core if flagged so in the
320 * decriptor.
321 */
322 if (config->enabled_at_boot)
323 gflags = GPIOD_OUT_HIGH | GPIOD_FLAGS_BIT_NONEXCLUSIVE;
324 else
325 gflags = GPIOD_OUT_LOW | GPIOD_FLAGS_BIT_NONEXCLUSIVE;
326
327 cfg.ena_gpiod = gpiod_get_optional(dev, "enable", gflags);
328 if (IS_ERR(cfg.ena_gpiod))
329 return PTR_ERR(cfg.ena_gpiod);
330
331 rdev = devm_regulator_register(dev, &drvdata->desc, &cfg);
332 if (IS_ERR(rdev)) {
333 ret = PTR_ERR(rdev);
334 dev_err(dev, "Failed to register regulator: %d\n", ret);
335 return ret;
336 }
337
338 platform_set_drvdata(pdev, drvdata);
339
340 return 0;
341 }
342
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
797
days inactive
797
days old
0 comments
1 participants
participants (1)
-
kernel test robot