From nobody Sat Feb 7 20:41:05 2026 Received: from mail-lf1-f65.google.com (mail-lf1-f65.google.com [209.85.167.65]) (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 7F4CC25393E for ; Tue, 7 Oct 2025 10:11:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.65 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759831911; cv=none; b=JRegyxT5EfdsdyTsCSn/kQLF3ZGBGsh2ioq4L4iqq9QfmgPc1cLetX800uyNgXrfhXIdqKKOCBtpL5Ae/P0awtU6jxys9pBzzMUzNTwjhRiBUbU3mtkNUOqd8X2A2eXVwvCnbuij2g7rAkxjeoZvD297avTpCKncSoMmXcgcu6g= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759831911; c=relaxed/simple; bh=aUB14bYr62eO+KCUUkphBpyq+IkN35gDqbsIwntERA8=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=fR8KpZNyQARhCHi8MDdPacV4/l4W7OgaJcLIDVsq+8QjctBtQWA25dJ3wI2r8YWjNwcbYcnT3yY9021tQxaG931i4w/lQRVtipaDWOnK11smmZBcUr0HerrgweisAQlnaPQQZS5p8JnIGT78k3cxRBaXzAYMgUY4ufI6TMF7hIk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=caSKUQlN; arc=none smtp.client-ip=209.85.167.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="caSKUQlN" Received: by mail-lf1-f65.google.com with SMTP id 2adb3069b0e04-5797c8612b4so7995031e87.2 for ; Tue, 07 Oct 2025 03:11:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1759831908; x=1760436708; 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=fh57GThCQIn/IKZtDhnvozk8Ln4QEkjmD5REYNdcWXw=; b=caSKUQlN8LOhG2VsO48j7x0Vk7TMVcollFUNCI8VOWuTozAslbz04B/aT14jqGpncT 7LXWchGAfFL20kxAfBYg7yeFFLeUIQ9+P3IjsqGwG9uRwtWXm57at7q3rlQL9Ld4Bxfl O0lq3OCrNI/nc6DxPclxykMxufKs6VoViu0+Pm4WUV0qsW4TKmpGcgg1soHwrL1mDKCz CkDQ047+E0vzHz/cdifYEc+k9krxu1XOGBe/iYh5bNM9Yfrarxtyc1O/xHDThN9ZzTyE aG0jgeFrhZc4twCCx7GbvIht2MvuHeUHqZGnF5Fz9VFzWvg+bSP6d2YG1yL1qzykttJi CNWQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1759831908; x=1760436708; 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=fh57GThCQIn/IKZtDhnvozk8Ln4QEkjmD5REYNdcWXw=; b=tn6ajVZS25WK6saXL75TRNGtD1UXzsQ8UoHXH/IIVjigzWmsv2fnf4ZxKWmmKZQ2l0 h1DIQg+BHdAPkrEwygo89El84Go9rh6JIUAcQ71oDX31rsNo6x/6v7mXgRIpkWhjHgST zst1eJ2o/H2ALEDz2mrzr9Afg8dekuxfLqtGhU118XkAGMMciniCpWyn6WNkAR7qrJel p4wnS8VpMvI/Wo/8MsItbK6e5zygdFlQI6nCCIDlq6TlAU1GljAo9BWU4JBNH34Mo/6I JVUoPadeYHpv4cn76xtDHSO0N6qnIoJnkOpb56UOoAB+FE26SUA/8puoQcxkejA6qaMK LUTA== X-Forwarded-Encrypted: i=1; AJvYcCW2dNkmgQ8/K5HxpZv7ddkxa8yKliEovTyeB/D782G+mhLtfH3alc83MXuKNdTgS+No1VWxYqHy5bhB+nU=@vger.kernel.org X-Gm-Message-State: AOJu0Yyddo23DUJjnmxXj7n95s1NnKL7D0QJGekm0J31AxCr+dd3/luB sBzeuyoUffTu6KSf5VkQ8j5UJfm4FZYY4O6S1PxuS+3ugM/xyandm579 X-Gm-Gg: ASbGncsjBfUx/Da/EzMI+IHWkHiwYY/+BrZ59iS1/+LCAJ0/0rLXPAcQ70dJz8Mg64F IV1bhMvZAV5MvQdzNDatOfy0RwS+KJ1qwQl71cAjrHg9Hee1vde7Qm+TXjZCBjUk1BGM1xMEMzt GQxmcd+iyfG+sGv4ai8l7h8KdNgsG/9RPybPP9W6UOllQxNKnw6zEJwvtBFBN68UZI8jlGEhIvM eaffIyy02Zmp8IO+wJIBG5FcvrJVjKi5605ky3MEL8BtSDSa9/Y8tDHNViZQsU5qo8svMO7qghs xMuR4BtIHgRybhG9A3KZQn90MOGKKCY0dTKLjkmolggb2OFJ8Yq1WB+Go44KRCCfMqSiCMOddf7 iAxz3uq4oXpnn/lRkN0ckzz7tOmA1rZJZsP/lzc5/ln4MhIe0hihOuK/x099JUQti X-Google-Smtp-Source: AGHT+IG8ywaUQNYKe9y5g2Et4+4rpuglgF9x9p0fXHFiYpqyFxiX6s/whUYu29aKmZ2P0BQJMTWsJA== X-Received: by 2002:a05:6512:3e04:b0:57c:6376:b42d with SMTP id 2adb3069b0e04-58cbbdd8267mr3873957e87.29.1759831907290; Tue, 07 Oct 2025 03:11:47 -0700 (PDT) Received: from NB-6746.corp.yadro.com ([188.243.183.84]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-58b011b14aesm5952260e87.128.2025.10.07.03.11.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Oct 2025 03:11:47 -0700 (PDT) From: Artem Shimko To: Mark Brown , Philipp Zabel Cc: a.shimko.dev@gmail.com, linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] spi: dw-mmio: add error handling for reset_control_deassert() Date: Tue, 7 Oct 2025 13:11:33 +0300 Message-ID: <20251007101134.1912895-1-a.shimko.dev@gmail.com> X-Mailer: git-send-email 2.43.0 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" Currently reset_control_deassert() is called without checking its return value. This can lead to silent failures when reset deassertion fails. Add proper error handling to: 1. Check the return value of reset_control_deassert() 2. Return the error to the caller 3. Provide meaningful error message using dev_err_probe() This ensures that reset-related failures are properly reported during probe and helps with debugging reset issues. Signed-off-by: Artem Shimko --- This patch addresses an issue in the DesignWare SPI MMIO driver where the return value of reset_control_deassert() was not being checked, potentially leading to silent failures. The problem occurs when reset deassertion fails - the driver would continue probing without the device being properly reset, which could result in unpredictable behavior or hardware malfunctions. Patch adds proper error checking and propagation for the reset_control_deassert() call, ensuring that probe fails gracefully when reset operations cannot be completed successfully. This change improves the robustness of the driver and provides better debugging information when reset-related issues occur. drivers/spi/spi-dw-mmio.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi-dw-mmio.c b/drivers/spi/spi-dw-mmio.c index f0f576fac77a..7a5197586919 100644 --- a/drivers/spi/spi-dw-mmio.c +++ b/drivers/spi/spi-dw-mmio.c @@ -358,7 +358,9 @@ static int dw_spi_mmio_probe(struct platform_device *pd= ev) if (IS_ERR(dwsmmio->rstc)) return PTR_ERR(dwsmmio->rstc); =20 - reset_control_deassert(dwsmmio->rstc); + ret =3D reset_control_deassert(dwsmmio->rstc); + if (ret) + return dev_err_probe(&pdev->dev, ret, "Failed to deassert resets\n"); =20 dws->bus_num =3D pdev->id; =20 --=20 2.43.0