Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757208AbaJaJzA (ORCPT ); Fri, 31 Oct 2014 05:55:00 -0400 Received: from mail-qg0-f46.google.com ([209.85.192.46]:35306 "EHLO mail-qg0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751211AbaJaJy6 (ORCPT ); Fri, 31 Oct 2014 05:54:58 -0400 MIME-Version: 1.0 In-Reply-To: <20141031093318.GS27405@n2100.arm.linux.org.uk> References: <1413795888-18559-1-git-send-email-k.kozlowski@samsung.com> <1413795888-18559-2-git-send-email-k.kozlowski@samsung.com> <1414746895.6797.3.camel@AMDC1943> <20141031093318.GS27405@n2100.arm.linux.org.uk> Date: Fri, 31 Oct 2014 10:54:58 +0100 Message-ID: Subject: Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option From: Ulf Hansson To: Russell King - ARM Linux Cc: Krzysztof Kozlowski , "Rafael J. Wysocki" , Len Brown , Pavel Machek , Jonathan Corbet , Dan Williams , Vinod Koul , Alan Stern , "linux-pm@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , dmaengine@vger.kernel.org, Lars-Peter Clausen , Michal Simek , Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 31 October 2014 10:33, Russell King - ARM Linux wrote: > On Fri, Oct 31, 2014 at 10:29:36AM +0100, Ulf Hansson wrote: >> On 31 October 2014 10:14, Krzysztof Kozlowski wrote: >> > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: >> >> Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime >> >> PM IRQ safe was set or not. >> >> >> >> Various bus drivers implementing runtime PM may use choose to suspend >> >> differently based on IRQ safeness status of child driver (e.g. do not >> >> unprepare the clock if IRQ safe is not set). >> >> >> >> Signed-off-by: Krzysztof Kozlowski >> >> Reviewed-by: Ulf Hansson >> > >> > Rafael, Len, Pavel, >> > >> > Is proposed API ok? Do you have any comments? >> > >> > I'll upload whole patchset to Russell's patch tracking system. However >> > an ack from PM maintainer is probably needed. >> >> I would actually prefer the opposite. It's better if we can take these >> patches through Rafaels tree. >> >> 1) I have posted patches for the amba bus, these may have merge >> conflicts with your changes. >> 2) We may start using the new runtime PM API within genpd as of now. > > Stop this broken thinking. The solution is simple. > > Put the generic runtime PM changes into the PM tree in a separate /clean/ > branch. Merge them into the PM-next branch. Notify me where I can > pull that /clean/ branch. > > I pull that clean branch into my tree, and then apply the AMBA specific > patches on top. > > That means Rafael has the PM changes, I have the dependent PM changes, > and I also have the AMBA changes, and no one ends up carrying changes > which are inappropriate. Okay, I have no concerns with this approach. I have also checked the potential upcoming merge conflict in the amba bus, since I doubt you will pull in bigger changes from his tree!? Anyway, that conflict should be trivial to resolve if that happens. Kind regards Uffe -- 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/