Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755016Ab3DXNQU (ORCPT ); Wed, 24 Apr 2013 09:16:20 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:60317 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754376Ab3DXNQS (ORCPT ); Wed, 24 Apr 2013 09:16:18 -0400 From: Sourav Poddar To: , , , CC: , , , Sourav Poddar , Santosh Shilimkar , Felipe Balbi , Rajendra nayak Subject: [PATCHv3 0/5] Serial Omap fixes and cleanups Date: Wed, 24 Apr 2013 18:45:48 +0530 Message-ID: <1366809353-19435-1-git-send-email-sourav.poddar@ti.com> X-Mailer: git-send-email 1.7.1 MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2968 Lines: 76 Hi, This patch series contains fixes and cleanups around the issue that the console UART should not idled on suspend while using "no_console_suspend" in bootargs. The approach thought of is to modify the serial core/serial driver to bypass runtime PM if the UART in contention is a console and we are using "no_console_suspend" in our bootargs. While fixing the above issue, there are other cleanups also done as part of this series which are no longer required. This cleanups mainly include getting rid of using "omap_device_disable_idle_on_suspend" api for both dt and non dt case as the serial driver will be self sufficient to handle the "no_idle_on_suspend" issue. Serial was the only one making use of "omap_device_disable_idle_on_suspend" There were discussions about how to handle "no_idle_on_suspend" issue and all the discussions are as follows: https://lkml.org/lkml/2013/4/5/239 https://lkml.org/lkml/2013/4/2/350 https://lkml.org/lkml/2013/3/18/199 https://lkml.org/lkml/2013/3/18/295 Due to the amount of change in approach and other cleanups coming around it, I am posting this as a new series. Test info Omap4430sdp: - Tested wakeup from UART after suspend for dt and non dt case. Omap5430evm: - Tested wakeup from UART after suspend for dt case. This patches are based on 3.9-rc3 custom tree which has Santosh Shilimkar serial patch[1] [1]: http://permalink.gmane.org/gmane.linux.ports.arm.omap/95828 v2->v3 1. Use "-EBUSY" for no_console_suspend case 2. Bypass runtime PM only during suspend 3. Improve the commit log based on community suggestion. v1->v2 1. Remove the prepare/complete callback. 2. Adapt runtime PM callback to deal with the issue. 3. Fold patch(1,2) of previous series into 1. 4. Reordered the patch. 5. Change $subject and chage log for few patches. Cc: Santosh Shilimkar Cc: Felipe Balbi Cc: Rajendra nayak Sourav Poddar (5): driver: tty: serial: Move "uart_console" def to core header file. driver: serial: omap: prevent runtime PM for "no_console_suspend" case arm: omap2+: serial: remove no_console_suspend support arm: dts: am33xx: Remove "ti,no_idle_on_suspend" property. arm: omap2+: device: remove "OMAP_DEVICE_NO_IDLE_ON_SUSPEND" check arch/arm/boot/dts/am33xx.dtsi | 1 - arch/arm/mach-omap2/omap_device.c | 12 +++--------- arch/arm/mach-omap2/omap_device.h | 10 ---------- arch/arm/mach-omap2/serial.c | 7 ------- drivers/tty/serial/mpc52xx_uart.c | 10 ---------- drivers/tty/serial/omap-serial.c | 5 ++++- drivers/tty/serial/serial_core.c | 6 ------ include/linux/serial_core.h | 7 +++++++ 8 files changed, 14 insertions(+), 44 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/