From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ulf Hansson Subject: Re: [PATCH 1/4] ARM: shmobile: Ignore callbacks for subsys generic_pm_domain_data Date: Fri, 2 May 2014 11:13:23 +0200 Message-ID: References: <1397230004-22306-1-git-send-email-ulf.hansson@linaro.org> <20140430102308.GA23426@verge.net.au> <1639184.jtW6HVkuRH@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Received: from mail-qa0-f42.google.com ([209.85.216.42]:39032 "EHLO mail-qa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751187AbaEBJNY (ORCPT ); Fri, 2 May 2014 05:13:24 -0400 Received: by mail-qa0-f42.google.com with SMTP id j5so1491791qaq.15 for ; Fri, 02 May 2014 02:13:23 -0700 (PDT) In-Reply-To: <1639184.jtW6HVkuRH@vostro.rjw.lan> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: "Rafael J. Wysocki" Cc: Simon Horman , Len Brown , Pavel Machek , "linux-pm@vger.kernel.org" , Kevin Hilman , Greg Kroah-Hartman , Tomasz Figa , Daniel Lezcano , Magnus Damm , Alan Stern , "linux-arm-kernel@lists.infradead.org" On 30 April 2014 22:21, Rafael J. Wysocki wrote: > On Wednesday, April 30, 2014 07:23:08 PM Simon Horman wrote: >> On Wed, Apr 30, 2014 at 10:53:58AM +0200, Ulf Hansson wrote: >> > On 30 April 2014 00:45, Rafael J. Wysocki wrote: >> > > On Friday, April 25, 2014 10:06:51 AM Simon Horman wrote: >> > >> On Thu, Apr 24, 2014 at 01:25:14PM +0200, Ulf Hansson wrote: >> > >> > On 14 April 2014 11:22, Ulf Hansson wrote: >> > >> > > On 14 April 2014 03:20, Simon Horman wrote: >> > >> > >> On Fri, Apr 11, 2014 at 05:26:41PM +0200, Ulf Hansson wrote: >> > >> > >>> There are no active users of these callbacks, thus there are no benefit >> > >> > >>> of trying to invoke them. >> > >> > >>> >> > >> > >>> Cc: Simon Horman >> > >> > >>> Cc: Magnus Damm >> > >> > >>> Signed-off-by: Ulf Hansson >> > >> > >>> --- >> > >> > >>> >> > >> > >>> I am seeking ack from the SOC maintainers since this needs to go with the other >> > >> > >>> patches in this patchset on the PM core. >> > >> > >> >> > >> > >> In order to reduce the possibility of conflicts I would prefer >> > >> > >> to take this patch through my tree. >> > >> > > >> > >> > > Okay, go ahead! >> > >> > > >> > >> > >> > >> > Hi Simon, >> > >> > >> > >> > Just wanted to know, if you decided to queue this patch in your tree. >> > >> > Or were there anything else you needed me to fix here? >> > >> >> > >> Sorry for dropping the ball. I will queue-up this patch. >> > > >> > > OK >> > > >> > > Ulf, since the rest of the series depends on this one (or at least that's >> > > how it looks to me), would you mind resending it when this one gets into the >> > > Linus' tree? >> > >> > Well, I would prefer if we could go ahead with the rest of the patches >> > somehow for 3.16, but I guess it will be hard if Simon picks up this >> > patch!? >> > >> > Simon, >> > I guess the reason for your concern were about conflicts, but the file >> > this patch touch has actually not been modified since Sep 2012. So, >> > maybe we could go through Rafaels tree anyway? >> >> Yes, my primary concern was conflicts. >> But as you point out that seems unlikely. >> >> I have already picked up the patch but the pull request I sent to >> the ARM SoC maintainers has not been pulled. And regardless it >> won't appear in Linus's tree via that route until v3.16-rc1. >> >> Perhaps a good way forwards would be for Rafael to also apply the patch >> with my Ack. > > Not really. No duplicate patches with different changelogs coming from > different trees. Please! > >> I could also revert the version applied to my tree >> if that is of any help. >> >> Acked-by: Simon Horman > > Ulf, can you please do as I said? Okay! It seems like I have no choice. :-) Does that mean you were happy with the rest from this patchset? Additionally, I intend to post some more patches for the PM domain soon. Thus I will need to fold in these patches to any other upcoming patchset. Thought it could be good to know. Kind regards Ulf Hansson > > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. From mboxrd@z Thu Jan 1 00:00:00 1970 From: ulf.hansson@linaro.org (Ulf Hansson) Date: Fri, 2 May 2014 11:13:23 +0200 Subject: [PATCH 1/4] ARM: shmobile: Ignore callbacks for subsys generic_pm_domain_data In-Reply-To: <1639184.jtW6HVkuRH@vostro.rjw.lan> References: <1397230004-22306-1-git-send-email-ulf.hansson@linaro.org> <20140430102308.GA23426@verge.net.au> <1639184.jtW6HVkuRH@vostro.rjw.lan> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 30 April 2014 22:21, Rafael J. Wysocki wrote: > On Wednesday, April 30, 2014 07:23:08 PM Simon Horman wrote: >> On Wed, Apr 30, 2014 at 10:53:58AM +0200, Ulf Hansson wrote: >> > On 30 April 2014 00:45, Rafael J. Wysocki wrote: >> > > On Friday, April 25, 2014 10:06:51 AM Simon Horman wrote: >> > >> On Thu, Apr 24, 2014 at 01:25:14PM +0200, Ulf Hansson wrote: >> > >> > On 14 April 2014 11:22, Ulf Hansson wrote: >> > >> > > On 14 April 2014 03:20, Simon Horman wrote: >> > >> > >> On Fri, Apr 11, 2014 at 05:26:41PM +0200, Ulf Hansson wrote: >> > >> > >>> There are no active users of these callbacks, thus there are no benefit >> > >> > >>> of trying to invoke them. >> > >> > >>> >> > >> > >>> Cc: Simon Horman >> > >> > >>> Cc: Magnus Damm >> > >> > >>> Signed-off-by: Ulf Hansson >> > >> > >>> --- >> > >> > >>> >> > >> > >>> I am seeking ack from the SOC maintainers since this needs to go with the other >> > >> > >>> patches in this patchset on the PM core. >> > >> > >> >> > >> > >> In order to reduce the possibility of conflicts I would prefer >> > >> > >> to take this patch through my tree. >> > >> > > >> > >> > > Okay, go ahead! >> > >> > > >> > >> > >> > >> > Hi Simon, >> > >> > >> > >> > Just wanted to know, if you decided to queue this patch in your tree. >> > >> > Or were there anything else you needed me to fix here? >> > >> >> > >> Sorry for dropping the ball. I will queue-up this patch. >> > > >> > > OK >> > > >> > > Ulf, since the rest of the series depends on this one (or at least that's >> > > how it looks to me), would you mind resending it when this one gets into the >> > > Linus' tree? >> > >> > Well, I would prefer if we could go ahead with the rest of the patches >> > somehow for 3.16, but I guess it will be hard if Simon picks up this >> > patch!? >> > >> > Simon, >> > I guess the reason for your concern were about conflicts, but the file >> > this patch touch has actually not been modified since Sep 2012. So, >> > maybe we could go through Rafaels tree anyway? >> >> Yes, my primary concern was conflicts. >> But as you point out that seems unlikely. >> >> I have already picked up the patch but the pull request I sent to >> the ARM SoC maintainers has not been pulled. And regardless it >> won't appear in Linus's tree via that route until v3.16-rc1. >> >> Perhaps a good way forwards would be for Rafael to also apply the patch >> with my Ack. > > Not really. No duplicate patches with different changelogs coming from > different trees. Please! > >> I could also revert the version applied to my tree >> if that is of any help. >> >> Acked-by: Simon Horman > > Ulf, can you please do as I said? Okay! It seems like I have no choice. :-) Does that mean you were happy with the rest from this patchset? Additionally, I intend to post some more patches for the PM domain soon. Thus I will need to fold in these patches to any other upcoming patchset. Thought it could be good to know. Kind regards Ulf Hansson > > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center.