From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net [217.70.183.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 193E23A1B6; Mon, 19 Feb 2024 15:54:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.194 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708358101; cv=none; b=jMqsQ01YGHRFljX9N9cZUIPL92ENbm0u1ZtbJlknSSnx7snhi2WPxXsuukAEHXPB9iR1m6uPI8jJb00D+jtRsId+Sik0r6+Hx4ShWAqbzn67Njc7WTIVwViy1WSOVGZISvESmEWW4uuNjTAnpb4oDjEzbMYhwLmapNDWWO4l9IA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708358101; c=relaxed/simple; bh=nC6aNc13U8MY/Slx3JehF2zXFuomJ7JF1NoXY9rqw9E=; h=Mime-Version:Content-Type:Date:Message-Id:Subject:Cc:To:From: References:In-Reply-To; b=lKA6x/h1rRskyE74xkGMVwvUZisFSlmYKbA/j5i91q1yLKaoKxWsHPqJocV5fBvNxNUWXRuzRCSkDALc16Wk/E7RKhdPk7Joilwl4GmDEFHALs3nuSrsZWyxbC9ZrVeNBQkcZ9VB7UP33vx8Smbp4782Y1oZ+dE4rOOAP4enO3M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=c2Musslz; arc=none smtp.client-ip=217.70.183.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="c2Musslz" Received: by mail.gandi.net (Postfix) with ESMTPSA id 8374C40004; Mon, 19 Feb 2024 15:54:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1708358096; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=JuVnxXLKBjOZXpqRb8ZUGYryBu30CeayscqdEmLRicA=; b=c2MusslzRHELYgxEV8vD36ddn12pVPyko1ZZ/zs6hqIWrg1An262098RPUuU6Zt7BdqLCF 5Z8laMNgpFdVYKME13I4EgK13u6VQR+Xyu4Nb0YVurEsKAnL3IfCSZlHNQ+en3LcE/yiNM IHXUkIMNu4jNrbex14/R09P/kOCjcvluN5xxyQRmTjbsfRx5zULw9SNSDb1L6TzM1FlHii bM0Ae80hmG3isfFQanv7M7q53LuopxX9EpRjMoPffWJSmCEzg4XQcFOSU4IA4eSn8NRldK S9zWCUdapYB4zuP0vgAtcwWNImh9UFb+4omk1WeF5Z1hlGuAzXoD/ykvFwgOyg== Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Mon, 19 Feb 2024 16:54:55 +0100 Message-Id: Subject: Re: [PATCH 16/23] gpio: nomadik: support shared GPIO IRQs Cc: "Linus Walleij" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Philipp Zabel" , "Thomas Bogendoerfer" , , , , , , "Gregory CLEMENT" , "Vladimir Kondratiev" , "Thomas Petazzoni" , "Tawfik Bayouk" To: "Bartosz Golaszewski" From: =?utf-8?q?Th=C3=A9o_Lebrun?= X-Mailer: aerc 0.15.2 References: <20240214-mbly-gpio-v1-0-f88c0ccf372b@bootlin.com> <20240214-mbly-gpio-v1-16-f88c0ccf372b@bootlin.com> In-Reply-To: X-GND-Sasl: theo.lebrun@bootlin.com Hello, On Mon Feb 19, 2024 at 4:48 PM CET, Bartosz Golaszewski wrote: > On Wed, Feb 14, 2024 at 5:24=E2=80=AFPM Th=C3=A9o Lebrun wrote: > > > > Support a single IRQs used by multiple GPIO banks. Change the IRQ > > handler type from a chained handler (as used by gpiolib > > for ->parent_handler) to a threaded IRQ. > > > > Use a fake raw spinlock to ensure generic_handle_irq() is called in a > > no-irq context. See Documentation/driver-api/gpio/driver.rst, "CHAINED > > CASCADED GPIO IRQCHIPS" for additional information. > > > > Any reason for not using preempt_disable()? I did what the doc recommended: > The generic_handle_irq() is expected to be called with IRQ disabled, > so the IRQ core will complain if it is called from an IRQ handler which i= s > forced to a thread. The "fake?" raw lock can be used to work around this > problem:: > > raw_spinlock_t wa_lock; > static irqreturn_t omap_gpio_irq_handler(int irq, void *gpiobank) > unsigned long wa_lock_flags; > raw_spin_lock_irqsave(&bank->wa_lock, wa_lock_flags); > generic_handle_irq(irq_find_mapping(bank->chip.irq.domain, bit)); > raw_spin_unlock_irqrestore(&bank->wa_lock, wa_lock_flags); If you confirm I should be using preempt_disable() that's what I'll do in the next revision. I could even throw in a documentation patch if the advice is outdated. Thanks Bartosz, -- Th=C3=A9o Lebrun, Bootlin Embedded Linux and Kernel engineering https://bootlin.com 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1184CC48BF8 for ; Mon, 19 Feb 2024 15:55:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:References:From:To:Cc: Subject:Message-Id:Date:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=2sW9Lshxu693pZZ56Q7etIGCx9yarr6Hvel5SoHOf/g=; b=fmEIDO5+XGwTSb fF01OpFaTFWp5ydpuVMJVrX2fabHwm2tuiJKxG7Ih797qHBsTaMF1kJqFsy/RbzHQgQvpyCRLAdzd a/mPKOg4OryTYBHns4WlxBLTNwNdLNr6ejsTY91xu+Fr4H6q69OwbbX4YrvBUK62X7wL1ZD5nvAam IABV6Cdmh/x21QSxd1S4F8xQI3VojC/7dJdXGbl/WoC9Gw01YqUVFOprBZTSMRX1hFkiQQM29FJUQ dIgagOqtzgUuvBkdukXKQ8BaVNjdWW65uwNlLyNHoyZpWOCqCoYsVPbZXdKzCYyPMYuyscDgla5SU i+3l1bcZQsl57jhx4Vqw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rc5z4-0000000BDeB-0IjG; Mon, 19 Feb 2024 15:55:10 +0000 Received: from relay2-d.mail.gandi.net ([2001:4b98:dc4:8::222]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rc5yt-0000000BDYt-2RDj for linux-arm-kernel@lists.infradead.org; Mon, 19 Feb 2024 15:55:01 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id 8374C40004; Mon, 19 Feb 2024 15:54:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1708358096; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=JuVnxXLKBjOZXpqRb8ZUGYryBu30CeayscqdEmLRicA=; b=c2MusslzRHELYgxEV8vD36ddn12pVPyko1ZZ/zs6hqIWrg1An262098RPUuU6Zt7BdqLCF 5Z8laMNgpFdVYKME13I4EgK13u6VQR+Xyu4Nb0YVurEsKAnL3IfCSZlHNQ+en3LcE/yiNM IHXUkIMNu4jNrbex14/R09P/kOCjcvluN5xxyQRmTjbsfRx5zULw9SNSDb1L6TzM1FlHii bM0Ae80hmG3isfFQanv7M7q53LuopxX9EpRjMoPffWJSmCEzg4XQcFOSU4IA4eSn8NRldK S9zWCUdapYB4zuP0vgAtcwWNImh9UFb+4omk1WeF5Z1hlGuAzXoD/ykvFwgOyg== Mime-Version: 1.0 Date: Mon, 19 Feb 2024 16:54:55 +0100 Message-Id: Subject: Re: [PATCH 16/23] gpio: nomadik: support shared GPIO IRQs Cc: "Linus Walleij" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Philipp Zabel" , "Thomas Bogendoerfer" , , , , , , "Gregory CLEMENT" , "Vladimir Kondratiev" , "Thomas Petazzoni" , "Tawfik Bayouk" To: "Bartosz Golaszewski" From: =?utf-8?q?Th=C3=A9o_Lebrun?= X-Mailer: aerc 0.15.2 References: <20240214-mbly-gpio-v1-0-f88c0ccf372b@bootlin.com> <20240214-mbly-gpio-v1-16-f88c0ccf372b@bootlin.com> In-Reply-To: X-GND-Sasl: theo.lebrun@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240219_075459_954427_FB8C226D X-CRM114-Status: GOOD ( 17.14 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGVsbG8sCgpPbiBNb24gRmViIDE5LCAyMDI0IGF0IDQ6NDggUE0gQ0VULCBCYXJ0b3N6IEdvbGFz emV3c2tpIHdyb3RlOgo+IE9uIFdlZCwgRmViIDE0LCAyMDI0IGF0IDU6MjTigK9QTSBUaMOpbyBM ZWJydW4gPHRoZW8ubGVicnVuQGJvb3RsaW4uY29tPiB3cm90ZToKPiA+Cj4gPiBTdXBwb3J0IGEg c2luZ2xlIElSUXMgdXNlZCBieSBtdWx0aXBsZSBHUElPIGJhbmtzLiBDaGFuZ2UgdGhlIElSUQo+ ID4gaGFuZGxlciB0eXBlIGZyb20gYSBjaGFpbmVkIGhhbmRsZXIgKGFzIHVzZWQgYnkgZ3Bpb2xp Ygo+ID4gZm9yIC0+cGFyZW50X2hhbmRsZXIpIHRvIGEgdGhyZWFkZWQgSVJRLgo+ID4KPiA+IFVz ZSBhIGZha2UgcmF3IHNwaW5sb2NrIHRvIGVuc3VyZSBnZW5lcmljX2hhbmRsZV9pcnEoKSBpcyBj YWxsZWQgaW4gYQo+ID4gbm8taXJxIGNvbnRleHQuIFNlZSBEb2N1bWVudGF0aW9uL2RyaXZlci1h cGkvZ3Bpby9kcml2ZXIucnN0LCAiQ0hBSU5FRAo+ID4gQ0FTQ0FERUQgR1BJTyBJUlFDSElQUyIg Zm9yIGFkZGl0aW9uYWwgaW5mb3JtYXRpb24uCj4gPgo+Cj4gQW55IHJlYXNvbiBmb3Igbm90IHVz aW5nIHByZWVtcHRfZGlzYWJsZSgpPwoKSSBkaWQgd2hhdCB0aGUgZG9jIHJlY29tbWVuZGVkOgoK PiBUaGUgZ2VuZXJpY19oYW5kbGVfaXJxKCkgaXMgZXhwZWN0ZWQgdG8gYmUgY2FsbGVkIHdpdGgg SVJRIGRpc2FibGVkLAo+IHNvIHRoZSBJUlEgY29yZSB3aWxsIGNvbXBsYWluIGlmIGl0IGlzIGNh bGxlZCBmcm9tIGFuIElSUSBoYW5kbGVyIHdoaWNoIGlzCj4gZm9yY2VkIHRvIGEgdGhyZWFkLiBU aGUgImZha2U/IiByYXcgbG9jayBjYW4gYmUgdXNlZCB0byB3b3JrIGFyb3VuZCB0aGlzCj4gcHJv YmxlbTo6Cj4KPiAgIHJhd19zcGlubG9ja190IHdhX2xvY2s7Cj4gICBzdGF0aWMgaXJxcmV0dXJu X3Qgb21hcF9ncGlvX2lycV9oYW5kbGVyKGludCBpcnEsIHZvaWQgKmdwaW9iYW5rKQo+ICAgICAg IHVuc2lnbmVkIGxvbmcgd2FfbG9ja19mbGFnczsKPiAgICAgICByYXdfc3Bpbl9sb2NrX2lycXNh dmUoJmJhbmstPndhX2xvY2ssIHdhX2xvY2tfZmxhZ3MpOwo+ICAgICAgIGdlbmVyaWNfaGFuZGxl X2lycShpcnFfZmluZF9tYXBwaW5nKGJhbmstPmNoaXAuaXJxLmRvbWFpbiwgYml0KSk7Cj4gICAg ICAgcmF3X3NwaW5fdW5sb2NrX2lycXJlc3RvcmUoJmJhbmstPndhX2xvY2ssIHdhX2xvY2tfZmxh Z3MpOwoKSWYgeW91IGNvbmZpcm0gSSBzaG91bGQgYmUgdXNpbmcgcHJlZW1wdF9kaXNhYmxlKCkg dGhhdCdzIHdoYXQgSSdsbCBkbwppbiB0aGUgbmV4dCByZXZpc2lvbi4gSSBjb3VsZCBldmVuIHRo cm93IGluIGEgZG9jdW1lbnRhdGlvbiBwYXRjaCBpZgp0aGUgYWR2aWNlIGlzIG91dGRhdGVkLgoK VGhhbmtzIEJhcnRvc3osCgotLQpUaMOpbyBMZWJydW4sIEJvb3RsaW4KRW1iZWRkZWQgTGludXgg YW5kIEtlcm5lbCBlbmdpbmVlcmluZwpodHRwczovL2Jvb3RsaW4uY29tCgpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxp bmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3Rz LmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVsCg==