From: kbuild test robot <lkp@intel.com>
To: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Cc: kbuild-all@01.org, lee.jones@linaro.org, robh+dt@kernel.org,
mark.rutland@arm.com, lgirdwood@gmail.com, broonie@kernel.org,
sboyd@kernel.org, mazziesaccount@gmail.com,
matti.vaittinen@fi.rohmeurope.com, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, heikki.haikola@fi.rohmeurope.com,
mikko.mutanen@fi.rohmeurope.com
Subject: Re: [PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances
Date: Fri, 14 Sep 2018 04:39:28 +0800 [thread overview]
Message-ID: <201809140420.mcXVA0yk%fengguang.wu@intel.com> (raw)
In-Reply-To: <74dc36388bc48a59191cc212094c85b157fd5197.1536750863.git.matti.vaittinen@fi.rohmeurope.com>
[-- Attachment #1: Type: text/plain, Size: 11766 bytes --]
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 #2: .config.gz --]
[-- Type: application/gzip, Size: 28786 bytes --]
next prev parent reply other threads:[~2018-09-13 20:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-12 11:43 [PATCH v2 0/7] regulator/mfd/dt_bindings: bd718x7: Support ROHM bd71847 Matti Vaittinen
2018-09-12 11:44 ` [PATCH v2 1/7] regulator: Support ROHM BD71847 power management IC Matti Vaittinen
2018-09-13 16:51 ` kbuild test robot
2018-09-12 11:47 ` [PATCH v2 2/7] regulator: dt bindings: add BD71847 device-tree binding documentation Matti Vaittinen
2018-09-12 11:49 ` [PATCH v2 3/7] mfd: " Matti Vaittinen
2018-09-12 11:50 ` [PATCH v2 4/7] regulator: Support regulators where voltage ranges are selectable Matti Vaittinen
2018-09-12 11:51 ` [PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances Matti Vaittinen
2018-09-13 20:39 ` kbuild test robot [this message]
2018-09-14 5:42 ` Matti Vaittinen
2018-09-12 11:52 ` [PATCH v2 6/7] regulator: bd718XX use pickable ranges Matti Vaittinen
2018-09-12 11:54 ` [PATCH v2 7/7] regulator: bd718xx: rename bd71837 to 718xx Matti Vaittinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201809140420.mcXVA0yk%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=heikki.haikola@fi.rohmeurope.com \
--cc=kbuild-all@01.org \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=matti.vaittinen@fi.rohmeurope.com \
--cc=mazziesaccount@gmail.com \
--cc=mikko.mutanen@fi.rohmeurope.com \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®