From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Pieralisi Subject: Re: [PATCH v6 6/7] drivers: cpuidle: initialize big.LITTLE driver through DT Date: Wed, 6 Aug 2014 10:02:20 +0100 Message-ID: <20140806090220.GA24431@e102568-lin.cambridge.arm.com> References: <1405958786-17243-1-git-send-email-lorenzo.pieralisi@arm.com> <1405958786-17243-7-git-send-email-lorenzo.pieralisi@arm.com> <20140723162626.GE20966@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140723162626.GE20966@arm.com> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Catalin Marinas Cc: Mark Rutland , Tomasz Figa , Chander Kashyap , Vincent Guittot , Nicolas Pitre , Daniel Lezcano , "linux-arm-kernel@lists.infradead.org" , "grant.likely@linaro.org" , Charles Garcia-Tobin , "devicetree@vger.kernel.org" , Kevin Hilman , "linux-pm@vger.kernel.org" , Sebastian Capella , Mark Brown , Antti Miettinen , Paul Walmsley , Geoff Levand , Peter De Schrijver , Stephen Boyd , Amit Kucheria , Bartlomi List-Id: devicetree@vger.kernel.org On Wed, Jul 23, 2014 at 05:26:26PM +0100, Catalin Marinas wrote: > On Mon, Jul 21, 2014 at 05:06:25PM +0100, Lorenzo Pieralisi wrote: > > With the introduction of DT based idle states, CPUidle drivers for ARM > > can now initialize idle states data through properties in the device tree. > > > > This patch adds code to the big.LITTLE CPUidle driver to dynamically > > initialize idle states data through the updated device tree source file. > > > > Cc: Chander Kashyap > > Signed-off-by: Lorenzo Pieralisi > > I think for this patch (and the exynos one) we should still allow the > driver to work fine with old DT. Basically if the information is not in > DT, fall back to the original static definitions. Yes, this makes sense, I will implement the fall-back mechanism on v7, that should be final. If in the meanwhile Samsung guys have time to test the Exynos patch that would be very appreciated. Thanks, Lorenzo From mboxrd@z Thu Jan 1 00:00:00 1970 From: lorenzo.pieralisi@arm.com (Lorenzo Pieralisi) Date: Wed, 6 Aug 2014 10:02:20 +0100 Subject: [PATCH v6 6/7] drivers: cpuidle: initialize big.LITTLE driver through DT In-Reply-To: <20140723162626.GE20966@arm.com> References: <1405958786-17243-1-git-send-email-lorenzo.pieralisi@arm.com> <1405958786-17243-7-git-send-email-lorenzo.pieralisi@arm.com> <20140723162626.GE20966@arm.com> Message-ID: <20140806090220.GA24431@e102568-lin.cambridge.arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Jul 23, 2014 at 05:26:26PM +0100, Catalin Marinas wrote: > On Mon, Jul 21, 2014 at 05:06:25PM +0100, Lorenzo Pieralisi wrote: > > With the introduction of DT based idle states, CPUidle drivers for ARM > > can now initialize idle states data through properties in the device tree. > > > > This patch adds code to the big.LITTLE CPUidle driver to dynamically > > initialize idle states data through the updated device tree source file. > > > > Cc: Chander Kashyap > > Signed-off-by: Lorenzo Pieralisi > > I think for this patch (and the exynos one) we should still allow the > driver to work fine with old DT. Basically if the information is not in > DT, fall back to the original static definitions. Yes, this makes sense, I will implement the fall-back mechanism on v7, that should be final. If in the meanwhile Samsung guys have time to test the Exynos patch that would be very appreciated. Thanks, Lorenzo