From nobody Sat Oct 11 04:10:46 2025 Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) (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 7375427D782 for ; Wed, 11 Jun 2025 14:39:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.54 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749652765; cv=none; b=Mf2ZqiboxD4NdNw4dUBiw5KXLTWzFzbm9Fmj3yhqBtQc+TboV4L2Q7G4wQu6gocoCwHBQvvinNOI7EubQkvLfwsePe5Xfl+fF4idxScKTR6NYqa5DZW71NOunjA8zFgGH7e6S/k+WxNymzMyv8zjwWa7cawVATKUUmqDF4N7SuM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749652765; c=relaxed/simple; bh=H8gsYADbFcrqyC2iiieOy0OPyzJacy+R+vXTaFgq3kQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=tDW1sOOBW7cyiqMRLZ+DyaDMMuRb9Tfvr6+O76ClTYkQti1TPQ2bKJ9fjr1eetVWWErLmoRBo2H3cYBjWN9ixzEw4HHsstvoDFp665Op/O2r673U2Mk93yQK3MGL4ZBt7sLY2iqKHPCfVr5Xys80++4jUg4k9sceHhQyWd4c+xE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=VSVjXMwp; arc=none smtp.client-ip=209.85.128.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="VSVjXMwp" Received: by mail-wm1-f54.google.com with SMTP id 5b1f17b1804b1-45024721cbdso58123335e9.2 for ; Wed, 11 Jun 2025 07:39:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1749652762; x=1750257562; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=dMuC13ZuL2H9XthVI7VqSTJSYoQbIcVMKBT1sR6CtOo=; b=VSVjXMwpkeJU7215LtcMMaTzi5Z+d5K3YEkrrATUO4admLGqn21cZewxFIF40uiV5S PRaltWSLn6RYfsvqugcHHuNeBWx7fES61f4NuKKsPhECJszwAd0YI50kLPa7kGX9Qt31 WMJmTvdpBPzM3yavw2P4w6b0g1Fk7xq13/ZSAU56AQQ33obmmMPW/QNLBCfpfDa+yNkU LSlLKOTkDCU7fR0b6ONQ0EEnW2T/OUTZxzXHDTAgvGTdH735DQFTDOYMcR0T71aNlYa8 RFyjrrFj+TKCxbzMc/8gOtHoP3fbhaWk+UTBpC9bStjhEUOWicDbg7J95q080/7eWcgo mKWA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1749652762; x=1750257562; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=dMuC13ZuL2H9XthVI7VqSTJSYoQbIcVMKBT1sR6CtOo=; b=CfGv9xyH3jYHvSE5w/E7pKi2SmpvGyhIUh7vYvrqjNTSLvXAvgBylwpPZPccHPz5E/ OOcj96RGgdU+ak6bXFh1wXN8pCxIoSzCfGl6sWKrBD3CqzDWkHBlResaVKVx/0hH0Y/u IAoCWYBLq7ciBQuNDMShpuShM/yZrXqLNQgzhnuYyTmRr7dXKNFNdLiXp127d6cAdm7i X633ZngXIcBxPOvh0d/931uuFWNpU/65abW7WFVyRvHUKQXznNINj4DHOJhJWxNh24eD 1sRa3O1oNsnOkLFF+4bPRZWDpT99GWVy+yfW/c9pn3dCt1/HD6TRgkq3/6EUaJi6p5E6 kPlw== X-Forwarded-Encrypted: i=1; AJvYcCUMAjb1dbua1B/dM7Lsdt9rpsVtDp7vP2IadcVZlVs8hRUyn76rbDkYyNmu0TdTz4OFnn8d4Xb0A0Fq2ps=@vger.kernel.org X-Gm-Message-State: AOJu0YwoiUiqawSJJhU5AR5Q+yh56O93YjB9/xYXFtjiV7doNv3LjE89 MbLc+83S+N1m0SkvPfNraFZqFX5/ugBAlSJaMyNdZFBna2Dw59gcJ+7X7oJ1O9xiy3s= X-Gm-Gg: ASbGncunSuzoKNDotUWv4SuWbrMbH2f8vcYoJr0oosH2N2WgcnrkfyAEnyxRVqRJgcH mjQtw1hky6aiiE+9yD3oCpdcqyHiySAbe87L9ZO8ngHMcacncTBcPgicP32+gVhHSsbDC8Ub9Pt OF353UEas/TH/wpd20YieAAuLpbyrCeXREtkGxKHu99PJslDndYw1ZQgaH71nbHRCV3oUrawVYh OlmDuiL3MUlkJhT9dWRaMFOuT2T3DCVoQf/Z+s+RHe0LzgNFG+CZRShlkEDs06FT+4GrGDvvjR1 ipL6xCPFGX4hTRtQPRzN5yLHmgTI73kLzcVK4xV1C+NhMlvIf3g7/TErpmGjLx60OFXEhbGU2w= = X-Google-Smtp-Source: AGHT+IFgabml/oB/zap3gijznKuefYyLzl72L15Zdx3U7B8qLlQYx/5lfGPCZwHGnJ1ITnCk8LVHwQ== X-Received: by 2002:a05:600c:1d87:b0:43c:ed61:2c26 with SMTP id 5b1f17b1804b1-453248c08e4mr33894775e9.17.1749652761663; Wed, 11 Jun 2025 07:39:21 -0700 (PDT) Received: from brgl-uxlite.home ([2a01:cb1d:dc:7e00:3994:acce:fbc8:2f34]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-45325141245sm23208905e9.5.2025.06.11.07.39.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 11 Jun 2025 07:39:21 -0700 (PDT) From: Bartosz Golaszewski To: Bjorn Andersson , Linus Walleij Cc: linux-arm-msm@vger.kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski , stable@vger.kernel.org Subject: [PATCH] pinctrl: qcom: msm: mark certain pins as invalid for interrupts Date: Wed, 11 Jun 2025 16:39:11 +0200 Message-ID: <20250611143911.48501-1-brgl@bgdev.pl> X-Mailer: git-send-email 2.48.1 Precedence: bulk X-Mailing-List: linux-kernel@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" From: Bartosz Golaszewski When requesting pins whose intr_detection_width setting is not 1 or 2 for interrupts (for example by running `gpiomon -c 0 113` on RB2), we'll hit a BUG() in msm_gpio_irq_set_type(). Potentially crashing the kernel due to an invalid request from user-space is not optimal, so let's go through the pins and mark those that would fail the check as invalid for the irq chip as we should not even register them as available irqs. This function can be extended if we determine that there are more corner-cases like this. Fixes: f365be092572 ("pinctrl: Add Qualcomm TLMM driver") Cc: stable@vger.kernel.org Signed-off-by: Bartosz Golaszewski Reviewed-by: Bjorn Andersson --- drivers/pinctrl/qcom/pinctrl-msm.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/drivers/pinctrl/qcom/pinctrl-msm.c b/drivers/pinctrl/qcom/pinc= trl-msm.c index f012ea88aa22c..77e0c2f023455 100644 --- a/drivers/pinctrl/qcom/pinctrl-msm.c +++ b/drivers/pinctrl/qcom/pinctrl-msm.c @@ -1038,6 +1038,24 @@ static bool msm_gpio_needs_dual_edge_parent_workarou= nd(struct irq_data *d, test_bit(d->hwirq, pctrl->skip_wake_irqs); } =20 +static void msm_gpio_irq_init_valid_mask(struct gpio_chip *gc, + unsigned long *valid_mask, + unsigned int ngpios) +{ + struct msm_pinctrl *pctrl =3D gpiochip_get_data(gc); + const struct msm_pingroup *g; + int i; + + bitmap_fill(valid_mask, ngpios); + + for (i =3D 0; i < ngpios; i++) { + g =3D &pctrl->soc->groups[i]; + if (g->intr_detection_width !=3D 1 && + g->intr_detection_width !=3D 2) + clear_bit(i, valid_mask); + } +} + static int msm_gpio_irq_set_type(struct irq_data *d, unsigned int type) { struct gpio_chip *gc =3D irq_data_get_irq_chip_data(d); @@ -1441,6 +1459,7 @@ static int msm_gpio_init(struct msm_pinctrl *pctrl) girq->default_type =3D IRQ_TYPE_NONE; girq->handler =3D handle_bad_irq; girq->parents[0] =3D pctrl->irq; + girq->init_valid_mask =3D msm_gpio_irq_init_valid_mask; =20 ret =3D gpiochip_add_data(&pctrl->chip, pctrl); if (ret) { --=20 2.48.1