From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] omap3: Add cpuidle parameters table for omap3430 Date: Tue, 16 Feb 2016 08:08:59 -0800 Message-ID: <20160216160859.GE19432@atomide.com> References: <20160215152542.GS19432@atomide.com> <1455559328-19783-1-git-send-email-pali.rohar@gmail.com> <56C2F387.3040803@linaro.org> <20160216100752.GF1476@pali> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <20160216100752.GF1476@pali> Sender: linux-kernel-owner@vger.kernel.org To: Pali =?utf-8?B?Um9ow6Fy?= Cc: Daniel Lezcano , Nishanth Menon , Pavel Machel , Aaro Koskinen , Sebastian Reichel , Ivaylo Dimitrov , Lorenzo Pieralisi , linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org List-Id: linux-omap@vger.kernel.org * Pali Roh=C3=A1r [160216 02:09]: > On Tuesday 16 February 2016 11:01:43 Daniel Lezcano wrote: > > On 02/15/2016 07:02 PM, Pali Roh=C3=A1r wrote: > > >Based on CPU type choose generic omap3 or omap3430 specific cpuidl= e > > >parameters. Parameters for omap3430 were measured on Nokia N900 de= vice and > > >added by commit 5a1b1d3a9efa ("OMAP3: RX-51: Pass cpu idle paramet= ers") > > >which were later removed by commit 231900afba52 ("ARM: OMAP3: cpui= dle - > > >remove rx51 cpuidle parameters table") due to huge code complexity= =2E > > > > > >This patch brings cpuidle parameters for omap3430 devices again, b= ut uses > > >simple condition based on CPU type. > > > > > >Signed-off-by: Pali Roh=C3=A1r > > >Fixes: 231900afba52d6faddfb480cde4132d4edc089bc > > >--- > >=20 > > Acked-by: Daniel Lezcano >=20 > Tony, Nishanth (or somebody else...) can you test this patch on more > omap3 devices to check if it does not break existing cpuidle code? Seems to work for me on a ldp, which is GP 3430 ES3.0. Regards, Tony