From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [FOR DISCUSSION 0/9] Dove PMU support Date: Fri, 20 Mar 2015 12:16:59 +0000 Message-ID: <20150320121659.GJ8656@n2100.arm.linux.org.uk> References: <20150312183020.GU8656@n2100.arm.linux.org.uk> <1575145.KH9tPKGaeR@vostro.rjw.lan> <1824813.kJQ02QuMbA@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1824813.kJQ02QuMbA@vostro.rjw.lan> Sender: linux-pm-owner@vger.kernel.org To: "Rafael J. Wysocki" Cc: Andrew Lunn , Jason Cooper , Sebastian Hesselbarth , Mark Rutland , devicetree@vger.kernel.org, Pawel Moll , Len Brown , Ian Campbell , Greg Kroah-Hartman , linux-pm@vger.kernel.org, Rob Herring , Kumar Gala , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Thu, Mar 19, 2015 at 11:02:35PM +0100, Rafael J. Wysocki wrote: > On Thursday, March 19, 2015 10:59:20 PM Rafael J. Wysocki wrote: > > On Thursday, March 12, 2015 06:30:21 PM Russell King - ARM Linux wrote: > > > This is a re-posting of the patch set which I posted almost 10 months > > > ago to support the Dove PMU, with a few additional changes. This set > > > is based upon 3.19. > > > > > > In this set are: > > > > > > * two patches which Rafael originally acked, but there was indecision > > > last time around how to handle them due to potential conflicts with > > > work that Ulf was doing. These patches have been updated to apply > > > cleanly to 3.19. I don't know if people want to take these as > > > fixes to the PM domain code or not (hence why I'm posting this > > > series during the merge window - if it weren't for this, I'd hold > > > it off.) > > > > > > * what I regard as a fix to the PM domain code; as a result of a > > > previous commit, the PM domain code mismatches the runtime PM state, > > > which leads to the PM domain being unexpectedly left on. This patch > > > has been re-worked to try an alternative approach, syncing the PM > > > domain state with the runtime PM state after the probe has completed. > > > > > > * the addition of the core Dove PMU driver, which consists of a reset, > > > IRQ controller, and power domains. The reset and power domain code > > > has to be closely related due to the power up/down requirements of > > > the GPU/VPU subsystems needing to be performed atomically. (This > > > requirement prevents it using the MFD infrastructure, because we > > > would need to hold spinlocks while calling several different > > > sub-drivers.) > > > > > > * addition of the RTC interrupt, so we can now receive and act on > > > alarms generated by the Dove RTC. > > > > > > * addition of the DT descriptions for the GPU and VPU power domains. > > > These patches do not themselves add the DT descriptions for these > > > units, so these patches serve as illustrations how these should be > > > described. > > > > > > > I can apply patches [1-3/9] from this series if that helps. > > I mean from the next version of it ([FOR DISCUSSION 0/10] Dove PMU support). Kevin had some comments on patch 2 which I think ought to be addressed. He's quite right that dev_to_genpd() should be hidden from view and replaced with the safer pm_genpd_lookup_dev() version - even though pm_genpd_lookup_dev() itself is not fully safe against the domain on a device changing beneath it. If you want to hold on, I'll respin with his comments there addressed, or if you prefer to merge them as-is and fix Kevin's comments afterwards, that's also fine by me. I'd just be happy to see some progress on this series. :) Thanks. -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net. From mboxrd@z Thu Jan 1 00:00:00 1970 From: linux@arm.linux.org.uk (Russell King - ARM Linux) Date: Fri, 20 Mar 2015 12:16:59 +0000 Subject: [FOR DISCUSSION 0/9] Dove PMU support In-Reply-To: <1824813.kJQ02QuMbA@vostro.rjw.lan> References: <20150312183020.GU8656@n2100.arm.linux.org.uk> <1575145.KH9tPKGaeR@vostro.rjw.lan> <1824813.kJQ02QuMbA@vostro.rjw.lan> Message-ID: <20150320121659.GJ8656@n2100.arm.linux.org.uk> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Mar 19, 2015 at 11:02:35PM +0100, Rafael J. Wysocki wrote: > On Thursday, March 19, 2015 10:59:20 PM Rafael J. Wysocki wrote: > > On Thursday, March 12, 2015 06:30:21 PM Russell King - ARM Linux wrote: > > > This is a re-posting of the patch set which I posted almost 10 months > > > ago to support the Dove PMU, with a few additional changes. This set > > > is based upon 3.19. > > > > > > In this set are: > > > > > > * two patches which Rafael originally acked, but there was indecision > > > last time around how to handle them due to potential conflicts with > > > work that Ulf was doing. These patches have been updated to apply > > > cleanly to 3.19. I don't know if people want to take these as > > > fixes to the PM domain code or not (hence why I'm posting this > > > series during the merge window - if it weren't for this, I'd hold > > > it off.) > > > > > > * what I regard as a fix to the PM domain code; as a result of a > > > previous commit, the PM domain code mismatches the runtime PM state, > > > which leads to the PM domain being unexpectedly left on. This patch > > > has been re-worked to try an alternative approach, syncing the PM > > > domain state with the runtime PM state after the probe has completed. > > > > > > * the addition of the core Dove PMU driver, which consists of a reset, > > > IRQ controller, and power domains. The reset and power domain code > > > has to be closely related due to the power up/down requirements of > > > the GPU/VPU subsystems needing to be performed atomically. (This > > > requirement prevents it using the MFD infrastructure, because we > > > would need to hold spinlocks while calling several different > > > sub-drivers.) > > > > > > * addition of the RTC interrupt, so we can now receive and act on > > > alarms generated by the Dove RTC. > > > > > > * addition of the DT descriptions for the GPU and VPU power domains. > > > These patches do not themselves add the DT descriptions for these > > > units, so these patches serve as illustrations how these should be > > > described. > > > > > > > I can apply patches [1-3/9] from this series if that helps. > > I mean from the next version of it ([FOR DISCUSSION 0/10] Dove PMU support). Kevin had some comments on patch 2 which I think ought to be addressed. He's quite right that dev_to_genpd() should be hidden from view and replaced with the safer pm_genpd_lookup_dev() version - even though pm_genpd_lookup_dev() itself is not fully safe against the domain on a device changing beneath it. If you want to hold on, I'll respin with his comments there addressed, or if you prefer to merge them as-is and fix Kevin's comments afterwards, that's also fine by me. I'd just be happy to see some progress on this series. :) Thanks. -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net.