From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f48.google.com (mail-wr1-f48.google.com [209.85.221.48]) (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 91D85340A6C for ; Thu, 26 Mar 2026 09:31:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774517471; cv=none; b=OlIiFASeXBE3JwsHNv+RQ8Ei2Fhtk/2QdB00qvdbR6lyT5WZATzl4tWvgtSjUjGWz0lNPyM3mE+R1Z2tA293e7A3874IMVeou440G43RdwmLNsVjnD9D3odLKX03VQZ4NixcHxsOBtzlq4Ukkgz8m9TLEz3HD30vCayuvAfUQls= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774517471; c=relaxed/simple; bh=pwN5FRRxqmIQfiRU5lLTXT3FDrbmu23OTSn70Ln03V8=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=N/Ve16CjWioJzBRB0HuECCRge71INIXHf2p2sw2i2e9qWmOIDk7+fwZxHmXxdh3jp2PxUZ2lY7lvVFzjQwzDtqn/wP+4B79BOhwyQgBKqn9yrTmaT8kA6tNEg6tZ4S03D7DhcrAbjP1gVyuSDYmTSAz7I93xdWjh7jjwRE5nKJQ= 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=qyJqC1Dk; arc=none smtp.client-ip=209.85.221.48 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="qyJqC1Dk" Received: by mail-wr1-f48.google.com with SMTP id ffacd0b85a97d-439b2965d4bso547679f8f.2 for ; Thu, 26 Mar 2026 02:31:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1774517467; x=1775122267; 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=pwN5FRRxqmIQfiRU5lLTXT3FDrbmu23OTSn70Ln03V8=; b=qyJqC1DkNP2jB4kt/uE0j/v01TVqMT6qlv0bkSz7tbTy9K0x1S7hpgfTPRmSbdXNZ4 sPC3Qm+4ISkZt978k4KD76GvZiDLPHqlIRsYw6rjeoBK2IbbJuuweiVvgI0IB/ILQuaF MO5IHT6HTzgjO6ZkIpQ5Y1WrM/6u5BXfKjjzYXKnVssVAUAxxonoHV5+J1fTWR9811eh 0YjZqD4t9QXXW1nWu8H5SnCQjOVDgP5Ap4foWbJzAAUSFKfuIBY9isHGwCYVe7fF+77E SPQEay9EN9ENIQQnOk40FAFg+DAV0EcqdPlc+V2oz1NaYuSy2jv3G0qqdNskWRyuf7UP vv5w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774517467; x=1775122267; 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=pwN5FRRxqmIQfiRU5lLTXT3FDrbmu23OTSn70Ln03V8=; b=VRDT0VJI0uM5A6Cd7pHJH6Kw24kVyqHLTD7I92mat2GvNZu05zZl8+qxU9+mxJbQ+3 RnE2V5e9AJzLIm+WZhWEmqldnJXRmm+KrbidSUiOZRaSJU8l8y9bYl9xcZArx4A7YxDV ZngS5Z0qZmT1AwJhfYmYvWVZ0+x2DdkGTXZzvsj1mTg/EzNGun8ftybKjvTVAp84spd2 yjUYy0BJfOqkIcB7TTgJAi/yeV8Xqe3g8PYeBJIGP0JF2UaKmpp9/WzZps6f9fHeAX71 QMrDzA/rz6iUDdi8jKk9nJOr209iIIGffXDx2c9YexZgpBnesPSwjNuPmKyqnBHURDiU 5FeQ== X-Forwarded-Encrypted: i=1; AJvYcCUGnIvLsSA/W5sXy1YRWbL1e2xQf96o5GvXC/Ja9j91bHEcQbMjj+8lpu2MvI7WZr177cuLPFR5b3dskew=@vger.kernel.org X-Gm-Message-State: AOJu0Yyd9H9+3/302Qg3dOsm554g5wBu0cEMIYVHz0e+U/DBzJjFivAE BGm6L3pTP9nW0ynUcDzfztzAyGN5q1UAXI/LYqpxmUXv5I9DM5x8whHF X-Gm-Gg: ATEYQzzVd+dJOqQJFlleXxIK7LahIz20Af+Hv55aCrrAGqnn/bgqkznsLippY/6S/Bv /iY7pezYS6GhzuIG+jviBCrHeSoqcd/eKbJlEgMxgY/RK380D+VMZ59aYIAHVdO5ej8uembe3qK iQskzMqaP9LbE660vBDAijImtiiyOez2cc03UMxVzJRvphJcLlLzml9+r6AmAk+i3hkZc9QC+0m Ycpd0ixrmTsstDdOgV63VLBbU4nifqgW1kfI4XB0HXnOIAM0/IajSrJUvkWFj7ZSu2gmnZ2yoy8 RFXXG3EmVl7i3xLcbEEIDJGaz6jT6ORkY3FaMU5JYzzzrQxn1ihKcba5Pyg/dHhuI7tDQYTT0Pk wrmEcF0Jwu0s85Jx3W769XsVbof3JpoLUW+76Mp1JgWhoE27X/Oee4KJuQ8z0wXEUYMKQk16Yiu ykFaMoUbTVQiMZFbYT2rWg0IMuOFu136g= X-Received: by 2002:a05:6000:2481:b0:439:b791:f914 with SMTP id ffacd0b85a97d-43b889a4a5fmr9755313f8f.18.1774517467372; Thu, 26 Mar 2026 02:31:07 -0700 (PDT) Received: from [192.168.1.187] ([148.63.225.166]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43b91942f24sm7173306f8f.11.2026.03.26.02.31.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 26 Mar 2026 02:31:07 -0700 (PDT) Message-ID: Subject: Re: [PATCH v2 0/2] ASoC: adau1372: Fix error handling in adau1372_set_power() From: Nuno =?ISO-8859-1?Q?S=E1?= To: Jihed Chaibi , lars@metafoo.de, nuno.sa@analog.com Cc: lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 26 Mar 2026 09:31:53 +0000 In-Reply-To: <20260325210704.76847-1-jihed.chaibi.dev@gmail.com> References: <20260325210704.76847-1-jihed.chaibi.dev@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.3 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Wed, 2026-03-25 at 22:07 +0100, Jihed Chaibi wrote: > adau1372_set_power() had two related error handling issues in its enable > path: clk_prepare_enable() was called but its return value discarded, and > adau1372_enable_pll() was a void function that silently swallowed lock > failures, leaving mclk enabled and adau1372->enabled set to true despite > the device being in a broken state. >=20 > Patch 1 fixes the unchecked clk_prepare_enable() by making > adau1372_set_power() return int and propagating the error. >=20 > Patch 2 converts adau1372_enable_pll() to return int and adds a full > unwind in adau1372_set_power() if PLL lock fails, reversing the regcache, > GPIO power-down, and clock state. >=20 > Changes in v2: > - [1/2]: No changes. > - [2/2]: Also unwind regcache and GPIO power-down state on PLL lock > =C2=A0 failure, as noted by Mark Brown. >=20 > Jihed Chaibi (2): > =C2=A0 ASoC: adau1372: Fix unchecked clk_prepare_enable() return value > =C2=A0 ASoC: adau1372: Fix clock leak on PLL lock failure >=20 > =C2=A0sound/soc/codecs/adau1372.c | 34 ++++++++++++++++++++++----------- > =C2=A01 file changed, 24 insertions(+), 10 deletions(-) >=20 > -- > 2.47.3 With proper tags: Reviewed-by: Nuno S=C3=A1