From: kernel test robot <lkp@intel.com>
To: Rosen Penev <rosenp@gmail.com>, linux-mediatek@lists.infradead.org
Subject: Re: [PATCH] soc: mediatek: mtk-dvfsrc: use flex array
Date: Fri, 6 Mar 2026 12:45:48 +0800 [thread overview]
Message-ID: <202603061220.aCSWpwU6-lkp@intel.com> (raw)
In-Reply-To: <20260304232014.164408-1-rosenp@gmail.com>
Hi Rosen,
kernel test robot noticed the following build errors:
[auto build test ERROR on kees/for-next/kspp]
[also build test ERROR on linus/master v7.0-rc2 next-20260305]
[cannot apply to kees/for-next/pstore]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Rosen-Penev/soc-mediatek-mtk-dvfsrc-use-flex-array/20260305-072355
base: https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/kspp
patch link: https://lore.kernel.org/r/20260304232014.164408-1-rosenp%40gmail.com
patch subject: [PATCH] soc: mediatek: mtk-dvfsrc: use flex array
config: arm-allyesconfig (https://download.01.org/0day-ci/archive/20260306/202603061220.aCSWpwU6-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 15.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260306/202603061220.aCSWpwU6-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603061220.aCSWpwU6-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from include/linux/build_bug.h:5,
from include/linux/init.h:5,
from include/linux/arm-smccc.h:9,
from drivers/soc/mediatek/mtk-dvfsrc.c:8:
drivers/soc/mediatek/mtk-dvfsrc.c: In function 'dvfsrc_get_hw_opps_v4':
>> drivers/soc/mediatek/mtk-dvfsrc.c:522:66: error: 'num_ops' undeclared (first use in this function); did you mean 'num_opps'?
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~
include/linux/compiler.h:326:55: note: in definition of macro '__is_constexpr'
326 | (sizeof(int) == sizeof(*(8 ? ((void *)((long)(x) * 0l)) : (int *)8)))
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:522:42: note: in expansion of macro 'struct_size'
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:522:66: note: each undeclared identifier is reported only once for each function it appears in
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~
include/linux/compiler.h:326:55: note: in definition of macro '__is_constexpr'
326 | (sizeof(int) == sizeof(*(8 ? ((void *)((long)(x) * 0l)) : (int *)8)))
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:522:42: note: in expansion of macro 'struct_size'
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~~~~~
In file included from include/linux/string.h:13,
from include/linux/uuid.h:11,
from include/linux/arm-smccc.h:12:
>> include/linux/overflow.h:419:9: error: first argument to '__builtin_choose_expr' not a constant
419 | __builtin_choose_expr(__is_constexpr(count), \
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/overflow.h:436:32: note: in expansion of macro 'flex_array_size'
436 | sizeof(*(p)) + flex_array_size(p, member, count), \
| ^~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:522:42: note: in expansion of macro 'struct_size'
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~~~~~
>> include/linux/overflow.h:419:9: error: first argument to '__builtin_choose_expr' not a constant
419 | __builtin_choose_expr(__is_constexpr(count), \
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/overflow.h:437:40: note: in expansion of macro 'flex_array_size'
437 | size_add(sizeof(*(p)), flex_array_size(p, member, count)))
| ^~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:522:42: note: in expansion of macro 'struct_size'
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~~~~~
include/linux/overflow.h:435:9: error: first argument to '__builtin_choose_expr' not a constant
435 | __builtin_choose_expr(__is_constexpr(count), \
| ^~~~~~~~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:522:42: note: in expansion of macro 'struct_size'
522 | desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
| ^~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c: At top level:
>> drivers/soc/mediatek/mtk-dvfsrc.c:726:25: error: initialization of 'unsigned int' from 'const struct dvfsrc_opp *' makes integer from pointer without a cast [-Wint-conversion]
726 | .opps = dvfsrc_opp_mt6893_lp4,
| ^~~~~~~~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:726:25: note: (near initialization for 'dvfsrc_opp_mt6893_desc[0].opps[0].vcore_opp')
>> drivers/soc/mediatek/mtk-dvfsrc.c:727:17: error: initialization of flexible array member in a nested context
727 | .num_opp = ARRAY_SIZE(dvfsrc_opp_mt6893_lp4),
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:727:17: note: (near initialization for 'dvfsrc_opp_mt6893_desc[0].opps')
drivers/soc/mediatek/mtk-dvfsrc.c:724:64: warning: missing braces around initializer [-Wmissing-braces]
724 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt6893_desc[] = {
| ^
725 | [0] = {
726 | .opps = dvfsrc_opp_mt6893_lp4,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:724:64: warning: missing braces around initializer [-Wmissing-braces]
724 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt6893_desc[] = {
| ^
725 | [0] = {
726 | .opps = dvfsrc_opp_mt6893_lp4,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:759:25: error: initialization of 'unsigned int' from 'const struct dvfsrc_opp *' makes integer from pointer without a cast [-Wint-conversion]
759 | .opps = dvfsrc_opp_mt8183_lp4,
| ^~~~~~~~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:759:25: note: (near initialization for 'dvfsrc_opp_mt8183_desc[0].opps[0].vcore_opp')
drivers/soc/mediatek/mtk-dvfsrc.c:760:17: error: initialization of flexible array member in a nested context
760 | .num_opp = ARRAY_SIZE(dvfsrc_opp_mt8183_lp4),
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:760:17: note: (near initialization for 'dvfsrc_opp_mt8183_desc[0].opps')
drivers/soc/mediatek/mtk-dvfsrc.c:757:64: warning: missing braces around initializer [-Wmissing-braces]
757 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt8183_desc[] = {
| ^
758 | [0] = {
759 | .opps = dvfsrc_opp_mt8183_lp4,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:763:25: error: initialization of 'unsigned int' from 'const struct dvfsrc_opp *' makes integer from pointer without a cast [-Wint-conversion]
763 | .opps = dvfsrc_opp_mt8183_lp3,
| ^~~~~~~~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:763:25: note: (near initialization for 'dvfsrc_opp_mt8183_desc[1].opps[0].vcore_opp')
drivers/soc/mediatek/mtk-dvfsrc.c:764:17: error: initialization of flexible array member in a nested context
764 | .num_opp = ARRAY_SIZE(dvfsrc_opp_mt8183_lp3),
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:764:17: note: (near initialization for 'dvfsrc_opp_mt8183_desc[1].opps')
drivers/soc/mediatek/mtk-dvfsrc.c:757:64: warning: missing braces around initializer [-Wmissing-braces]
757 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt8183_desc[] = {
| ^
758 | [0] = {
759 | .opps = dvfsrc_opp_mt8183_lp4,
| {{ }}
......
763 | .opps = dvfsrc_opp_mt8183_lp3,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:767:25: error: initialization of 'unsigned int' from 'const struct dvfsrc_opp *' makes integer from pointer without a cast [-Wint-conversion]
767 | .opps = dvfsrc_opp_mt8183_lp3,
| ^~~~~~~~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:767:25: note: (near initialization for 'dvfsrc_opp_mt8183_desc[2].opps[0].vcore_opp')
drivers/soc/mediatek/mtk-dvfsrc.c:768:17: error: initialization of flexible array member in a nested context
768 | .num_opp = ARRAY_SIZE(dvfsrc_opp_mt8183_lp3),
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:768:17: note: (near initialization for 'dvfsrc_opp_mt8183_desc[2].opps')
drivers/soc/mediatek/mtk-dvfsrc.c:757:64: warning: missing braces around initializer [-Wmissing-braces]
757 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt8183_desc[] = {
| ^
758 | [0] = {
759 | .opps = dvfsrc_opp_mt8183_lp4,
| {{ }}
......
763 | .opps = dvfsrc_opp_mt8183_lp3,
| {{ }}
......
767 | .opps = dvfsrc_opp_mt8183_lp3,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:757:64: warning: missing braces around initializer [-Wmissing-braces]
757 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt8183_desc[] = {
| ^
758 | [0] = {
759 | .opps = dvfsrc_opp_mt8183_lp4,
| {{ }}
......
763 | .opps = dvfsrc_opp_mt8183_lp3,
| {{ }}
......
767 | .opps = dvfsrc_opp_mt8183_lp3,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:799:25: error: initialization of 'unsigned int' from 'const struct dvfsrc_opp *' makes integer from pointer without a cast [-Wint-conversion]
799 | .opps = dvfsrc_opp_mt8195_lp4,
| ^~~~~~~~~~~~~~~~~~~~~
drivers/soc/mediatek/mtk-dvfsrc.c:799:25: note: (near initialization for 'dvfsrc_opp_mt8195_desc[0].opps[0].vcore_opp')
drivers/soc/mediatek/mtk-dvfsrc.c:800:17: error: initialization of flexible array member in a nested context
800 | .num_opp = ARRAY_SIZE(dvfsrc_opp_mt8195_lp4),
| ^
drivers/soc/mediatek/mtk-dvfsrc.c:800:17: note: (near initialization for 'dvfsrc_opp_mt8195_desc[0].opps')
drivers/soc/mediatek/mtk-dvfsrc.c:797:64: warning: missing braces around initializer [-Wmissing-braces]
797 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt8195_desc[] = {
| ^
798 | [0] = {
799 | .opps = dvfsrc_opp_mt8195_lp4,
| {{ }}
drivers/soc/mediatek/mtk-dvfsrc.c:797:64: warning: missing braces around initializer [-Wmissing-braces]
797 | static const struct dvfsrc_opp_desc dvfsrc_opp_mt8195_desc[] = {
| ^
798 | [0] = {
799 | .opps = dvfsrc_opp_mt8195_lp4,
| {{ }}
vim +522 drivers/soc/mediatek/mtk-dvfsrc.c
489
490 static int dvfsrc_get_hw_opps_v4(struct mtk_dvfsrc *dvfsrc)
491 {
492 struct dvfsrc_opp_desc *desc;
493 u32 num_opps, gear_info;
494 u8 num_vcore, num_dram;
495 u8 num_emi;
496 int i;
497
498 num_opps = dvfsrc_get_opp_count_v4(dvfsrc);
499 if (num_opps == 0) {
500 dev_err(dvfsrc->dev, "No OPPs programmed in DVFSRC MCU.\n");
501 return -EINVAL;
502 }
503
504 /*
505 * The first 16 bits set in the gear info table says how many OPPs
506 * and how many vcore, dram and emi table entries are available.
507 */
508 gear_info = dvfsrc_readl(dvfsrc, DVFSRC_GEAR_INFO_L);
509 if (gear_info == 0) {
510 dev_err(dvfsrc->dev, "No gear info in DVFSRC MCU.\n");
511 return -EINVAL;
512 }
513
514 num_vcore = FIELD_GET(DVFSRC_V4_GEAR_INFO_VCORE, gear_info) + 1;
515 num_dram = FIELD_GET(DVFSRC_V4_GEAR_INFO_DRAM, gear_info) + 1;
516 num_emi = FIELD_GET(DVFSRC_V4_GEAR_INFO_EMI, gear_info) + 1;
517 dev_info(dvfsrc->dev,
518 "Discovered %u gears and %u vcore, %u dram, %u emi table entries.\n",
519 num_opps, num_vcore, num_dram, num_emi);
520
521 /* Allocate everything now as anything else after that cannot fail */
> 522 desc = devm_kzalloc(dvfsrc->dev, struct_size(desc, opps, num_ops + 1), GFP_KERNEL);
523 if (!desc)
524 return -ENOMEM;
525
526 desc->num_opp = num_opps + 1;
527
528 /* Read the OPP table gear indices */
529 for (i = 0; i <= num_opps; i++) {
530 gear_info = dvfsrc_get_opp_gear(dvfsrc, num_opps - i);
531 desc->opps[i].vcore_opp = FIELD_GET(DVFSRC_V4_GEAR_INFO_VCORE, gear_info);
532 desc->opps[i].dram_opp = FIELD_GET(DVFSRC_V4_GEAR_INFO_DRAM, gear_info);
533 desc->opps[i].emi_opp = FIELD_GET(DVFSRC_V4_GEAR_INFO_EMI, gear_info);
534 };
535
536 /* Assign to main structure now that everything is done! */
537 dvfsrc->curr_opps = desc;
538
539 return 0;
540 }
541
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2026-03-06 4:46 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-04 23:20 [PATCH] soc: mediatek: mtk-dvfsrc: use flex array Rosen Penev
2026-03-05 17:13 ` kernel test robot
2026-03-05 17:57 ` kernel test robot
2026-03-06 4:20 ` Gustavo A. R. Silva
2026-03-06 19:25 ` Rosen Penev
2026-03-06 4:45 ` kernel test robot [this message]
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=202603061220.aCSWpwU6-lkp@intel.com \
--to=lkp@intel.com \
--cc=linux-mediatek@lists.infradead.org \
--cc=rosenp@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.