From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH 3/5] ARM: dove: create a proper PMU driver for power domains, PMU IRQs and resets Date: Mon, 28 Apr 2014 13:17:26 +0100 Message-ID: <20140428121726.GK26756@n2100.arm.linux.org.uk> References: <20140427132312.GC26756@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-pm-owner@vger.kernel.org To: Ulf Hansson Cc: "linux-arm-kernel@lists.infradead.org" , Sebastian Hesselbarth , "devicetree@vger.kernel.org" , "linux-pm@vger.kernel.org" , Mark Rutland , Pawel Moll , "Rafael J. Wysocki" , Rob Herring , Tomasz Figa List-Id: devicetree@vger.kernel.org On Mon, Apr 28, 2014 at 01:55:40PM +0200, Ulf Hansson wrote: > On 27 April 2014 15:29, Russell King wr= ote: > > The PMU device contains an interrupt controller, power control and > > resets. The interrupt controller is a little sub-standard in that > > there is no race free way to clear down pending interrupts, so we t= ry > > to avoid problems by reducing the window as much as possible, and > > clearing as infrequently as possible. > > > > The interrupt support is implemented using an IRQ domain, and the > > parent interrupt referenced in the standard DT way. > > > > The power domains and reset support is closely related - there is a > > defined sequence for powering down a domain which is tightly couple= d > > with asserting the reset. Hence, it makes sense to group these two > > together. > > > > This patch adds the core PMU driver: power domains must be defined = in > > the DT file in order to make use of them. The reset controller can > > be referenced in the standard way for reset controllers. >=20 > Hi Russell, >=20 > This patch would be simplified if this was based upon the not yet > merged patchset from Tomasz Figa, "[PATCH v3 0/3] Generic Device Tree > based power domain look-up". >=20 > For example you would likely not need to add some of the marvel > specific DT bindings, and you wouldn=E2=80=99t need the bus_notifiers= to add > devices to the power domain. I guess I just though it could be useful > input to consider while going forward, unless you already knew. Does that apply to 3.14? --=20 =46TTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... sl= owly improving, and getting towards what was expected from it.