From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2626C43217 for ; Mon, 28 Nov 2022 09:51:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229586AbiK1Jvh (ORCPT ); Mon, 28 Nov 2022 04:51:37 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35506 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229822AbiK1Jvc (ORCPT ); Mon, 28 Nov 2022 04:51:32 -0500 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 59A76193D1; Mon, 28 Nov 2022 01:51:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1669629091; x=1701165091; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=kg8gSoBWfRNuX9Ds7mhHZmSGNm7a8mdcL2RtLflzVgQ=; b=LwcARyLHfEBhkQFez7FyQI258sbMq/G2c3CUv+CoXqc0VdafUbQZK2X9 bEdy+KpBftl1f6zCpz86NIpkXxcMv3S+16toehKw0e09ZF5iJoPDmifQP CnAs4I/BVuxSB1flKMh36C/hMzE9yLJRxqv+WEU4IKecs9Hxw+8/kIyRw pzbgCiWte9vOgBFtjgApb0i3nxQzEQTJGHjSwbs79gkJ/bdEHUCD+nEVP w78zieRLFdzdzCkOif//k4AzkS9/NYSb/jnozbiR6ds9K+nTrArMALXeh fYSx6OjC/FP2q91Zejz0u7+PjDzJJDZKlVgKANXryPaXZrJBjuf0JOAI1 w==; X-IronPort-AV: E=McAfee;i="6500,9779,10544"; a="379048950" X-IronPort-AV: E=Sophos;i="5.96,200,1665471600"; d="scan'208";a="379048950" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Nov 2022 01:51:14 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10544"; a="620988277" X-IronPort-AV: E=Sophos;i="5.96,200,1665471600"; d="scan'208";a="620988277" Received: from smile.fi.intel.com ([10.237.72.54]) by orsmga006.jf.intel.com with ESMTP; 28 Nov 2022 01:51:11 -0800 Received: from andy by smile.fi.intel.com with local (Exim 4.96) (envelope-from ) id 1ozan8-0013vS-0p; Mon, 28 Nov 2022 11:51:10 +0200 Date: Mon, 28 Nov 2022 11:51:09 +0200 From: Andy Shevchenko To: William Breathitt Gray Cc: Michael Walle , linus.walleij@linaro.org, brgl@bgdev.pl, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, broonie@kernel.org Subject: Re: [PATCH v3 3/9] gpio: 104-dio-48e: Migrate to the regmap-irq API Message-ID: References: <80fc819bcafe9697b6e02c0750d3cf0ea4ec9e1b.1669100542.git.william.gray@linaro.org> <3a23df35a35cdba19eeb10c75b5bca97@walle.cc> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On Sun, Nov 27, 2022 at 05:00:40PM -0500, William Breathitt Gray wrote: > On Sun, Nov 27, 2022 at 07:31:48PM +0100, Michael Walle wrote: > > Am 2022-11-22 11:29, schrieb William Breathitt Gray: ... > > gpiochip doesn't seem to be the correct place, gpiochip_add_irqchip() > > is a noop for gpio-regmap, right? So using gpiochip_irqchip_init_hw() > > seems wrong. > > > > Maybe make gpio-regmap call it on its own? But really we just connect > > the regmap-irq to the gpiochip irqdomain. > > I think you're right, it feels strange to handle IRQ initialization via > the GPIO framework. Maybe somewhere in regmap_irq might be more > appropriate? The problem that that callback solves is possible interrupt storm, spurious interrupts, and Use Before Initialized. If you can guarantee that in your case it never happens, add a comment and go on. (It might be useful to tweak code a bit and try CONFIG_DEBUG_SHIRQ=y) > > What is the purpose of the > > .init_hw callback? I've looked at other drivers which use regmap-irq > > and they all seem to just initialize the hardware in their _probe(). > > > > -michael > > I'm not opposed to initializing the hardware in _probe(), although I can > see merit in pushing that operation instead closer to the framework > where the initialization is actually relevant. > > Andy, maybe you can shed some light about .init_hw; I think you > introduced it to gpiolib in commit 9411e3aaa6342. It seems that commit message doesn't fully explain the situation behind that change. But it was observed in real life, see above. -- With Best Regards, Andy Shevchenko