From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932658Ab2EJWf6 (ORCPT ); Thu, 10 May 2012 18:35:58 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:43719 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761678Ab2EJWf4 (ORCPT ); Thu, 10 May 2012 18:35:56 -0400 From: Grant Likely Subject: Re: [PATCH] gpio/langwell: re-read the IRQ status register after each iteration To: Linus Walleij , Mika Westerberg Cc: linux-kernel@vger.kernel.org, linus.walleij@stericsson.com, Julia Lawall In-Reply-To: References: <1336644082-14137-1-git-send-email-mika.westerberg@linux.intel.com> Date: Thu, 10 May 2012 16:35:54 -0600 Message-Id: <20120510223554.183F83E04A6@localhost> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 10 May 2012 14:16:56 +0200, Linus Walleij wrote: > On Thu, May 10, 2012 at 12:01 PM, Mika Westerberg > wrote: > > > Spotted by Grant Likely. The IRQ status register should be re-read after > > each iteration. Otherwise the loop misses the interrupt if it gets raised > > immediately after handled. > > > > Signed-off-by: Mika Westerberg > > --- > > This patch applies on top of my previous patch ("gpio: langwell: convert to > > use irq_domain"). > > Oh yeah that thing I recognize! > Acked-by: Linus Walleij Applied, thanks. g.