From: kernel test robot <lkp@intel.com>
To: "Duje Mihanović" <duje.mihanovic@skole.hr>,
"Michael Turquette" <mturquette@baylibre.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Rob Herring" <robh+dt@kernel.org>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>
Cc: oe-kbuild-all@lists.linux.dev, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
"Duje Mihanović" <duje.mihanovic@skole.hr>
Subject: Re: [PATCH 1/4] clk: mmp2: Move number of clocks into driver source
Date: Fri, 11 Aug 2023 10:17:22 +0800 [thread overview]
Message-ID: <202308110800.KkX1CemC-lkp@intel.com> (raw)
In-Reply-To: <20230809-mmp-nr-clks-v1-1-5f3cdbbb89b8@skole.hr>
Hi Duje,
kernel test robot noticed the following build errors:
[auto build test ERROR on 52a93d39b17dc7eb98b6aa3edb93943248e03b2f]
url: https://github.com/intel-lab-lkp/linux/commits/Duje-Mihanovi/clk-mmp2-Move-number-of-clocks-into-driver-source/20230809-202111
base: 52a93d39b17dc7eb98b6aa3edb93943248e03b2f
patch link: https://lore.kernel.org/r/20230809-mmp-nr-clks-v1-1-5f3cdbbb89b8%40skole.hr
patch subject: [PATCH 1/4] clk: mmp2: Move number of clocks into driver source
config: arm-allmodconfig (https://download.01.org/0day-ci/archive/20230811/202308110800.KkX1CemC-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 12.3.0
reproduce: (https://download.01.org/0day-ci/archive/20230811/202308110800.KkX1CemC-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/202308110800.KkX1CemC-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from include/linux/bits.h:5,
from include/linux/bitops.h:6,
from include/linux/of.h:15,
from include/linux/clk-provider.h:9,
from drivers/clk/mmp/clk-audio.c:8:
drivers/clk/mmp/clk-audio.c: In function 'mmp2_audio_clk_probe':
>> drivers/clk/mmp/clk-audio.c:354:41: error: 'MMP2_CLK_AUDIO_NR_CLKS' undeclared (first use in this function); did you mean 'MMP2_CLK_AUDIO_SYSCLK'?
354 | MMP2_CLK_AUDIO_NR_CLKS),
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/const.h:12:55: note: in definition of macro '__is_constexpr'
12 | (sizeof(int) == sizeof(*(8 ? ((void *)((long)(x) * 0l)) : (int *)8)))
| ^
drivers/clk/mmp/clk-audio.c:353:29: note: in expansion of macro 'struct_size'
353 | struct_size(priv, clk_data.hws,
| ^~~~~~~~~~~
drivers/clk/mmp/clk-audio.c:354:41: note: each undeclared identifier is reported only once for each function it appears in
354 | MMP2_CLK_AUDIO_NR_CLKS),
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/const.h:12:55: note: in definition of macro '__is_constexpr'
12 | (sizeof(int) == sizeof(*(8 ? ((void *)((long)(x) * 0l)) : (int *)8)))
| ^
drivers/clk/mmp/clk-audio.c:353:29: note: in expansion of macro 'struct_size'
353 | struct_size(priv, clk_data.hws,
| ^~~~~~~~~~~
In file included from include/linux/vmalloc.h:11,
from include/asm-generic/io.h:994,
from arch/arm/include/asm/io.h:416,
from include/linux/io.h:13,
from drivers/clk/mmp/clk-audio.c:9:
>> include/linux/overflow.h:276:9: error: first argument to '__builtin_choose_expr' not a constant
276 | __builtin_choose_expr(__is_constexpr(count), \
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/overflow.h:293:32: note: in expansion of macro 'flex_array_size'
293 | sizeof(*(p)) + flex_array_size(p, member, count), \
| ^~~~~~~~~~~~~~~
drivers/clk/mmp/clk-audio.c:353:29: note: in expansion of macro 'struct_size'
353 | struct_size(priv, clk_data.hws,
| ^~~~~~~~~~~
>> include/linux/overflow.h:276:9: error: first argument to '__builtin_choose_expr' not a constant
276 | __builtin_choose_expr(__is_constexpr(count), \
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/overflow.h:294:40: note: in expansion of macro 'flex_array_size'
294 | size_add(sizeof(*(p)), flex_array_size(p, member, count)))
| ^~~~~~~~~~~~~~~
drivers/clk/mmp/clk-audio.c:353:29: note: in expansion of macro 'struct_size'
353 | struct_size(priv, clk_data.hws,
| ^~~~~~~~~~~
include/linux/overflow.h:292:9: error: first argument to '__builtin_choose_expr' not a constant
292 | __builtin_choose_expr(__is_constexpr(count), \
| ^~~~~~~~~~~~~~~~~~~~~
drivers/clk/mmp/clk-audio.c:353:29: note: in expansion of macro 'struct_size'
353 | struct_size(priv, clk_data.hws,
| ^~~~~~~~~~~
vim +354 drivers/clk/mmp/clk-audio.c
725262d29139cc Lubomir Rintel 2020-05-20 346
725262d29139cc Lubomir Rintel 2020-05-20 347 static int mmp2_audio_clk_probe(struct platform_device *pdev)
725262d29139cc Lubomir Rintel 2020-05-20 348 {
725262d29139cc Lubomir Rintel 2020-05-20 349 struct mmp2_audio_clk *priv;
725262d29139cc Lubomir Rintel 2020-05-20 350 int ret;
725262d29139cc Lubomir Rintel 2020-05-20 351
725262d29139cc Lubomir Rintel 2020-05-20 352 priv = devm_kzalloc(&pdev->dev,
725262d29139cc Lubomir Rintel 2020-05-20 353 struct_size(priv, clk_data.hws,
725262d29139cc Lubomir Rintel 2020-05-20 @354 MMP2_CLK_AUDIO_NR_CLKS),
725262d29139cc Lubomir Rintel 2020-05-20 355 GFP_KERNEL);
725262d29139cc Lubomir Rintel 2020-05-20 356 if (!priv)
725262d29139cc Lubomir Rintel 2020-05-20 357 return -ENOMEM;
725262d29139cc Lubomir Rintel 2020-05-20 358
725262d29139cc Lubomir Rintel 2020-05-20 359 spin_lock_init(&priv->lock);
725262d29139cc Lubomir Rintel 2020-05-20 360 platform_set_drvdata(pdev, priv);
725262d29139cc Lubomir Rintel 2020-05-20 361
725262d29139cc Lubomir Rintel 2020-05-20 362 priv->mmio_base = devm_platform_ioremap_resource(pdev, 0);
725262d29139cc Lubomir Rintel 2020-05-20 363 if (IS_ERR(priv->mmio_base))
725262d29139cc Lubomir Rintel 2020-05-20 364 return PTR_ERR(priv->mmio_base);
725262d29139cc Lubomir Rintel 2020-05-20 365
725262d29139cc Lubomir Rintel 2020-05-20 366 pm_runtime_enable(&pdev->dev);
725262d29139cc Lubomir Rintel 2020-05-20 367 ret = pm_clk_create(&pdev->dev);
725262d29139cc Lubomir Rintel 2020-05-20 368 if (ret)
725262d29139cc Lubomir Rintel 2020-05-20 369 goto disable_pm_runtime;
725262d29139cc Lubomir Rintel 2020-05-20 370
725262d29139cc Lubomir Rintel 2020-05-20 371 ret = pm_clk_add(&pdev->dev, "audio");
725262d29139cc Lubomir Rintel 2020-05-20 372 if (ret)
725262d29139cc Lubomir Rintel 2020-05-20 373 goto destroy_pm_clk;
725262d29139cc Lubomir Rintel 2020-05-20 374
725262d29139cc Lubomir Rintel 2020-05-20 375 ret = register_clocks(priv, &pdev->dev);
725262d29139cc Lubomir Rintel 2020-05-20 376 if (ret)
725262d29139cc Lubomir Rintel 2020-05-20 377 goto destroy_pm_clk;
725262d29139cc Lubomir Rintel 2020-05-20 378
725262d29139cc Lubomir Rintel 2020-05-20 379 return 0;
725262d29139cc Lubomir Rintel 2020-05-20 380
725262d29139cc Lubomir Rintel 2020-05-20 381 destroy_pm_clk:
725262d29139cc Lubomir Rintel 2020-05-20 382 pm_clk_destroy(&pdev->dev);
725262d29139cc Lubomir Rintel 2020-05-20 383 disable_pm_runtime:
725262d29139cc Lubomir Rintel 2020-05-20 384 pm_runtime_disable(&pdev->dev);
725262d29139cc Lubomir Rintel 2020-05-20 385
725262d29139cc Lubomir Rintel 2020-05-20 386 return ret;
725262d29139cc Lubomir Rintel 2020-05-20 387 }
725262d29139cc Lubomir Rintel 2020-05-20 388
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-08-11 2:17 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-09 12:18 [PATCH 0/4] clk: marvell: Move number of clocks into driver source Duje Mihanović
2023-08-09 12:18 ` [PATCH 1/4] clk: mmp2: " Duje Mihanović
2023-08-11 2:17 ` kernel test robot [this message]
2023-08-09 12:18 ` [PATCH 2/4] clk: pxa168: Move number of clocks to " Duje Mihanović
2023-08-09 12:18 ` [PATCH 3/4] clk: pxa1928: " Duje Mihanović
2023-08-09 12:18 ` [PATCH 4/4] clk: pxa910: " Duje Mihanović
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=202308110800.KkX1CemC-lkp@intel.com \
--to=lkp@intel.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=duje.mihanovic@skole.hr \
--cc=krzk@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=oe-kbuild-all@lists.linux.dev \
--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;
as well as URLs for NNTP newsgroup(s).