Re: [PATCH v3] powerpc/64s: reimplement book3s idle code in C

[Nicholas Piggin <npiggin@gmail.com>]

On Fri, 31 Aug 2018 12:38:19 +0800
kbuild test robot <lkp@intel.com> wrote:

> Hi Nicholas,
> 
> I love your patch! Yet something to improve:
> 
> [auto build test ERROR on powerpc/next]
> [also build test ERROR on v4.19-rc1 next-20180830]
> [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/Nicholas-Piggin/powerpc-64s-reimplement-book3s-idle-code-in-C/20180829-014912
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
> config: powerpc-skiroot_defconfig (attached as .config)
> compiler: powerpc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.2.0 make.cross ARCH=powerpc 
> 
> All errors (new ones prefixed by >>):
> 
> >> arch/powerpc/platforms/powernv/idle.c:773:22: error: 'power9_offline_stop' defined but not used [-Werror=unused-function]  
>     static unsigned long power9_offline_stop(unsigned long psscr)
>                          ^~~~~~~~~~~~~~~~~~~
> >> arch/powerpc/platforms/powernv/idle.c:486:22: error: 'power7_offline' defined but not used [-Werror=unused-function]  
>     static unsigned long power7_offline(void)
>                          ^~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors
> 
> vim +/power9_offline_stop +773 arch/powerpc/platforms/powernv/idle.c
> 
>    772	
>  > 773	static unsigned long power9_offline_stop(unsigned long psscr)  
>    774	{
>    775		unsigned long srr1;
>    776	

Ahh, no hotplug. Good bot.

Thanks,
Nick