Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751752Ab3GHJli (ORCPT ); Mon, 8 Jul 2013 05:41:38 -0400 Received: from kdh-gw.itdev.co.uk ([89.21.227.133]:16032 "EHLO hermes.kdh.itdev.co.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751045Ab3GHJlg (ORCPT ); Mon, 8 Jul 2013 05:41:36 -0400 Message-ID: <51DA894D.3050208@itdev.co.uk> Date: Mon, 08 Jul 2013 10:41:33 +0100 From: Nick Dyer User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Dmitry Torokhov CC: Daniel Kurtz , Henrik Rydberg , Joonyoung Shim , Alan Bowens , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Peter Meerwald , Benson Leung , Olof Johansson Subject: Re: [PATCH 26/51] Input: atmel_mxt_ts - Move input device init into separate function References: <1372337366-9286-1-git-send-email-nick.dyer@itdev.co.uk> <1372337366-9286-27-git-send-email-nick.dyer@itdev.co.uk> <20130707053407.GC15453@core.coreip.homeip.net> In-Reply-To: <20130707053407.GC15453@core.coreip.homeip.net> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 988 Lines: 20 Dmitry Torokhov wrote: > On Thu, Jun 27, 2013 at 01:49:01PM +0100, Nick Dyer wrote: >> Signed-off-by: Nick Dyer > > So before we allocated input device before requesting IRQ, now we fo it > afterwards so there is moment where the interrupt is requested and not > disabled and input device is not allocated yet. Is it possible for > interrupt to happen at that moment? Yes, and it will be handled correctly, there are guards in the correct places to ensure the input device will not be used before being registered. It is registered at this point since there are several paths that might need the interrupt handler (for example, to handle flash if device is in failed state, or to upload configuration if necessary). -- 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/