Re: [PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances

From: kbuild test robot
Date: Thu Sep 13 2018 - 16:40:11 EST

Next message: Tejun Heo: "[GIT PULL] percpu MAINTAINERS update"
Previous message: Robert WÄcÅawski: "[PATCH] staging: rtl8188eu: remove code that is valid only for 5 GHz"
In reply to: Matti Vaittinen: "[PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances"
Next in thread: Matti Vaittinen: "Re: [PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Matti,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on regulator/for-next]
[also build test ERROR on next-20180913]
[cannot apply to v4.19-rc3]
[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/Matti-Vaittinen/regulator-mfd-dt_bindings-bd718x7-Support-ROHM-bd71847/20180914-000129
base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
config: i386-randconfig-s0-201836 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

Note: the linux-review/Matti-Vaittinen/regulator-mfd-dt_bindings-bd718x7-Support-ROHM-bd71847/20180914-000129 HEAD 0df8e57cacf59e85ad40a25fbb1bd7e842608be8 builds fine.
It only hurts bisectibility.

All errors (new ones prefixed by >>):

>> drivers/regulator/bd71837-regulator.c:407:18: error: 'BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM' undeclared here (not in a function)
.n_voltages = BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/regulator/bd71837-regulator.c:735:18: error: initializer element is not constant
.n_voltages = BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/regulator/bd71837-regulator.c:735:18: note: (near initialization for 'bd71837_regulators[7].desc.n_voltages')

vim +/BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM +407 drivers/regulator/bd71837-regulator.c

280
281 static const struct bd718xx_regulator_data bd71847_regulators[] = {
282 {
283 .desc = {
284 .name = "buck1",
285 .of_match = of_match_ptr("BUCK1"),
286 .regulators_node = of_match_ptr("regulators"),
287 .id = BD718XX_BUCK1,
288 .ops = &bd718xx_dvs_buck_regulator_ops,
289 .type = REGULATOR_VOLTAGE,
290 .n_voltages = BD718XX_DVS_BUCK_VOLTAGE_NUM,
291 .linear_ranges = bd718xx_dvs_buck_volts,
292 .n_linear_ranges =
293 ARRAY_SIZE(bd718xx_dvs_buck_volts),
294 .vsel_reg = BD718XX_REG_BUCK1_VOLT_RUN,
295 .vsel_mask = DVS_BUCK_RUN_MASK,
296 .enable_reg = BD718XX_REG_BUCK1_CTRL,
297 .enable_mask = BD718XX_BUCK_EN,
298 .owner = THIS_MODULE,
299 },
300 .init = {
301 .reg = BD718XX_REG_BUCK1_CTRL,
302 .mask = BD718XX_BUCK_SEL,
303 .val = BD718XX_BUCK_SEL,
304 },
305 },
306 {
307 .desc = {
308 .name = "buck2",
309 .of_match = of_match_ptr("BUCK2"),
310 .regulators_node = of_match_ptr("regulators"),
311 .id = BD718XX_BUCK2,
312 .ops = &bd718xx_dvs_buck_regulator_ops,
313 .type = REGULATOR_VOLTAGE,
314 .n_voltages = BD718XX_DVS_BUCK_VOLTAGE_NUM,
315 .linear_ranges = bd718xx_dvs_buck_volts,
316 .n_linear_ranges = ARRAY_SIZE(bd718xx_dvs_buck_volts),
317 .vsel_reg = BD718XX_REG_BUCK2_VOLT_RUN,
318 .vsel_mask = DVS_BUCK_RUN_MASK,
319 .enable_reg = BD718XX_REG_BUCK2_CTRL,
320 .enable_mask = BD718XX_BUCK_EN,
321 .owner = THIS_MODULE,
322 },
323 .init = {
324 .reg = BD718XX_REG_BUCK2_CTRL,
325 .mask = BD718XX_BUCK_SEL,
326 .val = BD718XX_BUCK_SEL,
327 },
328 },
329 {
330 .desc = {
331 .name = "buck3",
332 .of_match = of_match_ptr("BUCK3"),
333 .regulators_node = of_match_ptr("regulators"),
334 .id = BD718XX_BUCK3,
335 .ops = &bd718xx_buck_regulator_ops,
336 .type = REGULATOR_VOLTAGE,
337 .n_voltages = BD718XX_1ST_NODVS_BUCK_VOLTAGE_NUM,
338 .linear_ranges = bd718xx_1st_nodvs_buck_volts,
339 .n_linear_ranges =
340 ARRAY_SIZE(bd718xx_1st_nodvs_buck_volts),
341 .vsel_reg = BD718XX_REG_1ST_NODVS_BUCK_VOLT,
342 .vsel_mask = BD718XX_1ST_NODVS_BUCK_MASK,
343 .enable_reg = BD718XX_REG_1ST_NODVS_BUCK_CTRL,
344 .enable_mask = BD718XX_BUCK_EN,
345 .owner = THIS_MODULE,
346 },
347 .init = {
348 .reg = BD718XX_REG_1ST_NODVS_BUCK_CTRL,
349 .mask = BD718XX_BUCK_SEL,
350 .val = BD718XX_BUCK_SEL,
351 },
352 },
353 {
354 .desc = {
355 .name = "buck4",
356 .of_match = of_match_ptr("BUCK4"),
357 .regulators_node = of_match_ptr("regulators"),
358 .id = BD718XX_BUCK4,
359 .ops = &bd718xx_buck_regulator_ops,
360 .type = REGULATOR_VOLTAGE,
361 .n_voltages = BD71847_BUCK4_VOLTAGE_NUM,
362 .linear_ranges = bd71847_buck4_voltage_ranges,
363 .n_linear_ranges =
364 ARRAY_SIZE(bd71847_buck4_voltage_ranges),
365 .enable_reg = BD718XX_REG_2ND_NODVS_BUCK_CTRL,
366 .vsel_reg = BD718XX_REG_2ND_NODVS_BUCK_VOLT,
367 .vsel_mask = BD71847_BUCK4_MASK,
368 .enable_mask = BD718XX_BUCK_EN,
369 .owner = THIS_MODULE,
370 },
371 .init = {
372 .reg = BD718XX_REG_2ND_NODVS_BUCK_CTRL,
373 .mask = BD718XX_BUCK_SEL,
374 .val = BD718XX_BUCK_SEL,
375 },
376 },
377 {
378 .desc = {
379 .name = "buck5",
380 .of_match = of_match_ptr("BUCK5"),
381 .regulators_node = of_match_ptr("regulators"),
382 .id = BD718XX_BUCK5,
383 .ops = &bd718xx_buck_regulator_nolinear_ops,
384 .type = REGULATOR_VOLTAGE,
385 .volt_table = &bd718xx_3rd_nodvs_buck_volts[0],
386 .n_voltages = ARRAY_SIZE(bd718xx_3rd_nodvs_buck_volts),
387 .vsel_reg = BD718XX_REG_3RD_NODVS_BUCK_VOLT,
388 .vsel_mask = BD718XX_3RD_NODVS_BUCK_MASK,
389 .enable_reg = BD718XX_REG_3RD_NODVS_BUCK_CTRL,
390 .enable_mask = BD718XX_BUCK_EN,
391 .owner = THIS_MODULE,
392 },
393 .init = {
394 .reg = BD718XX_REG_3RD_NODVS_BUCK_CTRL,
395 .mask = BD718XX_BUCK_SEL,
396 .val = BD718XX_BUCK_SEL,
397 },
398 },
399 {
400 .desc = {
401 .name = "buck6",
402 .of_match = of_match_ptr("BUCK6"),
403 .regulators_node = of_match_ptr("regulators"),
404 .id = BD718XX_BUCK6,
405 .ops = &bd718xx_buck_regulator_ops,
406 .type = REGULATOR_VOLTAGE,
> 407 .n_voltages = BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM,
408 .linear_ranges = bd718xx_4th_nodvs_buck_volts,
409 .n_linear_ranges =
410 ARRAY_SIZE(bd718xx_4th_nodvs_buck_volts),
411 .vsel_reg = BD718XX_REG_4TH_NODVS_BUCK_VOLT,
412 .vsel_mask = BD718XX_4TH_NODVS_BUCK_MASK,
413 .enable_reg = BD718XX_REG_4TH_NODVS_BUCK_CTRL,
414 .enable_mask = BD718XX_BUCK_EN,
415 .owner = THIS_MODULE,
416 },
417 .init = {
418 .reg = BD718XX_REG_4TH_NODVS_BUCK_CTRL,
419 .mask = BD718XX_BUCK_SEL,
420 .val = BD718XX_BUCK_SEL,
421 },
422 },
423 {
424 .desc = {
425 .name = "ldo1",
426 .of_match = of_match_ptr("LDO1"),
427 .regulators_node = of_match_ptr("regulators"),
428 .id = BD718XX_LDO1,
429 .ops = &bd718xx_ldo_regulator_ops,
430 .type = REGULATOR_VOLTAGE,
431 .n_voltages = BD718XX_LDO1_VOLTAGE_NUM,
432 .linear_ranges = bd718xx_ldo1_volts,
433 .n_linear_ranges = ARRAY_SIZE(bd718xx_ldo1_volts),
434 .vsel_reg = BD718XX_REG_LDO1_VOLT,
435 .vsel_mask = BD718XX_LDO1_MASK,
436 .enable_reg = BD718XX_REG_LDO1_VOLT,
437 .enable_mask = BD718XX_LDO_EN,
438 .owner = THIS_MODULE,
439 },
440 .init = {
441 .reg = BD718XX_REG_LDO1_VOLT,
442 .mask = BD718XX_LDO_SEL,
443 .val = BD718XX_LDO_SEL,
444 },
445 },
446 {
447 .desc = {
448 .name = "ldo2",
449 .of_match = of_match_ptr("LDO2"),
450 .regulators_node = of_match_ptr("regulators"),
451 .id = BD718XX_LDO2,
452 .ops = &bd718xx_ldo_regulator_nolinear_ops,
453 .type = REGULATOR_VOLTAGE,
454 .volt_table = &ldo_2_volts[0],
455 .vsel_reg = BD718XX_REG_LDO2_VOLT,
456 .vsel_mask = BD718XX_LDO2_MASK,
457 .n_voltages = ARRAY_SIZE(ldo_2_volts),
458 .enable_reg = BD718XX_REG_LDO2_VOLT,
459 .enable_mask = BD718XX_LDO_EN,
460 .owner = THIS_MODULE,
461 },
462 .init = {
463 .reg = BD718XX_REG_LDO2_VOLT,
464 .mask = BD718XX_LDO_SEL,
465 .val = BD718XX_LDO_SEL,
466 },
467 },
468 {
469 .desc = {
470 .name = "ldo3",
471 .of_match = of_match_ptr("LDO3"),
472 .regulators_node = of_match_ptr("regulators"),
473 .id = BD718XX_LDO3,
474 .ops = &bd718xx_ldo_regulator_ops,
475 .type = REGULATOR_VOLTAGE,
476 .n_voltages = BD718XX_LDO3_VOLTAGE_NUM,
477 .linear_ranges = bd718xx_ldo3_volts,
478 .n_linear_ranges = ARRAY_SIZE(bd718xx_ldo3_volts),
479 .vsel_reg = BD718XX_REG_LDO3_VOLT,
480 .vsel_mask = BD718XX_LDO3_MASK,
481 .enable_reg = BD718XX_REG_LDO3_VOLT,
482 .enable_mask = BD718XX_LDO_EN,
483 .owner = THIS_MODULE,
484 },
485 .init = {
486 .reg = BD718XX_REG_LDO3_VOLT,
487 .mask = BD718XX_LDO_SEL,
488 .val = BD718XX_LDO_SEL,
489 },
490 },
491 {
492 .desc = {
493 .name = "ldo4",
494 .of_match = of_match_ptr("LDO4"),
495 .regulators_node = of_match_ptr("regulators"),
496 .id = BD718XX_LDO4,
497 .ops = &bd718xx_ldo_regulator_ops,
498 .type = REGULATOR_VOLTAGE,
499 .n_voltages = BD718XX_LDO4_VOLTAGE_NUM,
500 .linear_ranges = bd718xx_ldo4_volts,
501 .n_linear_ranges = ARRAY_SIZE(bd718xx_ldo4_volts),
502 .vsel_reg = BD718XX_REG_LDO4_VOLT,
503 .vsel_mask = BD718XX_LDO4_MASK,
504 .enable_reg = BD718XX_REG_LDO4_VOLT,
505 .enable_mask = BD718XX_LDO_EN,
506 .owner = THIS_MODULE,
507 },
508 .init = {
509 .reg = BD718XX_REG_LDO4_VOLT,
510 .mask = BD718XX_LDO_SEL,
511 .val = BD718XX_LDO_SEL,
512 },
513 },
514 {
515 .desc = {
516 .name = "ldo5",
517 .of_match = of_match_ptr("LDO5"),
518 .regulators_node = of_match_ptr("regulators"),
519 .id = BD718XX_LDO5,
520 .ops = &bd718xx_ldo_regulator_ops,
521 .type = REGULATOR_VOLTAGE,
522 .n_voltages = BD718XX_LDO5_VOLTAGE_NUM,
523 .linear_ranges = bd718xx_ldo5_volts,
524 .n_linear_ranges = ARRAY_SIZE(bd718xx_ldo5_volts),
525 .vsel_reg = BD718XX_REG_LDO5_VOLT,
526 .vsel_mask = BD71847_LDO5_MASK,
527 .enable_reg = BD718XX_REG_LDO5_VOLT,
528 .enable_mask = BD718XX_LDO_EN,
529 .owner = THIS_MODULE,
530 },
531 .init = {
532 .reg = BD718XX_REG_LDO5_VOLT,
533 .mask = BD718XX_LDO_SEL,
534 .val = BD718XX_LDO_SEL,
535 },
536 },
537 {
538 .desc = {
539 .name = "ldo6",
540 .of_match = of_match_ptr("LDO6"),
541 .regulators_node = of_match_ptr("regulators"),
542 .id = BD718XX_LDO6,
543 .ops = &bd718xx_ldo_regulator_ops,
544 .type = REGULATOR_VOLTAGE,
545 .n_voltages = BD718XX_LDO6_VOLTAGE_NUM,
546 .linear_ranges = bd718xx_ldo6_volts,
547 .n_linear_ranges = ARRAY_SIZE(bd718xx_ldo6_volts),
548 /* LDO6 is supplied by buck5 */
549 .supply_name = "buck5",
550 .vsel_reg = BD718XX_REG_LDO6_VOLT,
551 .vsel_mask = BD718XX_LDO6_MASK,
552 .enable_reg = BD718XX_REG_LDO6_VOLT,
553 .enable_mask = BD718XX_LDO_EN,
554 .owner = THIS_MODULE,
555 },
556 .init = {
557 .reg = BD718XX_REG_LDO6_VOLT,
558 .mask = BD718XX_LDO_SEL,
559 .val = BD718XX_LDO_SEL,
560 },
561 },
562 };
563

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

Attachment: .config.gz
Description: application/gzip

Next message: Tejun Heo: "[GIT PULL] percpu MAINTAINERS update"
Previous message: Robert WÄcÅawski: "[PATCH] staging: rtl8188eu: remove code that is valid only for 5 GHz"
In reply to: Matti Vaittinen: "[PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances"
Next in thread: Matti Vaittinen: "Re: [PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]