From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] TWL4030-RTC: Remove obsolete comment Date: Wed, 20 Aug 2008 15:29:51 +0300 Message-ID: <20080820122950.GJ29374@atomide.com> References: <20080819144218.GE23346@gandalf.research.nokia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-bos.mailhop.org ([63.208.196.178]:63216 "EHLO mho-01-bos.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753850AbYHTMaI (ORCPT ); Wed, 20 Aug 2008 08:30:08 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Pakaravoor, Jagadeesh" Cc: "felipe.balbi@nokia.com" , "linux-omap@vger.kernel.org" * Pakaravoor, Jagadeesh [080819 18:09]: > > -----Original Message----- > > From: Felipe Balbi [mailto:felipe.balbi@nokia.com] > > Sent: Tuesday, August 19, 2008 8:12 PM > > To: Pakaravoor, Jagadeesh > > Cc: linux-omap@vger.kernel.org > > Subject: Re: [PATCH] TWL4030-RTC: Remove obsolete comment > > > > On Tue, Aug 19, 2008 at 07:50:26PM +0530, ext Pakaravoor, Jagadeesh wrote: > > > From: Jagadeesh Bhaskar Pakaravoor > > > > > > TWL4030-RTC: Remove obsolete comment. > > > > > > Signed-off-by: Jagadeesh Bhaskar Pakaravoor > > > Index: my-local-git-dir/drivers/rtc/rtc-twl4030.c > > > =================================================================== > > > --- my-local-git-dir.orig/drivers/rtc/rtc-twl4030.c 2008-08-19 19:20:24.000000000 +0530 > > > +++ my-local-git-dir/drivers/rtc/rtc-twl4030.c 2008-08-19 19:28:43.037746119 +0530 > > > @@ -446,12 +446,6 @@ static irqreturn_t twl4030_rtc_interrupt > > > REG_RTC_STATUS_REG); > > > if (res) > > > goto out; > > > - /* > > > - * Workaround for strange behaviour with T2. Need to write into ISR > > > - * register one more time to clear the interrupt. Otherwise, the same > > > - * RTC event generates 2 interrupts in a row. > > > - * (no errata document available) > > > - */ > > > res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, > > > PWR_RTC_INT_CLR, REG_PWR_ISR1); > > > if (res) > > > > Why is this obsolete? Seems quite a good comment to me. Removing it > > might turn into someone removing that line and make t2's rtc misbehave. > > > > Maybe you wanna remove only the "no errata" line if there is any errata > > available but the comment itself is quite valuable. > > > > Currently there is only one write to the ISR (REG_PWR_ISR1), where as previously there used to be two writes. So the comment is no more valid. Sounds like a valid patch then. Pushing today. Tony > > -- > Jagadeesh > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html