Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965702AbbDWAF3 (ORCPT ); Wed, 22 Apr 2015 20:05:29 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:57911 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932963AbbDWAF2 (ORCPT ); Wed, 22 Apr 2015 20:05:28 -0400 Message-ID: <55383724.2080806@ti.com> Date: Wed, 22 Apr 2015 19:04:52 -0500 From: Nishanth Menon User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Alexandre Belloni CC: Alessandro Zummo , , , Subject: Re: [PATCH V2] drivers/rtc/rtc-ds1307.c: Enable the mcp794xx alarm after programming time References: <1429577494-15087-1-git-send-email-nm@ti.com> <20150421234139.GF8539@piout.net> <5536E433.90103@ti.com> <20150422010925.GH8539@piout.net> <55370073.3060809@ti.com> <20150422113053.GI8539@piout.net> In-Reply-To: <20150422113053.GI8539@piout.net> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1713 Lines: 45 On 04/22/2015 06:30 AM, Alexandre Belloni wrote: Apologies on a tardy response, got dragged into another issue and got cooped up in lab whole day. > On 21/04/2015 at 20:59:15 -0500, Nishanth Menon wrote : >>>> Why is that so? when set alarm is requested for time X, you want >>>> interrupt at time X, not an interrupt for previous configured RTC >>>> alarm time! >>>> >>> >>> You expect at least an interrupt. >> >> And you will get an interrupt if the event occurs before the i2c burst >> starts. Once the i2cburst does start, you are committing to the new time. >> > > You mean that even if ALM0EN is set after ALM0IF was set to 1, it will > trigger the interrupt? I had a look at the MFP output block diagram > would let me think that this is the case. I was thinking otherwise > before. If that is so, then indeed, your patch is OK. At least based on my testing it seems to be the case, and as you can see in the block diagram, the ALM0EN mux comes after ALM0IF.. agreed that I am not sure the mux to have some internal buffers/latches etc.. dont have access to rtl to make more comments. > > My concern was about the time between ds1307->write_block_data() and > i2c_smbus_write_byte_data() which actually calls cond_sched(). > > I fully agree that your patch doesn't change the behaviour for the other > cases you presented and further clean up is to be done in a separate set > of patches. > Can I take this as an Acked-by? -- Regards, Nishanth Menon -- 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/