From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olof Johansson Subject: Re: [PATCH 3/6] ARM: SAMSUNG: pm: Adjust for pinctrl- and DT-enabled platforms Date: Tue, 11 Jun 2013 00:45:52 -0700 Message-ID: <20130611074552.GK6688@quad.lixom.net> References: <1368807872-2601-1-git-send-email-t.figa@samsung.com> <1368807872-2601-4-git-send-email-t.figa@samsung.com> <62314159.5vrvK4k2Mm@amdc1227> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pb0-f41.google.com ([209.85.160.41]:59215 "EHLO mail-pb0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751649Ab3FKHps (ORCPT ); Tue, 11 Jun 2013 03:45:48 -0400 Received: by mail-pb0-f41.google.com with SMTP id rp16so4655225pbb.0 for ; Tue, 11 Jun 2013 00:45:48 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Linus Walleij Cc: Tomasz Figa , "arm@kernel.org" , linux-samsung-soc , Kukjin Kim , "linux-arm-kernel@lists.infradead.org" , Arnd Bergmann , Doug Anderson , Heiko =?iso-8859-1?Q?St=FCbner?= , Stephen Warren , Thomas Abraham , Prathyush K , Kyungmin Park On Mon, Jun 10, 2013 at 06:13:10PM +0200, Linus Walleij wrote: > On Mon, Jun 10, 2013 at 4:45 PM, Tomasz Figa wrote: > > On Friday 17 of May 2013 18:24:29 Tomasz Figa wrote: > >> This patch makes legacy code on suspend/resume path being executed > >> conditionally, on non-DT platforms only, to fix suspend/resume of > >> DT-enabled systems, for which the code is inappropriate. > >> > >> Signed-off-by: Tomasz Figa > >> Signed-off-by: Kyungmin Park > >> --- > >> arch/arm/plat-samsung/pm.c | 17 ++++++++++++----- > >> 1 file changed, 12 insertions(+), 5 deletions(-) > > > > It seems like this patch did not make it into 3.10-rc5, while rest of patches > > did, which ended up with suspend/resume still being broken. > > > > What should we do in this case? > > I did not apply this patch to the pinctrl tree because it begins > with ARM: but it appears I promised to carry it anyway... > > I need an ACK from the ARM SoC people to do that though, > or you can take it through ARM SoC fixes directly. > > I would just ask Olof/Arnd to apply it directly as a separate > patch. It makes sense to take this through arm-soc since it's no longer dependent on anything else. Applied to fixes. Please do follow through with the promised cleanups at some point, Tomasz. :-) -Olof From mboxrd@z Thu Jan 1 00:00:00 1970 From: olof@lixom.net (Olof Johansson) Date: Tue, 11 Jun 2013 00:45:52 -0700 Subject: [PATCH 3/6] ARM: SAMSUNG: pm: Adjust for pinctrl- and DT-enabled platforms In-Reply-To: References: <1368807872-2601-1-git-send-email-t.figa@samsung.com> <1368807872-2601-4-git-send-email-t.figa@samsung.com> <62314159.5vrvK4k2Mm@amdc1227> Message-ID: <20130611074552.GK6688@quad.lixom.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Jun 10, 2013 at 06:13:10PM +0200, Linus Walleij wrote: > On Mon, Jun 10, 2013 at 4:45 PM, Tomasz Figa wrote: > > On Friday 17 of May 2013 18:24:29 Tomasz Figa wrote: > >> This patch makes legacy code on suspend/resume path being executed > >> conditionally, on non-DT platforms only, to fix suspend/resume of > >> DT-enabled systems, for which the code is inappropriate. > >> > >> Signed-off-by: Tomasz Figa > >> Signed-off-by: Kyungmin Park > >> --- > >> arch/arm/plat-samsung/pm.c | 17 ++++++++++++----- > >> 1 file changed, 12 insertions(+), 5 deletions(-) > > > > It seems like this patch did not make it into 3.10-rc5, while rest of patches > > did, which ended up with suspend/resume still being broken. > > > > What should we do in this case? > > I did not apply this patch to the pinctrl tree because it begins > with ARM: but it appears I promised to carry it anyway... > > I need an ACK from the ARM SoC people to do that though, > or you can take it through ARM SoC fixes directly. > > I would just ask Olof/Arnd to apply it directly as a separate > patch. It makes sense to take this through arm-soc since it's no longer dependent on anything else. Applied to fixes. Please do follow through with the promised cleanups at some point, Tomasz. :-) -Olof