From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (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 5C3E42EFDA6 for ; Sat, 9 May 2026 08:45:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778316319; cv=none; b=d0lAJdHGi/qo95h652d60j/ma5fqPFE1XXIGLmf7DjFj+zdV1L8Fpcsy0tkPtWphb9vonWov1dsHMZ2zrgiQqMbyCiZ1yP0ANM6EL//1+5Agk3i/uGYGZfdVu88vz66YWXM8NTrU/DydWQ0mud21P0g1qYtpykrK5UrM9znBZfA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778316319; c=relaxed/simple; bh=TFa+xEui4mi+TZmuskFzKaQ5HYjVAER89kwrNi7tb1o=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=SynF0N0YBwyfikRx+Bmnj1zUCthyekD1whlxOVSkNurjjvwMqr2xt/D0w0/57Fg/KPx1eqZYEJWek2GHI8tmXEHKHnImI89KXMI+k0l+L4e8zY/uHrxerRGs0X/exmXHUWtXvsk8ROREoM5nCmTbyybXGFGLru/Nx5yP4Cwxe+k= 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=DX1YhGFs; arc=none smtp.client-ip=209.85.128.53 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="DX1YhGFs" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-48e56c1bf5dso16668115e9.3 for ; Sat, 09 May 2026 01:45:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778316317; x=1778921117; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=9M25LVxkvy4DTqh7u0Q+7wnu8lKX2GhtudATPtKQPlk=; b=DX1YhGFsP4zuLXMtQ13bu4Fz9R6tHhOCXpH23rbPB/bdmzmKOkfDEK/VqkfWIY2foq XyUGDuvoB07TlCaKaC+D3aq0Ir2BVOGeCdqGoRwmcMQklZragCTZnnJIW7bslG+UByLo 73+nzOVhvNY97OVz6QwLlYIO3MBdKLy5vCT1j+buc6ONCCSp818fLL1xR7lrIvv61C9M i7ipmfVTPaXWuIm+MomINVal0XoLeVMGMfw9K+HjLZIbmy8SLppU7xaf8+UI9ZQzl3FK D0irR3TL45SFKAm73B3SZcf3yp11PsxEJMQs0pkIpXpx7AzGkHdjOL0fMuCsciiBETLs /r2Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778316317; x=1778921117; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=9M25LVxkvy4DTqh7u0Q+7wnu8lKX2GhtudATPtKQPlk=; b=kZCYWsC0bP6t32LTeu5Y94JI8E5VYwvBhbhT5LfO5PddkoR/wXglNhZ7Th82CfsoT4 ziLLXwr8y8H+yAID+DewNh7JNGH5dOlAXfmblZgau5Fo4X7g+6I/Ztr8Ss7Aw42CuciS dkzUfnMTNlplGAq/Ck5WQKUuntAx03vN13SM69dV3izYGG81oCdsAHQx3iwEcJVoBEP/ qAemtfRFfa6T4qC/f+mpygH7zof1Cr1vTrVs6S6OqWRXy4DOZkUOQdchNgLpVc+KbiY7 HSns+06fxv57j9SQUp1Nedn6Wt4+/nMw9QgFsunuyM/0oXDOohO+9UTwBZU9IdDdkHqQ N36w== X-Forwarded-Encrypted: i=1; AFNElJ9GzniH9gTkdPdTQzIr5/mBijF8jqRaHf+/GtA4PhQQSa3cxe0iHkgUd04EtlITAxVanKwFdWsXpU/wbRc=@vger.kernel.org X-Gm-Message-State: AOJu0YzsiDPUBePXp8qW1LiJJC+UVU1567HSPmYznDdkZ/nH6Yv8n85K 2tZ0QLtB3NwFw9o5fbEaJoGB/Cj5W4Fyw38weNvPEXOqC3so+xuR3w8s X-Gm-Gg: Acq92OGQPJEbvn+NBIoq4bOnpd+/PCjv7xEpGsccjpr4eVtqXRlyVU9ZVQWolANSp2Q SmPrzkQOh2DMN0D9IME3KjH3sRPi8/DMheJxm2gZ+h48nGPNYS3Mc39Tk1ainOnOJOhalGyw0/d eH5/eWtr+P6VdCtzorkJhgnF7KsVYf298fqUOgIhXQL0HKj0oHlgt+YFsUo2tJiwnCAx9TW2X53 wnVJ5kjW+3/udV8NDE39eH0ISh7DvXMVA+dJQYSFg5D802QVjWhALQsdIhvJI08bQElAGsYNCW5 Exyoyw5weCei5reIjCDcbkbqx2TUYkj9u1O5RshwPC5Zo2N/eacxWhpx2h9CHCrKjflJfM6kAkC ID7enA2Ppzol5J0leXXCvPHc9teFcDjxsCJfGc3zda2oajwnBxza31J83QwUpWOmuebAICSCoDP PJQLcIbdqF2Karbb26ecPhWq4+FXEXrN3MD1aBSHZvHH/fDdONnIJ4pns5Cdw4v98TqgykSoe1d A== X-Received: by 2002:a05:600c:4ec7:b0:48e:51f8:eb37 with SMTP id 5b1f17b1804b1-48e706d8ad9mr26118485e9.25.1778316316665; Sat, 09 May 2026 01:45:16 -0700 (PDT) Received: from ?IPv6:2001:818:ea56:d000:56e0:ceba:7da4:6673? ([2001:818:ea56:d000:56e0:ceba:7da4:6673]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4548ec6b00fsm11698603f8f.11.2026.05.09.01.45.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 09 May 2026 01:45:16 -0700 (PDT) Message-ID: <6afc2b680efa5a64c59e9f6a65b3b42919cd8688.camel@gmail.com> Subject: Re: [PATCH v2] iio: adc: viperboard: Fix error handling in vprbrd_iio_read_raw From: Nuno =?ISO-8859-1?Q?S=E1?= To: Salah Triki , Jonathan Cameron , David Lechner , Nuno =?ISO-8859-1?Q?S=E1?= , Andy Shevchenko Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Joshua Crofts , Maxwell Doose Date: Sat, 09 May 2026 09:46:17 +0100 In-Reply-To: <20260507190751.28352-1-salah.triki@gmail.com> References: <20260507190751.28352-1-salah.triki@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.56.2 (3.56.2-2.fc42) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Thu, 2026-05-07 at 20:07 +0100, Salah Triki wrote: > The driver proceeds to the reception phase even if the preceding > transmission fails. >=20 > This uses a goto error label for an early bail out and ensures the mutex = is > properly unlocked in case of failure. >=20 > Fixes: ffd8a6e7a778 ("iio: adc: Add viperboard adc driver") > Signed-off-by: Salah Triki > Reviewed-by: Joshua Crofts > Reviewed-by: Maxwell Doose > --- LGTM (though Andy's comment makes sense): Reviewed-by: Nuno S=C3=A1 > Changes in v2: > =C2=A0 - Add "adc" to the patch title. > =C2=A0 - Move dev_err() out of the mutex critical section. > =C2=A0 - Include Reviewed-by tags from Joshua and Maxwell. >=20 > =C2=A0drivers/iio/adc/viperboard_adc.c | 4 +++- > =C2=A01 file changed, 3 insertions(+), 1 deletion(-) >=20 > diff --git a/drivers/iio/adc/viperboard_adc.c b/drivers/iio/adc/viperboar= d_adc.c > index 9bb0b83c8f67..f7ce3effd730 100644 > --- a/drivers/iio/adc/viperboard_adc.c > +++ b/drivers/iio/adc/viperboard_adc.c > @@ -70,8 +70,10 @@ static int vprbrd_iio_read_raw(struct iio_dev *iio_dev= , > =C2=A0 VPRBRD_USB_TYPE_OUT, 0x0000, 0x0000, admsg, > =C2=A0 sizeof(struct vprbrd_adc_msg), VPRBRD_USB_TIMEOUT_MS); > =C2=A0 if (ret !=3D sizeof(struct vprbrd_adc_msg)) { > - dev_err(&iio_dev->dev, "usb send error on adc read\n"); > =C2=A0 error =3D -EREMOTEIO; > + mutex_unlock(&vb->lock); > + dev_err(&iio_dev->dev, "usb send error on adc read\n"); > + goto error; > =C2=A0 } > =C2=A0 > =C2=A0 ret =3D usb_control_msg(vb->usb_dev,