Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755217AbdCTQAx (ORCPT ); Mon, 20 Mar 2017 12:00:53 -0400 Received: from muru.com ([72.249.23.125]:40880 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753348AbdCTQA2 (ORCPT ); Mon, 20 Mar 2017 12:00:28 -0400 Date: Mon, 20 Mar 2017 08:33:42 -0700 From: Tony Lindgren To: Charles Keepax Cc: Mark Brown , linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, Lee Jones , Marcel Partap , Michael Scott , Sebastian Reichel Subject: Re: [PATCH 1/4] regmap: irq: Fix lost interrupts by introducing handle_reread Message-ID: <20170320153341.GK20572@atomide.com> References: <20170317003633.7361-1-tony@atomide.com> <20170317003633.7361-2-tony@atomide.com> <20170320151413.GT6986@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170320151413.GT6986@localhost.localdomain> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1415 Lines: 31 * Charles Keepax [170320 08:15]: > On Thu, Mar 16, 2017 at 05:36:30PM -0700, Tony Lindgren wrote: > > At least Motorola CPCAP PMIC needs it's device interrupts re-read > > until there are no more interrupts. Otherwise the PMIC interrupt to > > the SoC will eventually stop toggling. > > > > Let's allow doing that by introducing a flag for handle_reread > > and by splitting regmap_irq_thread() into two separate functions > > for regmap_read_irq_status() and regmap_irq_handle_pending(). > > > > Is this actually a property of this hardware or is this just a > result of connecting a device that generates level IRQs to a host > that is expecting an edge triggered IRQ? Well the CPCAP PMIC interrupt is connected to a GPIO edge interrupt on the SoC in the Motorola v3.8 based kernel tree. But what the CPCAP PMIC interrupt handler does in the Motorola kernel is keep handling the CPCAP internal interrupts (and also keep reading the SoC GPIO line status!) until the GPIO line changes status. So yeah it's a property of the CPCAP PMIC hardware. Changing the GPIO interrupt to level makes no difference here. It's not clear why they had marked the CPCAP PMIC to SoC GPIO interrupt as edge though in the Motorola kernel. My guess is that some PMIC to SoC wake-up events are edge interrupts. So we have it set up as edge interrupt in the mainline kernel too. Regards, Tony