Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755856AbaFKP2Y (ORCPT ); Wed, 11 Jun 2014 11:28:24 -0400 Received: from mail-pb0-f42.google.com ([209.85.160.42]:44656 "EHLO mail-pb0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752003AbaFKP2W (ORCPT ); Wed, 11 Jun 2014 11:28:22 -0400 Message-ID: <5398758F.4080306@samsung.com> Date: Thu, 12 Jun 2014 00:28:15 +0900 From: Kukjin Kim User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.24) Gecko/20111108 Fedora/3.1.16-1.fc14 Lightning/1.0b3pre Thunderbird/3.1.16 MIME-Version: 1.0 To: Doug Anderson CC: Chander Kashyap , Kukjin Kim , Nicolas Pitre , "linux-arm-kernel@lists.infradead.org" , "linux@arm.linux.org.uk" , Kevin Hilman , Andrew Bresticker , Inderpal Singh , "linux-kernel@vger.kernel.org" , Chander Kashyap , "olof@lixom.net" , Tushar Behera , Lorenzo Pieralisi , Thomas Abraham , Abhilash Kesavan , Javier Martinez Canillas , "linux-samsung-soc@vger.kernel.org" Subject: Re: [PATCH] ARM: EXYNOS: mcpm: Don't rely on firmware's secondary_cpu_start References: <1402090985-8061-1-git-send-email-dianders@chromium.org> <20140607181221.GB25068@e102568-lin.cambridge.arm.com> <20140609223831.GB16889@e102568-lin.cambridge.arm.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/12/14 00:19, Doug Anderson wrote: > Chander, > > On Tue, Jun 10, 2014 at 9:52 PM, Chander Kashyap wrote: >> Hi Doug, >> >> On Tue, Jun 10, 2014 at 9:19 PM, Nicolas Pitre wrote: >>> On Tue, 10 Jun 2014, Doug Anderson wrote: >>> >>>> My S-state knowledge is not strong, but I believe that Lorenzo's >>>> questions matter if we're using S2 for CPUidle (where we actually turn >>>> off power and hot unplug CPUs) but not when we're using S1 for CPUidle >>>> (where we just enter WFI/WFE). >>>> >> >> No Its not plain WFI. >> >> All cores in Exynos5420 can be powered off independently. >> This functionality has been tested. >> >> Below is the link for the posted patches. >> >> https://lkml.org/lkml/2014/6/10/194 >> >> And as Nicolas wrote, these patches need MCPM for that. > > Most excellent! I should have been more clear that I only knew about > how CPUidle worked in our local production kernel. There I'm pretty > sure CPUidle is just WFI/WFE. If you've got patches to do better then > that's great! > > ...can you confirm that my patch doesn't interfere with your improved > CPUidle? It's been Acked by Nicolas (thanks!) so I'd imagine it will > land shortly. Kukjin: I assume you'll be taking this? > Sure, I will ;-) Thanks, Kukjin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/