From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [PATCH] ARM: fix exynos randconfig build error Date: Tue, 03 Mar 2015 13:44:34 +0100 Message-ID: <14257250.yOR9pFcIkO@amdc1032> References: <54EF9091.30802@kernel.org> <54F4BC92.2050405@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7Bit Return-path: Received: from mailout2.samsung.com ([203.254.224.25]:16593 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751625AbbCCMpR (ORCPT ); Tue, 3 Mar 2015 07:45:17 -0500 Received: from epcpsbgm1.samsung.com (epcpsbgm1 [203.254.230.26]) by mailout2.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0NKM000UDZFFC020@mailout2.samsung.com> for linux-samsung-soc@vger.kernel.org; Tue, 03 Mar 2015 21:45:15 +0900 (KST) In-reply-to: <54F4BC92.2050405@kernel.org> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Kukjin Kim Cc: Krzysztof Kozlowski , Russell King , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org Hi, On Tuesday, March 03, 2015 04:40:02 AM Kukjin Kim wrote: > On 02/27/15 06:30, Kukjin Kim wrote: > > On 02/25/15 20:46, Krzysztof Kozlowski wrote: > >> 2015-02-25 12:26 GMT+01:00 Russell King : > >>> The following error was observed with SMP=n in v4.0-rc1: > >>> > >>> arch/arm/mach-exynos/pm.c: In function 'exynos_cpu0_enter_aftr': > >>> arch/arm/mach-exynos/pm.c:246:4: error: implicit declaration of function 'arch_send_wakeup_ipi_mask' [-Werror=implicit-function-declaration] > >>> > >>> As the code unconditionally calls a function only available with SMP=y, > >>> make the Exynos PM support depend on SMP. > >>> > >>> Signed-off-by: Russell King > >> > >> Hi, > >> > >> Thanks for the patch but this already waits for Kukjin top be picked > >> up. The first patch was similar to yours (adds dependency on SMP), > >> sent on 4th of February: > >> https://patchwork.ozlabs.org/patch/436231/ > >> > >> But later Bartlomiej fixed this in other way (allowing to use cpuidle > >> on non-SMP): > >> https://patchwork.ozlabs.org/patch/436445/ > >> > >> Unfortunately none of them were picked up. > >> > > I've missed the fix, sorry. > > > > BTW, as you know, all of exynos SoCs are based on SMP so generally (in > > normal case) there is no reason to use non-SMP on exynos > > platforms...even though I understand the build error should be fixed... > > > > Anyway, I'll have a look Bart's patch and Russell's fix in this weekend. > > > > Firstly, let me take rmk's patch for the randconfig build error...BTW What is wrong with picking my patch instead? It is non-invasive and fixes cpuidle support on UP (which is a regression from previous kernels)? https://lkml.org/lkml/2015/2/4/521 > I'm still wondering exynos stuff needs to support non-SMP and need to > think more about its usefulness?... Currently UP is supported and at least I find it useful for testing/debug purposes. If you want to to make Exynos SMP only thats OK but it should be done globally for Exynos arch support not just for cpuidle support. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics