From patchwork Mon Apr 22 13:43:56 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Poddar, Sourav" X-Patchwork-Id: 2471941 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 760C3DF23A for ; Mon, 22 Apr 2013 13:45:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753897Ab3DVNoQ (ORCPT ); Mon, 22 Apr 2013 09:44:16 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:55199 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753782Ab3DVNoP (ORCPT ); Mon, 22 Apr 2013 09:44:15 -0400 Received: from dbdlxv05.itg.ti.com ([172.24.171.60]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id r3MDiB5S026857; Mon, 22 Apr 2013 08:44:12 -0500 Received: from DBDE70.ent.ti.com (dbde70.ent.ti.com [172.24.170.148]) by dbdlxv05.itg.ti.com (8.14.3/8.13.8) with ESMTP id r3MDi3rI031200; Mon, 22 Apr 2013 08:44:09 -0500 Received: from dbdp33.itg.ti.com (172.24.170.252) by dbde70.ent.ti.com (172.24.170.148) with Microsoft SMTP Server id 14.1.323.3; Mon, 22 Apr 2013 19:14:06 +0530 Received: from a0131647.apr.dhcp.ti.com (smtpvbd.itg.ti.com [172.24.170.250]) by dbdp33.itg.ti.com (8.13.8/8.13.8) with ESMTP id r3MDhxTH004666; Mon, 22 Apr 2013 19:14:06 +0530 From: Sourav Poddar To: , , , CC: , , , Sourav Poddar , Benoit Cousson , Santosh Shilimkar , Felipe Balbi , Rajendra nayak Subject: [PATCHv2 4/5] arm: dts: am33xx: Remove "ti, no_idle_on_suspend" property. Date: Mon, 22 Apr 2013 19:13:56 +0530 Message-ID: <1366638237-6880-5-git-send-email-sourav.poddar@ti.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1366638237-6880-1-git-send-email-sourav.poddar@ti.com> References: <1366638237-6880-1-git-send-email-sourav.poddar@ti.com> MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org The "ti,no_idle_on_suspend" property was required to keep ocmcram clocks running during idle. But the commit below[1], added in v3.6 should prevent the any automaic clock gating for devices without drivers bound. Since there is no driver for the OCM RAM block, we are not affected by the automatic idle on suspend anyways. [1]: commit 72bb6f9b51c82c820ddef892455a85b115460904 Author: Kevin Hilman Date: Tue Jul 10 15:29:04 2012 -0700 ARM: OMAP: omap_device: don't attempt late suspend if no driver bound Currently, the omap_device PM domain layer uses the late suspend and early resume callbacks to ensure devices are in their low power states. However, this is attempted even in cases where a driver probe has failed. If a driver's ->probe() method fails, the driver is likely in a state where it is not expecting its runtime PM callbacks to be called, yet currently the omap_device PM domain code attempts to call the drivers callbacks. To fix, use the omap_device driver_status field to check whether a driver is bound to the omap_device before attempting to trigger driver callbacks. Cc: Benoit Cousson Cc: Santosh Shilimkar Cc: Felipe Balbi Cc: Rajendra nayak Signed-off-by: Sourav Poddar Reviewed-by: Felipe Balbi --- arch/arm/boot/dts/am33xx.dtsi | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index 74a8125..49a050e 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -394,7 +394,6 @@ compatible = "ti,am3352-ocmcram"; reg = <0x40300000 0x10000>; ti,hwmods = "ocmcram"; - ti,no_idle_on_suspend; }; wkup_m3: wkup_m3@44d00000 {