From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Tue, 17 Jun 2014 17:20:22 +0200 Subject: [PATCH 01/14] ARM: mvebu: fix randconfig builds for pmsu driver In-Reply-To: <20140617171700.27908c03@free-electrons.com> References: <1402675276-538682-1-git-send-email-arnd@arndb.de> <4575903.g78pJlsGh9@wuerfel> <20140617171700.27908c03@free-electrons.com> Message-ID: <7711465.T7plxBX24Q@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday 17 June 2014 17:17:00 Thomas Petazzoni wrote: > > > > This unfortunately isn't really correct, and the patch I sent in > > > "[PATCH] ARM: mvebu: select ARM_CPU_SUSPEND for Marvell EBU v7 > > > platforms" on June, 11th already fixes the cpu_suspend()/cpu_resume() > > > problem. > > > > ok. > > > > > The issue with your solution is that it assumes this function is only > > > used for cpuidle. That is indeed the case in v3.16, but will be wrong > > > as soon as we add CPU hotplug support, which I have posted on May, 30th > > > in "[PATCH 0/5] CPU hotplug for Marvell Armada XP", and which I hope to > > > see included in v3.17. > > > > Right. I'll drop this patch from the submission then. You came just in > > time, as I had originally planned to send it out this morning. > > > > I'll also follow up with a separate patch to address the isb() issue. > > Ok, thanks a lot! > > So, should my patch adding the "select ARM_CPU_SUSPEND" be taken by > Jason and then pushed to you, or will you apply it directly? I'll wait for Jason to pick it up. It was clearly a mistake for me to include my own patch directly and without an explicit Ack, I'll go back to the normal way of doing things ;-) ARnd