From mboxrd@z Thu Jan  1 00:00:00 1970
From: dinguyen@altera.com (Dinh Nguyen)
Date: Tue, 17 Sep 2013 17:12:47 -0500
Subject: [RESEND PATCHv3 1/2] clocksource: dw_apb_timer: Move timer
 defines to header file.
In-Reply-To: <alpine.DEB.2.02.1309172337570.4089@ionos.tec.linutronix.de>
References: <1377744796-22175-1-git-send-email-dinguyen@altera.com>
 <1379434447.24347.0.camel@linux-builds1>
 <alpine.DEB.2.02.1309172337570.4089@ionos.tec.linutronix.de>
Message-ID: <1379455967.14195.3.camel@linux-builds1>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Tue, 2013-09-17 at 23:45 +0200, Thomas Gleixner wrote:
> On Tue, 17 Sep 2013, Dinh Nguyen wrote:
> 
> > On Wed, 2013-08-28 at 21:53 -0500, Dinh Nguyen wrote:
> > > From: Dinh Nguyen <dinguyen@altera.com>
> > > 
> > > Move all dw_apb_timer defines to include/linux/dw_apb_timer.h.
> 
> That's describing WHAT the patch does not WHY. I can see the WHAT part
> without that pointless changelog.

Will fix..
> 
> > 
> > Ping? This patch series has sat idle for 2-weeks now.
> 
> So what?
> 
> Sending non urgent (i.e. bug fixes, regression fixes etc.) patches at
> the beginning of the merge window begs for a 2 weeks delay. Well
> documented procedure...

Sorry, but it was just a friendly ping...didn't mean to imply any
urgency. The original patch was sent on 8/21, so patch has been around
for ~4 weeks..

Thanks for the review.

Dinh
> 
> 
> Thanks,
> 
> 	tglx
>