From nobody Fri Dec 19 17:53:09 2025 Received: from mail-ed1-f54.google.com (mail-ed1-f54.google.com [209.85.208.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 18ACD1DF73A for ; Mon, 26 May 2025 09:47:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.54 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748252834; cv=none; b=BlnQkklEZW76UHtmYVHtcKo9om/APgNSL+IsEoIdPuZf4xfQZIYmUAVTR4DGq7y/uGcF9MHhvNXmEx1oRW/FhAVisx9qbou5nUKdpp9ovqTGtr61tG3JYWxzQe6qWi1Oq3KglN5nAI3XeD/3jMxlhoaE8YB7mWoJCJIO8jDKhRg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748252834; c=relaxed/simple; bh=yW+lzGrBHFbCZuynhHylQaKFrss+eF7xeqrzeY4Gh7w=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=aIKv/RA7OJna8mkFeOUuHwYMlpyOtr8lOMXbMZmmR49Pu/BZtBThulr6piT1eHV/iS71dmVTE7KEqwl36ffY1FBjveF4YB0PyWncMA/m+XljaSCxnpwW69u+V7xSdF3cer/Yp15r7MpFkbBEm7cvmw4TUB+2pY2/Tnzsuo34mY0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=kluBaCCM; arc=none smtp.client-ip=209.85.208.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="kluBaCCM" Received: by mail-ed1-f54.google.com with SMTP id 4fb4d7f45d1cf-5fa828b4836so314592a12.2 for ; Mon, 26 May 2025 02:47:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1748252831; x=1748857631; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=fAZVyD8nLejv+dB2iGsn12fr15PTc/vSHjJT53bWPAM=; b=kluBaCCMak3UcFbm9dxrzkyZmOGiFuIQiN4A3nRYE5Wo4DUcaAgI7hzSQ3TBHfMwCA sEpzfJyamUfoewwtnVtVYMn2mdq+b2OBmDrb4H7q4DLJVsBJFkIokgm71I7vSrdtUCei Yf7epeu3Z7ckEVA9G/oTsbn6khCjcmpz9OTqXdhHq54dk/ATPF6/yzq/IPNrDrB7UBuf NsQtMS4O9B4ughUc2kqhheUi+9dIPSQ8gD/FBBcy9mEnhKhVyxKsOHRm7anEtp/1BcpZ 0CqVbRrEjHd4iYojCJbBzTNWquuBlrry9ueGWr/+3UX4nI9TlUxcFdoFVFWd9XKfNJTo xMww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1748252831; x=1748857631; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=fAZVyD8nLejv+dB2iGsn12fr15PTc/vSHjJT53bWPAM=; b=pCHPfAshkRGtnxC20Pmfclb7QEg08rrvdaFOWkkPs4EX5KNBTf6TDSJSN3rUSzD/CW R7QFm+I4dcd9fXiZqp1c+Mm+Bzde7h8INwcIwz6hr9INxrBRz5ZqF+fwpnJwaXkWq3qV odrUfFN2mpToLgQMDbPlc0m/vEpyyS9neWbGJ9R1yWk1z9+poUYMkDWdAcEo5rWWr6+s NJtO2H4P5R/9AeHn6b/m2KfkJ7VEY0PESmmxyZaOSRJTf9IyfaalbkMqh4cxovpUlpjv 7s2wWhnT2+dBakwWLt8bp5sxjCI9cA7eyx0Bu9XkOAB0f2hnW1fDFmF6hNIfWy4sdvh0 vlsw== X-Forwarded-Encrypted: i=1; AJvYcCV6I4/djqZFSpPjo4JMtKvG08/osUZ9JrMzCLUWBYW29P9Ap9inv99b3jIEsSagQ6wprJRQPr95HavR1TA=@vger.kernel.org X-Gm-Message-State: AOJu0YxyQPylnjBoF+GESpxZEdAo6xF/GwjIBl1iwfr25uysP54D29sP QXBeGGsSJc820mTbP3p1NG8+F7zEhOnGYSFHI+q4cQB1N/Oy+a8n3BnakBrmPHpJlEo= X-Gm-Gg: ASbGncvb7/L4vDrHXZvsCq9CDxvb/PEDjYCScwHGYQZGAio8Aal5h5bnI7tGbOVVLbp 6IHcuzJ8GMTcHA0avdDCKyeJXL9AKf/p3IMCWSJtM1YtCt5dki1JEt09KkLRUVK1Su3GRtqApiF vIlL/gKLKMRo43pbJuQAR6QsLcCk18LAZ9U0lNp6+ycOXqhdErwcsBNI+u6VolVEGB8buV1Nh22 xFtAcCQDhwuyxPJZ2sR7MrHLtKdsAVDZmgfFweNcWmmFpSAb2+I8LIYcZGjRwdjkkqguWlAk2Mm KykQOQIYwSaDDJA4uCRfbECwqqOvwahTTMSrsYsMmHILOo3m80/QxzF0tdM1UygOMzA6IrU= X-Google-Smtp-Source: AGHT+IE08nBeINi9bzH0RFLrXFLcP2pZ6ple+DY12qw1tSMqI7k7yqjWGEAFPZqPtJ0gqM/FV3JHwA== X-Received: by 2002:a05:6402:354f:b0:5f6:c5e3:faa0 with SMTP id 4fb4d7f45d1cf-602db3b4ebdmr2818936a12.10.1748252831299; Mon, 26 May 2025 02:47:11 -0700 (PDT) Received: from [192.168.1.29] ([178.197.223.125]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6049d482cc7sm1486427a12.19.2025.05.26.02.47.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 26 May 2025 02:47:10 -0700 (PDT) From: Krzysztof Kozlowski Date: Mon, 26 May 2025 11:47:01 +0200 Subject: [PATCH 1/3] ASoC: codecs: wcd9335: Fix missing free of regulator supplies Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-1-0b8a2993b7d3@linaro.org> References: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-0-0b8a2993b7d3@linaro.org> In-Reply-To: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-0-0b8a2993b7d3@linaro.org> To: Srinivas Kandagatla , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Vinod Koul , Mohammad Rafi Shaik Cc: linux-sound@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Krzysztof Kozlowski X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=2678; i=krzysztof.kozlowski@linaro.org; h=from:subject:message-id; bh=yW+lzGrBHFbCZuynhHylQaKFrss+eF7xeqrzeY4Gh7w=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBoNDiZVUjOu00z65ZdGdWgoOvgj6MWG6QuPFcRI uoGQ61XyjeJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaDQ4mQAKCRDBN2bmhouD 11PDD/4hHos24HAAdgoQUdwzReUuAHWBaP6+UBZHkLMxHIf0Iy3EhDSZAuyoz2/YyBwgD+7DZoY oI+Bv0N/XfBHGlpaBauERGcubz3iPud4RfdPI+IgUnbE5nVgZFdmbauTOphrVBHGs6IsZfpvxx2 NnQ8LAOKWj32tFfFgA74egDTzcqW5AuzyyYmRwhV39nEhnh3XT3iOeO4/TNvShGYYAEhcHn0DeA A5ebV3FXEKrpahdxw/woyqJPH/j7cOW2GVzI8Z4ERfSUSEhOECEl/H06RNU3upPNAu0WR/EAR17 6tOV9d5TEAGwkP/YljJfjKTDmD6mZEe0BS0Z1VaWczIuJm9kl2LHKwk2yj912vCzbFcdkhio20c y2phNAjT3c4mmnsQqjGaZjljkMjFy6t1amcwkfFGs46CRz4ui6WUtU9J6FlVglGCgx9prHo8C18 f40tTh6GeHSBfihEL5HyZbN2VVRc748DdKVQcdnRsphK9uF1IidyLseZpDCLwisQbPD7pNCoQy5 j8c2P5+0XxzvlgALq/XrBD//sFyXtMc99pau9wO1RovJwnuWpxOYJLay3Rk6Pm5FOQ7DEhdXJ/t osuxTMVX14+yfGXx1HKplVUI4BjTdOfHgddZlpQFrYbtsfmLSE2CwHAzCJuu0faVr9ZaWMDu/XA HNjoOOxq1DcKatQ== X-Developer-Key: i=krzysztof.kozlowski@linaro.org; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B Driver gets and enables all regulator supplies in probe path (wcd9335_parse_dt() and wcd9335_power_on_reset()), but does not cleanup in final error paths and in unbind (missing remove() callback). This leads to leaked memory and unbalanced regulator enable count during probe errors or unbind. Fix this by converting entire code into devm_regulator_bulk_get_enable() which also greatly simplifies the code. Fixes: 20aedafdf492 ("ASoC: wcd9335: add support to wcd9335 codec") Cc: Signed-off-by: Krzysztof Kozlowski --- sound/soc/codecs/wcd9335.c | 25 +++++++------------------ 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/sound/soc/codecs/wcd9335.c b/sound/soc/codecs/wcd9335.c index 8ee4360aff9293178e338e3ef300c37f6f2ac809..5e19e813748dfa0d06528749424= 0007d60478dea 100644 --- a/sound/soc/codecs/wcd9335.c +++ b/sound/soc/codecs/wcd9335.c @@ -332,7 +332,6 @@ struct wcd9335_codec { =20 int intr1; struct gpio_desc *reset_gpio; - struct regulator_bulk_data supplies[WCD9335_MAX_SUPPLY]; =20 unsigned int rx_port_value[WCD9335_RX_MAX]; unsigned int tx_port_value[WCD9335_TX_MAX]; @@ -355,6 +354,10 @@ struct wcd9335_irq { char *name; }; =20 +static const char * const wcd9335_supplies[] =3D { + "vdd-buck", "vdd-buck-sido", "vdd-tx", "vdd-rx", "vdd-io", +}; + static const struct wcd9335_slim_ch wcd9335_tx_chs[WCD9335_TX_MAX] =3D { WCD9335_SLIM_TX_CH(0), WCD9335_SLIM_TX_CH(1), @@ -4989,30 +4992,16 @@ static int wcd9335_parse_dt(struct wcd9335_codec *w= cd) if (IS_ERR(wcd->native_clk)) return dev_err_probe(dev, PTR_ERR(wcd->native_clk), "slimbus clock not f= ound\n"); =20 - wcd->supplies[0].supply =3D "vdd-buck"; - wcd->supplies[1].supply =3D "vdd-buck-sido"; - wcd->supplies[2].supply =3D "vdd-tx"; - wcd->supplies[3].supply =3D "vdd-rx"; - wcd->supplies[4].supply =3D "vdd-io"; - - ret =3D regulator_bulk_get(dev, WCD9335_MAX_SUPPLY, wcd->supplies); + ret =3D devm_regulator_bulk_get_enable(dev, ARRAY_SIZE(wcd9335_supplies), + wcd9335_supplies); if (ret) - return dev_err_probe(dev, ret, "Failed to get supplies\n"); + return dev_err_probe(dev, ret, "Failed to get and enable supplies\n"); =20 return 0; } =20 static int wcd9335_power_on_reset(struct wcd9335_codec *wcd) { - struct device *dev =3D wcd->dev; - int ret; - - ret =3D regulator_bulk_enable(WCD9335_MAX_SUPPLY, wcd->supplies); - if (ret) { - dev_err(dev, "Failed to get supplies: err =3D %d\n", ret); - return ret; - } - /* * For WCD9335, it takes about 600us for the Vout_A and * Vout_D to be ready after BUCK_SIDO is powered up. --=20 2.45.2 From nobody Fri Dec 19 17:53:09 2025 Received: from mail-ed1-f51.google.com (mail-ed1-f51.google.com [209.85.208.51]) (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 9B7761F3BB5 for ; Mon, 26 May 2025 09:47:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748252836; cv=none; b=in5MQ27t9y59ZihW9o1p2bVQHhKei/YffdUYLb4/FtWBjbLSSFAEpo3NcHItUqp2pYXgdVvKHD6m87xULTdGjADqd2bBzT5IrzQ2qLlFEjF8npCLi48usUY8cmXiCVSALJbA6qSTB5l3uM90xn/8njLPW99w54sFZqb/FsB1DVE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748252836; c=relaxed/simple; bh=mT/EMNtSwpdKAfwY6okQuII7UuVyMAXDJYDo9/I1RAw=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=YKIU9tw1tk5fLbNJ41dgFw5jcfbF9tBhJiN8qA/xBoZgEFj1rWJqHKFEYazQYbqOUhIUjDxpqjuFZzmWvzL/He68UvlwbURTu8WMpRfhUmbvznS/DRDFfcRDqdAifsJcYbb+aOmk2IOZnCErSCIiv7KFnKdtbbviiJ68MKaVmF8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=WsMSj1PU; arc=none smtp.client-ip=209.85.208.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="WsMSj1PU" Received: by mail-ed1-f51.google.com with SMTP id 4fb4d7f45d1cf-604593544aaso310269a12.3 for ; Mon, 26 May 2025 02:47:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1748252833; x=1748857633; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=mf98aVEPRnEWwXE+rikHvFZrdzzyYrcXz6WwCds+ve0=; b=WsMSj1PUuH74yBrzXjSFpGhwG2yF0SY8AJ126a7w1me0zeLOL5xaQDK2yUFOg30H5j mM17ryX/TlFiC7BiyKtMs7Xmt0XNGaysSN6ToXyi4p0s8XGwE1bNcpt+qxLqyhx9+zZE R3I1WUznQTE6mlkHShw0HkUjCiqj0KG5rkQkYZOmmvVOhd6A7bsYAKjQtmA1XfsqR1ek 5ovKaj9U5nWV70EOr4+1Hb/2ajNNwcQ9S6GbbkaCqUtpRPxi8ku3kRNPbvn4H8Rxcog2 OCpC/hJ8hOlaptz66nbwEPANwLYnab8Ribaj5BO7Qrln2LmagrfQkgSxV/O8B/g/30aF 5lPg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1748252833; x=1748857633; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=mf98aVEPRnEWwXE+rikHvFZrdzzyYrcXz6WwCds+ve0=; b=PuR4jMNyClmnoj7x3WZrL1PZdpcz6KLRUGDKj1grfQcoPSIQnuxQqspTz8dUb/UT3o 3J4Gw1ULb4X/iOxprMNeuB1Y89sB0zeyXJjoHsmVcXXP+G/uWHUYC3Ug8dBjsGhLyL0n vbE9z8dkGkvt1lLuGprBuTU/NGYsHx13AkeaNnxfJYns5DEgn+KYPMHglf3en4B2XHlu tr5fsDbi4sxYJROoVwacrx3uYDd0lkEfhlFgku6pJvIuT+vW6gbmKWkP8YlP7BTqdOoM yQ8nDTMgKek36tWdCgkU6Gz61M17tsz8GPQ97dtnTac9OpzmohLikuy9hsRHW5dmRmLo COgg== X-Forwarded-Encrypted: i=1; AJvYcCWXqwW7NBQrENSNUlVPN9QpxwK9npUBibN577/Lg429ACQRuG+6ZRk5Mu3wA+MPTQM/MkhJc15MFlHZJ/E=@vger.kernel.org X-Gm-Message-State: AOJu0YwylpcOGy3PLRUIRwjbEO5a0LJHRkLnaAl1icVTpZCvZlKvc8AD fhMsJxa1pLnV0Q4fxpdnx+xo5rwonD3y4jM5COkqSsJTMBPwRUuWuhYm+GpZbF1kqlw= X-Gm-Gg: ASbGnctnBOgLlymcfDNVrX/0Yf7mibpoli4lkR4A986jWVeuyhoTNcuYoSb1JWEhDZj oBIEIIdHyjkWlJootlKWKFKkE8jIFzh+GmznPU1wIlDwaNwrRvz2LLysPPngJexwvWZcZe/L7IM LrYXC9ByHLPzl6fGRglUSpqcOozdnPAivePfWBNWreFCTjlpNIZ6q3+gkSh0C9BLH/J7p55wN5J eVwGDSPZVjsVNwAH0xOVxubfocmxvAveHMjAS9dczlAFnIf6CIecD1gkGHIGA/Sld0Kef+kypOb AjbMcuOst0Ks2plrV5Wwx9oOxaZ0Kcj/apZjtCDZAaIh4VJdnA0pLe4SYW/Alf77VymLXhk= X-Google-Smtp-Source: AGHT+IFQ/yOEW6qFlSR6tl4VswK3VUmEUrQUe/Qse8CIDzE90hh/Y0WqsQgHDIHFf+lTQEEGJWMcWw== X-Received: by 2002:a05:6402:26c3:b0:601:abc4:1b44 with SMTP id 4fb4d7f45d1cf-602d9df6cedmr2589786a12.4.1748252832752; Mon, 26 May 2025 02:47:12 -0700 (PDT) Received: from [192.168.1.29] ([178.197.223.125]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6049d482cc7sm1486427a12.19.2025.05.26.02.47.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 26 May 2025 02:47:12 -0700 (PDT) From: Krzysztof Kozlowski Date: Mon, 26 May 2025 11:47:02 +0200 Subject: [PATCH 2/3] ASoC: codecs: wcd937x: Drop unused buck_supply Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-2-0b8a2993b7d3@linaro.org> References: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-0-0b8a2993b7d3@linaro.org> In-Reply-To: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-0-0b8a2993b7d3@linaro.org> To: Srinivas Kandagatla , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Vinod Koul , Mohammad Rafi Shaik Cc: linux-sound@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Krzysztof Kozlowski X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=996; i=krzysztof.kozlowski@linaro.org; h=from:subject:message-id; bh=mT/EMNtSwpdKAfwY6okQuII7UuVyMAXDJYDo9/I1RAw=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBoNDiahh7YsUBAm8Iu89FMVlZwn3xet93APz5h0 3uJi60zcxKJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaDQ4mgAKCRDBN2bmhouD 1+fUD/981M4YfExEo4SwrDuA7PVrVCLEGQOILs/9VATrukI3ZP0GscQGTIPCQY9Zcx5WlwnfTDb tnejUOcxqApFXlwuzbuepvNtVlRyRqF6jvdO3D5DtAZHuW59VB4atNkVhLAPn9JLh+O9DTQJZpl BI6AnEYCTqwuQkVk+e6JqWC24GNeN9TSmSW4izZfzoQYU475omw9dQ5CuWU2BpMDsqAo7+73lc5 2S759DiwR764pmXfakow15a6gLPjPtllS6DXQe17NKHD5NTuR//X0Xjj3JDBQM6/FKmr2fHFL26 YXvWHOMcm+nCJZrjniXILk7Wp2zjFJ0RVufnPlKM0nlMg2w7R+mSX7eNyHMvjGp1F4XPVEjjFo0 rGZFNef/g+6uUEcwgwPj6johBCnCLEf0DgcIEXmNMDSllQtr1KtWVhHX4TzgBZB/5qC1UnXcvv3 XfO1iq2mA9Gd95SaL3edLoPl/sZyjgZFr0h9aMJIfB084y/8H9zZGnupVP0iWD21wrxmY1XJIvT lNinP/AEW/bwPtqAFuKc07u8lbqfpVkqseimONoPZBFpBHJXah5Z7C/DzjZzI2f39jD3I30/3WO 62WMeue++WBVUvLJAwH8VUeVDJQmqtk+HMSiwECH6a56t0tVpNgzTZqe4/AL73BdzWyL5R0r//h TUjsAnDn0MQgA0A== X-Developer-Key: i=krzysztof.kozlowski@linaro.org; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B Last user of wcd937x_priv->buck_supply was removed in commit 216d04139a6d ("ASoC: codecs: wcd937x: Remove separate handling for vdd-buck supply"). Fixes: 216d04139a6d ("ASoC: codecs: wcd937x: Remove separate handling for v= dd-buck supply") Cc: Signed-off-by: Krzysztof Kozlowski --- sound/soc/codecs/wcd937x.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/wcd937x.c b/sound/soc/codecs/wcd937x.c index 3b1a1518e7647366b4de0e90828b8d97b82c5bd5..a3a4b1f53e88e7ade13455387a6= 5de736b1f0bec 100644 --- a/sound/soc/codecs/wcd937x.c +++ b/sound/soc/codecs/wcd937x.c @@ -91,7 +91,6 @@ struct wcd937x_priv { struct regmap_irq_chip *wcd_regmap_irq_chip; struct regmap_irq_chip_data *irq_chip; struct regulator_bulk_data supplies[WCD937X_MAX_BULK_SUPPLY]; - struct regulator *buck_supply; struct snd_soc_jack *jack; unsigned long status_mask; s32 micb_ref[WCD937X_MAX_MICBIAS]; --=20 2.45.2 From nobody Fri Dec 19 17:53:09 2025 Received: from mail-ed1-f47.google.com (mail-ed1-f47.google.com [209.85.208.47]) (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 131521F4604 for ; Mon, 26 May 2025 09:47:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748252838; cv=none; b=O8OSFSDAxKfH5u0iIoNkkNQvEmkqtfYSTc4Mgf7JpYKKjeCwMZZyxwJ5COhhLe4FOUuAA6yUJ8jrKhBvWgKc7SRoBJOM2nhxxEMe+4VyNdvX4pZ1VIy+uxNLkTv4m9ZBbrW55bL1HjlHrQDKpCjbVzpGujnKbGa+zn6CuBMbmlo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748252838; c=relaxed/simple; bh=JFqzsTfm9DdoRKLGO7u+iVRbtQ2IB4LCND0J4CbvlC8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=i1wjUKq6sxQ9C4Yvs3RmZfs+ZEfsoCTu2q0DNQlBeU/eC3DIsyi0SWTjsplbYTaKFVk8dlO7hj95260g7d1j87p6kHOm2SvPzPaqtGKFHeLNoDB51k1mNrPdR1qVZrzCzBvfX2qwyLBzzGMIWR4qScCSB09eQeCU6Nna+WYoolo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=hN62oGW8; arc=none smtp.client-ip=209.85.208.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="hN62oGW8" Received: by mail-ed1-f47.google.com with SMTP id 4fb4d7f45d1cf-5faaa717cfbso303984a12.0 for ; Mon, 26 May 2025 02:47:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1748252834; x=1748857634; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=kSZZR0qMZkVAUpEjFq17SeucbREofsX6jeLhn+A+x9c=; b=hN62oGW8jcD5k9dcQnE57f7awlc/z/N01Xa9aE4iCkblAPmUKbxW8+6SSqD05N8o/Z 6g3pwrW5guDhwaLhVbqEvnd8GPyJYf+FdAuiVrNIh6BPuZXPZdtrYa6KBfXuENIDrTxL LtLXqD72FDqQZpd0sqSasT40VyhaqrhhcjUPS2YTMbns6i2kTtk6v/mOQk3fVF2RThAs vGEB95PgAQH2qWygttgacRM5fIQBg1FhfIfP2O6xPDRNCZzUbPoAmBvz0rQ939IxaJO9 L1DGlTTiKC0z+k4+uOx1bJmzCWF/Di5Xez5y1kKno2XFzw4eMYWy7paOODw5m3nN/c7w sN/w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1748252834; x=1748857634; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=kSZZR0qMZkVAUpEjFq17SeucbREofsX6jeLhn+A+x9c=; b=hVVdTP96M6FFfN4/Hqby3161lkKopcQzTMzGJImcSHL4WN46yIEXkd5mHkyUjrFZOI gXEZbfE+PDPNpzQgTqspbJHih0w+Ut4qf4o93JTXcXSwNn3Qzms4ZBpruykeuC9P3yDx WgmJgu0L4dcM33a43zRX2MoOsXkLcKghOj3PalxX4jgWzOUw7mzFHBVt82H0ZD7atcet JBiimmgPTKPLDOYQZRkfSS7UaC03SyECZoEZEObbWkZ5w8cbkFJckVS4Zc809KnpUAlu B+vbmGbGScB1NIc3NyLdsCoNjzvpTGWwURRb2iqDb/US0hLn1km8ZKpX5i3mx7c30vLc qQZA== X-Forwarded-Encrypted: i=1; AJvYcCXspOGRiRb1ZHSaSGRerYOhRYZIEOafOq3v+JHZISHrbK/WEXzRLllQLrpHllwuw3BYMGdNkt/Fqovje4o=@vger.kernel.org X-Gm-Message-State: AOJu0YzbkC5gkh+Xt2f8R/qrxoPrNQePNkgkiLYOuq4/CGPhZllXwYNH ugUVWdO9ObD7p4L6cB+4lWRp3Uu9fxP1tV1ssYSJmgkpbYKjm2t2qJR6ggq8+JUiA1U= X-Gm-Gg: ASbGncsWYA0L7JV2HiTHKNI3ccXUCW19xRPoEjH9y8q1hnYMTWSaT+iMPiacGxMvpEP Ve2JmxJx8onTeWCuOQyyBtcV0eOFGZxgNQj7n3tiXNPkQMxrfX3flnGNIwfNjXvmqtCe9mTzA6y h9Hp4nXjGy/rO1td1cDDgt6ZE9WsPCd6Ao5jtUawrvkip+YqRoX8CG2/IIkbLwBF5zLEC+gRp5y Wzn+mwpWxs4kwL+s0bESP/DGqc5BQeZ2DapslWrRKX7w3l8E+jTlOT3qbAi6Ou/pIVYPResjrPV z7K0nyDSvegeqlMJETzSMtVGh+BOox/rF7SM6CVuSOon0FeuCkhmkrQ0zYhCgRpMqpfWGgA= X-Google-Smtp-Source: AGHT+IE1rLSqoI6VnW/NLYoSivs2MuuZWDVumnm9UU8M7AtHBTKkJd/NjnzZ8iSRnfZzOFIqVY6fmg== X-Received: by 2002:a05:6402:2742:b0:602:a0:1f47 with SMTP id 4fb4d7f45d1cf-602daeb6a85mr2510849a12.8.1748252834261; Mon, 26 May 2025 02:47:14 -0700 (PDT) Received: from [192.168.1.29] ([178.197.223.125]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6049d482cc7sm1486427a12.19.2025.05.26.02.47.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 26 May 2025 02:47:13 -0700 (PDT) From: Krzysztof Kozlowski Date: Mon, 26 May 2025 11:47:03 +0200 Subject: [PATCH 3/3] ASoC: codecs: wcd9375: Fix double free of regulator supplies Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-3-0b8a2993b7d3@linaro.org> References: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-0-0b8a2993b7d3@linaro.org> In-Reply-To: <20250526-b4-b4-asoc-wcd9395-vdd-px-fixes-v1-0-0b8a2993b7d3@linaro.org> To: Srinivas Kandagatla , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Vinod Koul , Mohammad Rafi Shaik Cc: linux-sound@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Krzysztof Kozlowski X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=1715; i=krzysztof.kozlowski@linaro.org; h=from:subject:message-id; bh=JFqzsTfm9DdoRKLGO7u+iVRbtQ2IB4LCND0J4CbvlC8=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBoNDibdl94YMZwfVYjatgUDK/f/MsGw7A+4YxeR 4h2FKW1fSeJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaDQ4mwAKCRDBN2bmhouD 1xdCD/9vbaZu8P3oo43OAFrxr0vkTiREG8aTp5vpTZPLskJ+htHbnhA044cu38T+qsQF9RQc34s MH9ZhhXXdTkJPECc+RyRkdOO6CV41gQOlhzV8ucdoTYo7EBwzHXx9/QkGxcFEU6EJH278hhCBfG 8SEZJdrdNHri6a2MavtaOl3ZSaTkmJZXfM+wT8hiVYi7Cc17b4ZFpH5IyyZg/UczuXuCu8ghOJu GAaN+AqBcxVtOVGR2p9OLksELtTp5ico4csmbo/fdx/jNKa9VcxKfq8jseZSFi/4o0mf5rI/c3t uUwWjes6UcL6Lumzlh1pMm5I7+kjY40avjkF2POvtPtN85MuwDPqZCI2RVA98Os1YlvR43YSs65 02JVU7LboKXWH5cmviFsAHsdJJT7LYW+AYMBK7VVxjey4cPYvkJ/YwBZ1ywIOLr8Z8DbTzYotSS +L+9FQtWWS6EU8D09M9A5rOh8bqUT1Dt0nDwZfArDBvTx2Sl4QUJlZPGX/QawbtigbAfdI0CAWu eEfou2JRH9F5d7Hve/pnDQSGrAKxqwfdmC3wCDqBvq9ng438hcxrZY1aM8gVde6d9rkb9Z28Kpl AZFaVOu0Nulq9n3Ad0lJqPqqTGaxR4queZIllBjl0EztEW5IDmYw87McvXlvQTdKcqS+ToPILj1 f+4LnRh+j9PDAHQ== X-Developer-Key: i=krzysztof.kozlowski@linaro.org; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B Driver gets regulator supplies in probe path with devm_regulator_bulk_get(), so should not call regulator_bulk_free() in error and remove paths to avoid double free. Fixes: 216d04139a6d ("ASoC: codecs: wcd937x: Remove separate handling for v= dd-buck supply") Cc: Signed-off-by: Krzysztof Kozlowski --- sound/soc/codecs/wcd937x.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/sound/soc/codecs/wcd937x.c b/sound/soc/codecs/wcd937x.c index a3a4b1f53e88e7ade13455387a65de736b1f0bec..b9df58b86ce953427e01ffb8c7e= b7e52f9c2392a 100644 --- a/sound/soc/codecs/wcd937x.c +++ b/sound/soc/codecs/wcd937x.c @@ -2944,10 +2944,8 @@ static int wcd937x_probe(struct platform_device *pde= v) return dev_err_probe(dev, ret, "Failed to get supplies\n"); =20 ret =3D regulator_bulk_enable(WCD937X_MAX_BULK_SUPPLY, wcd937x->supplies); - if (ret) { - regulator_bulk_free(WCD937X_MAX_BULK_SUPPLY, wcd937x->supplies); + if (ret) return dev_err_probe(dev, ret, "Failed to enable supplies\n"); - } =20 wcd937x_dt_parse_micbias_info(dev, wcd937x); =20 @@ -2983,7 +2981,6 @@ static int wcd937x_probe(struct platform_device *pdev) =20 err_disable_regulators: regulator_bulk_disable(WCD937X_MAX_BULK_SUPPLY, wcd937x->supplies); - regulator_bulk_free(WCD937X_MAX_BULK_SUPPLY, wcd937x->supplies); =20 return ret; } @@ -3000,7 +2997,6 @@ static void wcd937x_remove(struct platform_device *pd= ev) pm_runtime_dont_use_autosuspend(dev); =20 regulator_bulk_disable(WCD937X_MAX_BULK_SUPPLY, wcd937x->supplies); - regulator_bulk_free(WCD937X_MAX_BULK_SUPPLY, wcd937x->supplies); } =20 #if defined(CONFIG_OF) --=20 2.45.2