From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f52.google.com (mail-ej1-f52.google.com [209.85.218.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 78043176AB5 for ; Wed, 9 Oct 2024 08:11:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728461493; cv=none; b=RMsW75jl5m8vnOMXXoa2g1c3d/ha1DrQaRWNp1OnzYaFiTBf41o1My9BXKn2MTCMRhF6pZXp2H/h3qyoA7mih3PnijnsIijesOSj308mEQHzx/Yu5gHt0Ri0Bl3zWJYR0dAFwO7TmjZxNjIyBsI00s8bFvikh4RljpkaeuA5ltw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728461493; c=relaxed/simple; bh=uFkIJCmEHXFuGBYu8C7qbff+MFNDNAcB5CCG3QjJj8o=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=YMHbt8KJveO6xwCwTixND4+f9tDdR8pUvSTNIMJFHcBF9+9vQbUflKrZrw7uXXE3GmgVSHmFzmvPBUkufoDu0DX5vXmeORzsRmRwvdDLpf3cZwpNgQo4cuUBEkZOAAwO/4bB23R+mIyC+UqHkk3MhXqAoaaROgVWGWdZ/TmDGCo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=tuxon.dev; spf=pass smtp.mailfrom=tuxon.dev; dkim=pass (2048-bit key) header.d=tuxon.dev header.i=@tuxon.dev header.b=TVlYCyaR; arc=none smtp.client-ip=209.85.218.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=tuxon.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=tuxon.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=tuxon.dev header.i=@tuxon.dev header.b="TVlYCyaR" Received: by mail-ej1-f52.google.com with SMTP id a640c23a62f3a-a9950d27234so457769466b.1 for ; Wed, 09 Oct 2024 01:11:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tuxon.dev; s=google; t=1728461488; x=1729066288; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=tbRJKAa8Db7miijxCfbPNlNKjyHa9eXLPD3ZC1KS7p0=; b=TVlYCyaRjTexbcvIsyfCH8STycGt005yiiWtjCQ4Gz54FMxcIB/RoHG37U72RY4XML oTQzn7DSwRJPqjZwuELZ37pPpWgrwUQA6Zu3fVdTHeVEF7yYNSecqHg/Oc/G2zJAiNY2 Q6GMZCZzBk2dW53Sb2baDJtj0l0LWtht3m3P4zSKq0mUQu+o+CMsIDmfZ+KLRVYBeA42 XoJkzIKs/tCME37szeEP6Kpb7Qv9bcPcNaOQ1z/m7f5qFbit/JnDNl8qRB2+jBn341Jz L8pOiodTAsMZfgAPma9/ZfzITI0IkOrvwtG+7mDb6RLEbCNlLQEg83kJewOPDyATUhW/ 5/ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728461488; x=1729066288; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=tbRJKAa8Db7miijxCfbPNlNKjyHa9eXLPD3ZC1KS7p0=; b=ei6J+iojgdcBNyzcjX1N6vKke/9cVtsIu9eOGo5hpuPnENuj3209YyJbdITLY3e9ZG u0eIVsd0/vYzt1DMlzQxiXn6pPnCnsLTFPljAv+lzyhN8y93YeGo7mseGU6bVEz0DJPS LhNL7Hae5WjPaF0rbC376ifOBoyeAfDQdQY39d7gVkqntQLdltV9iGDljzvoUYY9wG3X JZNmKlMYzyzLPKSuMt1P+TNz+Y4HMfGsZVL2wpBvobSqAFrUHYc5eCk/xFHG9l4mJhQX b0h7mjGGquvMG639YzkhnZ5WU7WH/uz686o0uhyShJmyZpk3zb9UuygOq+9MyWTiD0Bs HbPA== X-Forwarded-Encrypted: i=1; AJvYcCV+XGXQAu9Nf2lQAHCssTkVfsQkCWOPpcSQOtr2lk/l42ByQ202oogUjMbkywvf/asgNoWhZeI1yJlRdD4=@vger.kernel.org X-Gm-Message-State: AOJu0YxtYW8oPUfwtNQvBdpdJcUQZ3yWaIJ/ZlS09SIGCY5K0NFCHP9V 9PpIlQnBVEVFgWwkT+kQPDxrM2WomRhCxo9CbUOm1NXpDMys1GVjnG0/A3OzIW8= X-Google-Smtp-Source: AGHT+IFi+kq5mbi2fyjEQUlOIPw0e4a1jwgaju6x57/KbakaFKYMNz/FU2pJttoym1jAaVJ9UMyz7Q== X-Received: by 2002:a17:907:60ce:b0:a99:50b8:9a10 with SMTP id a640c23a62f3a-a998d3301b8mr99613766b.64.1728461487610; Wed, 09 Oct 2024 01:11:27 -0700 (PDT) Received: from [192.168.50.4] ([82.78.167.23]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a994a8da5d3sm470893966b.134.2024.10.09.01.11.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 09 Oct 2024 01:11:27 -0700 (PDT) Message-ID: <7fe863c8-b883-45b0-b0e9-e376764d0bb7@tuxon.dev> Date: Wed, 9 Oct 2024 11:11:25 +0300 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] pinctrl: renesas: rzg2l: Always call rzg2l_gpio_request() for interrupt pins Content-Language: en-US To: Prabhakar , Linus Walleij , Geert Uytterhoeven Cc: linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Biju Das , Fabrizio Castro , Lad Prabhakar References: <20241003131642.472298-1-prabhakar.mahadev-lad.rj@bp.renesas.com> From: claudiu beznea In-Reply-To: <20241003131642.472298-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi, Prabhakar, On 03.10.2024 16:16, Prabhakar wrote: > From: Lad Prabhakar > > Ensure that rzg2l_gpio_request() is called for GPIO pins configured as > interrupts, regardless of whether they are muxed in u-boot. This > guarantees that the pinctrl core is aware of the GPIO pin usage via > pinctrl_gpio_request(), which is invoked through rzg2l_gpio_request(). > > Fixes: 2fd4fe19d0150 ("pinctrl: renesas: rzg2l: Configure interrupt input mode") > Signed-off-by: Lad Prabhakar > --- > Output before this patch on G2L/SMARC: > root@smarc-rzg2l:~# cat /sys/kernel/debug/pinctrl/11030000.pinctrl-pinctrl-rzg2l/pinmux-pins | grep P2_1 > pin 17 (P2_1): UNCLAIMED > > Output after this patch G2L/SMARC: > root@smarc-rzg2l:~# cat /sys/kernel/debug/pinctrl/11030000.pinctrl-pinctrl-rzg2l/pinmux-pins | grep P2_1 > pin 17 (P2_1): GPIO 11030000.pinctrl:529 > --- > drivers/pinctrl/renesas/pinctrl-rzg2l.c | 15 +++------------ > 1 file changed, 3 insertions(+), 12 deletions(-) > > diff --git a/drivers/pinctrl/renesas/pinctrl-rzg2l.c b/drivers/pinctrl/renesas/pinctrl-rzg2l.c > index 60ef20ca3ccf..1dceaf8290ea 100644 > --- a/drivers/pinctrl/renesas/pinctrl-rzg2l.c > +++ b/drivers/pinctrl/renesas/pinctrl-rzg2l.c > @@ -2368,20 +2368,11 @@ static const struct irq_chip rzg2l_gpio_irqchip = { > > static int rzg2l_gpio_interrupt_input_mode(struct gpio_chip *chip, unsigned int offset) > { > - struct rzg2l_pinctrl *pctrl = gpiochip_get_data(chip); > - const struct pinctrl_pin_desc *pin_desc = &pctrl->desc.pins[offset]; > - u64 *pin_data = pin_desc->drv_data; > - u32 off = RZG2L_PIN_CFG_TO_PORT_OFFSET(*pin_data); > - u8 bit = RZG2L_PIN_ID_TO_PIN(offset); > - u8 reg8; > int ret; > > - reg8 = readb(pctrl->base + PMC(off)); > - if (reg8 & BIT(bit)) { > - ret = rzg2l_gpio_request(chip, offset); > - if (ret) > - return ret; > - } > + ret = rzg2l_gpio_request(chip, offset); > + if (ret) > + return ret; > With this approach I'm getting the following on RZ/G3S SMARC Carrier II board: [ 0.368129] pinctrl-rzg2l 11030000.pinctrl: pinctrl-rzg2l support registered [ 0.390426] 1004b800.serial: ttySC0 at MMIO 0x1004b800 (irq = 42, base_baud = 0) is a scif [ 0.390558] printk: legacy console [ttySC0] enabled [ 1.601991] pinctrl-rzg2l 11030000.pinctrl: pin P12_0 already requested by 11030000.pinctrl:608; cannot claim for 11030000.pinctrl:608 [ 1.614208] pinctrl-rzg2l 11030000.pinctrl: error -EINVAL: pin-96 (11030000.pinctrl:608) [ 1.622313] gpio gpiochip0: (11030000.pinctrl): can't look up hwirq 96 [ 1.631801] ravb 11c30000.ethernet eth0: Base address at 0x11c30000, d2:7b:7f:8f:d8:52, IRQ 46. [ 1.645752] pinctrl-rzg2l 11030000.pinctrl: pin P12_1 already requested by 11030000.pinctrl:609; cannot claim for 11030000.pinctrl:609 [ 1.657923] pinctrl-rzg2l 11030000.pinctrl: error -EINVAL: pin-97 (11030000.pinctrl:609) [ 1.666035] gpio gpiochip0: (11030000.pinctrl): can't look up hwirq 97 [ 1.675573] ravb 11c40000.ethernet eth1: Base address at 0x11c40000, d2:7b:7f:8f:d8:52, IRQ 47. [ 1.700907] pinctrl-rzg2l 11030000.pinctrl: pin P18_0 already requested by 11030000.pinctrl:656; cannot claim for 11030000.pinctrl:656 [ 1.713272] pinctrl-rzg2l 11030000.pinctrl: error -EINVAL: pin-144 (11030000.pinctrl:656) [ 1.721496] gpio gpiochip0: (11030000.pinctrl): can't look up hwirq 144 [ 1.729209] pinctrl-rzg2l 11030000.pinctrl: pin P0_1 already requested by 11030000.pinctrl:513; cannot claim for 11030000.pinctrl:513 [ 1.741345] pinctrl-rzg2l 11030000.pinctrl: error -EINVAL: pin-1 (11030000.pinctrl:513) [ 1.749432] gpio gpiochip0: (11030000.pinctrl): can't look up hwirq 1 [ 1.756285] pinctrl-rzg2l 11030000.pinctrl: pin P0_3 already requested by 11030000.pinctrl:515; cannot claim for 11030000.pinctrl:515 [ 1.768475] pinctrl-rzg2l 11030000.pinctrl: error -EINVAL: pin-3 (11030000.pinctrl:515) [ 1.776524] gpio gpiochip0: (11030000.pinctrl): can't look up hwirq 3 [ 1.783124] gpio-keys keys: Found button without gpio or irq [ 1.788851] renesas_sdhi_internal_dmac 11c00000.mmc: mmc0 base at 0x0000000011c00000, max clock rate 125 MHz [ 1.798791] gpio-keys keys: probe with driver gpio-keys failed with error -22 All these ports are hogs to configure them as input. Removing the hog property make this patch work but I'm not sure this is the right approach (see below diff). diff --git a/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi b/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi index 71424e69939e..6e95933cd7ef 100644 --- a/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi +++ b/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi @@ -196,15 +196,6 @@ &sdhi2 { #endif &pinctrl { -#if SW_CONFIG3 == SW_ON - eth0-phy-irq-hog { - gpio-hog; - gpios = ; - input; - line-name = "eth0-phy-irq"; - }; -#endif - eth0_pins: eth0 { txc { pinmux = ; /* ET0_TXC */ @@ -239,15 +230,6 @@ mux { }; }; -#if SW_CONFIG3 == SW_ON - eth1-phy-irq-hog { - gpio-hog; - gpios = ; - input; - line-name = "eth1-phy-irq"; - }; -#endif - eth1_pins: eth1 { txc { pinmux = ; /* ET1_TXC */ diff --git a/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi index 4509151344c4..9dd313f6f8c2 100644 --- a/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi +++ b/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi @@ -71,26 +71,6 @@ &i2c0 { }; &pinctrl { - key-1-gpio-hog { - gpio-hog; - gpios = ; - input; - line-name = "key-1-gpio-irq"; - }; - - key-2-gpio-hog { - gpio-hog; - gpios = ; - input; - line-name = "key-2-gpio-irq"; - }; - - key-3-gpio-hog { - gpio-hog; - gpios = ; - input; - line-name = "key-3-gpio-irq"; - }; scif0_pins: scif0 { pinmux = , /* RXD */ > return rzg2l_gpio_direction_input(chip, offset); > }