MIME-Version: 1.0
In-Reply-To: <3226381.VKfgCFa2AC@wuerfel>
References: <201603291505.HKmOMR4P%fengguang.wu@intel.com>
	<3226381.VKfgCFa2AC@wuerfel>
Date: Thu, 31 Mar 2016 19:25:35 +0900
Message-ID: <CAK7LNAS5AE9daz5F8OKAWgPHKGLB-px3-yi-hgSBfjjAUyxYnQ@mail.gmail.com>
Subject: Re: [PATCH 2/2] ARM: uniphier: initialize outer cache for secondary CPUs
From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: kbuild test robot <lkp@intel.com>, Russell King <linux@arm.linux.org.uk>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        arm@kernel.org, kbuild-all@01.org,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 964
Lines: 30

Hi Arnd,

2016-03-29 17:11 GMT+09:00 Arnd Bergmann <arnd@arndb.de>:
> On Tuesday 29 March 2016 15:59:00 kbuild test robot wrote:
>>
>> All warnings (new ones prefixed by >>):
>>
>> >> WARNING: vmlinux.o(.text+0x557b4): Section mismatch in reference from the function uniphier_cache_secondary_init() to the function .init.text:uniphier_cache_activate_all_ways()
>>    The function uniphier_cache_secondary_init() references
>>    the function __init uniphier_cache_activate_all_ways().
>>    This is often because uniphier_cache_secondary_init lacks a __init
>>    annotation or the annotation of uniphier_cache_activate_all_ways is wrong.
>>
>
>
> I guess the former: uniphier_cache_secondary_init should be __init, as it will
> only be run at boot time.
>
> Please resend both patches.
>

I think the alternative solution suggested by Russell is much better.

I hope Russell will apply his one, and then I retract this series.


-- 
Best Regards
Masahiro Yamada