From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Mon, 10 Oct 2011 15:30:43 -0700 Subject: [PATCH v6 05/16] OMAP2+: UART: Cleanup part of clock gating mechanism for uart In-Reply-To: <1317380495-584-5-git-send-email-govindraj.raja@ti.com> (Govindraj R.'s message of "Fri, 30 Sep 2011 16:31:31 +0530") References: <1317380495-584-1-git-send-email-govindraj.raja@ti.com> <1317380495-584-5-git-send-email-govindraj.raja@ti.com> Message-ID: <87ty7gfpik.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org "Govindraj.R" writes: > Currently we use a shared irq handler to identify uart activity and then > trigger a timer. OK. > Based the timeout value set from sysfs the timer used to expire. Please re-phrase as I'm not sure what is being said here. > If no uart-activity was detected timer-expiry func sets can_sleep > flag. Based on this we will disable the uart clocks in idle path. > > Since the clock gating mechanism is outside the uart driver, we currently > use this mechanism. In preparation to runtime implementation for omap-serial > driver we can cleanup this mechanism and use runtime API's to gate uart clocks. > > Also remove timer related info from local uart_state struct and remove > the code used to set timeout value from sysfs. Remove un-used function > omap_uart_check_wakeup. > > Signed-off-by: Govindraj.R The patch itself fine, and is a well-neede cleanup, but changelog (especially the first part) does not read well. Kevin