From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932083AbbFXPi2 (ORCPT ); Wed, 24 Jun 2015 11:38:28 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:56570 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752338AbbFXPiT (ORCPT ); Wed, 24 Jun 2015 11:38:19 -0400 Message-ID: <558ACEB8.2020406@ti.com> Date: Wed, 24 Jun 2015 18:37:28 +0300 From: Grygorii Strashko User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: , Nishanth Menon CC: Alexandre Belloni , Alessandro Zummo , , , , Subject: Re: [PATCH V2 0/5] drivers/rtc/rtc-ds1307.c: Basic cleanups and support for wakeupirq References: <1435076114-12416-1-git-send-email-nm@ti.com> <20150624143432.GE21232@saruman.tx.rr.com> In-Reply-To: <20150624143432.GE21232@saruman.tx.rr.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/24/2015 05:34 PM, Felipe Balbi wrote: > Hi, > > On Tue, Jun 23, 2015 at 11:15:09AM -0500, Nishanth Menon wrote: >> Hi, >> >> Here is the revamp of the patch posted eariler to support wakeupirq. >> This series does a few cleanups as well prior to introducing the >> generic apis required to support the wakeirq mechanism used on certain >> TI SoCs. Cleanups done here are just minimal set of the cleanups we >> discussed in [2]. >> >> Series is based on next-20150623 >> >> Test log: http://pastebin.ubuntu.com/11763002/ >> >> NOTE1: I have only tested in mcp79410 device, and not on other >> ds1307-rtc driver compatbile devices as I dont have any other. >> additional testing might be nice to have. >> NOTE2: there are additional fixes that linux-next seems to need but >> those are in omap-rtc, cpufreq etc, but those are not related by the >> ds-1307. >> >> V1: https://patchwork.kernel.org/patch/4759171/ >> >> NOTE: Patch 5/5 of the series is meant to indicate the usage, and will >> be reposted along with other wakeup sources to be enabled once the >> driver changes are accepted. >> >> Felipe Balbi (1): >> drivers/rtc/rtc-ds1307.c: Convert to threaded IRQ >> >> Nishanth Menon (4): >> drivers/rtc/rtc-ds1307.c: Switch to managed irq allocation >> drivers/rtc/rtc-ds1307.c: Sort the headers >> drivers/rtc/rtc-ds1307.c: Support optional wakeup interrupt source > > for the series, except the patch which I wrote : > > Acked-by: Felipe Balbi > With minor comment to patch 4: Reviewed-by: Grygorii Strashko -- regards, -grygorii