Re: [PATCH] devfreq: Add COMPILE_TEST for build coverage

From: kbuild test robot
Date: Mon Jun 27 2016 - 14:38:01 EST


Hi,

[auto build test ERROR on v4.7-rc5]
[also build test ERROR on next-20160627]
[cannot apply to devfreq/for-rafael]
[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/Krzysztof-Kozlowski/devfreq-Add-COMPILE_TEST-for-build-coverage/20160627-220248
config: sh-allyesconfig (attached as .config)
compiler: sh4-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=sh

All errors (new ones prefixed by >>):

drivers/devfreq/tegra-devfreq.c: In function 'tegra_actmon_rate_notify_cb':
>> drivers/devfreq/tegra-devfreq.c:391:16: error: 'POST_RATE_CHANGE' undeclared (first use in this function)
if (action != POST_RATE_CHANGE)
^
drivers/devfreq/tegra-devfreq.c:391:16: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/devfreq/tegra-devfreq.c:396:24: error: dereferencing pointer to incomplete type 'struct clk_notifier_data'
tegra->cur_freq = data->new_rate / KHZ;
^
drivers/devfreq/tegra-devfreq.c: In function 'tegra_devfreq_probe':
>> drivers/devfreq/tegra-devfreq.c:654:8: error: implicit declaration of function 'clk_notifier_register' [-Werror=implicit-function-declaration]
err = clk_notifier_register(tegra->emc_clock, &tegra->rate_change_nb);
^
drivers/devfreq/tegra-devfreq.c: In function 'tegra_devfreq_remove':
>> drivers/devfreq/tegra-devfreq.c:734:2: error: implicit declaration of function 'clk_notifier_unregister' [-Werror=implicit-function-declaration]
clk_notifier_unregister(tegra->emc_clock, &tegra->rate_change_nb);
^
cc1: some warnings being treated as errors

vim +/POST_RATE_CHANGE +391 drivers/devfreq/tegra-devfreq.c

6234f380 Tomeu Vizoso 2014-11-24 385 struct clk_notifier_data *data = ptr;
11573e91 Tomeu Vizoso 2015-03-17 386 struct tegra_devfreq *tegra;
11573e91 Tomeu Vizoso 2015-03-17 387 struct tegra_devfreq_device *dev;
6234f380 Tomeu Vizoso 2014-11-24 388 unsigned int i;
6234f380 Tomeu Vizoso 2014-11-24 389 unsigned long flags;
6234f380 Tomeu Vizoso 2014-11-24 390
11573e91 Tomeu Vizoso 2015-03-17 @391 if (action != POST_RATE_CHANGE)
11573e91 Tomeu Vizoso 2015-03-17 392 return NOTIFY_OK;
11573e91 Tomeu Vizoso 2015-03-17 393
11573e91 Tomeu Vizoso 2015-03-17 394 tegra = container_of(nb, struct tegra_devfreq, rate_change_nb);
6234f380 Tomeu Vizoso 2014-11-24 395
6234f380 Tomeu Vizoso 2014-11-24 @396 tegra->cur_freq = data->new_rate / KHZ;
6234f380 Tomeu Vizoso 2014-11-24 397
11573e91 Tomeu Vizoso 2015-03-17 398 for (i = 0; i < ARRAY_SIZE(tegra->devices); i++) {
11573e91 Tomeu Vizoso 2015-03-17 399 dev = &tegra->devices[i];

:::::: The code at line 391 was first introduced by commit
:::::: 11573e9132ae40239195fe5162d48caf2b7d85bb PM / devfreq: tegra: Update to v5 of the submitted patches

:::::: TO: Tomeu Vizoso <tomeu.vizoso@xxxxxxxxxxxxx>
:::::: CC: MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>

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

Attachment: .config.gz
Description: Binary data