Re: [PATCH v4 6/6] perf: ARM DynamIQ Shared Unit PMU support

From: Suzuki K Poulose <Suzuki.Poulose@arm.com>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, linux-arm-kernel@lists.infradead.org,
	will.deacon@arm.com, marc.zyngier@arm.com, mark.rutland@arm.com,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	robh@kernel.org, frowand.list@gmail.com,
	mathieu.poirier@linaro.org, peterz@infradead.org
Subject: Re: [PATCH v4 6/6] perf: ARM DynamIQ Shared Unit PMU support
Date: Tue, 8 Aug 2017 11:58:57 +0100	[thread overview]
Message-ID: <c1a7a969-312e-c6b7-519c-801da8dc0a8e@arm.com> (raw)
In-Reply-To: <201708081851.zkRaPAIn%fengguang.wu@intel.com>

On 08/08/17 11:57, kbuild test robot wrote:
> Hi Suzuki,
>
> [auto build test ERROR on linus/master]
> [also build test ERROR on v4.13-rc4 next-20170807]
> [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/Suzuki-K-Poulose/perf-Support-for-ARM-DynamIQ-Shared-Unit-PMU/20170808-124822
> config: arm64-allmodconfig (attached as .config)
> compiler: aarch64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> reproduce:
>         wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=arm64
>
> All errors (new ones prefixed by >>):
>
>    drivers//perf/arm_dsu_pmu.c: In function 'dsu_pmu_dt_get_cpus':
>>> drivers//perf/arm_dsu_pmu.c:685:9: error: implicit declaration of function 'of_device_node_get_cpu' [-Werror=implicit-function-declaration]
>       cpu = of_device_node_get_cpu(cpu_node);
>             ^~~~~~~~~~~~~~~~~~~~~~
>    cc1: some warnings being treated as errors
>
> vim +/of_device_node_get_cpu +685 drivers//perf/arm_dsu_pmu.c
>
>    669	
>    670	/**
>    671	 * dsu_pmu_dt_get_cpus: Get the list of CPUs in the cluster.
>    672	 */
>    673	static int dsu_pmu_dt_get_cpus(struct device_node *dev, cpumask_t *mask)
>    674	{
>    675		int i = 0, n, cpu;
>    676		struct device_node *cpu_node;
>    677	
>    678		n = of_count_phandle_with_args(dev, "cpus", NULL);
>    679		if (n <= 0)
>    680			return -ENODEV;
>    681		for (; i < n; i++) {
>    682			cpu_node = of_parse_phandle(dev, "cpus", i);
>    683			if (!cpu_node)
>    684				break;
>  > 685			cpu = of_device_node_get_cpu(cpu_node);

I have fixed this locally, in for v5.

Suzuki