From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7406A5BAD4; Tue, 23 Jan 2024 01:45:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705974356; cv=none; b=IOLqYMGKZs+qunRoe5ychybIPhLmVMgmdylhC0fVeO7gk5M40spfs4TG0/uaa/6JH1cuo2+rfPyQeBLLTgn0uHNI6o37eCdPLDbTJIpifFORiD8V6VqqzdWXJ7fNHE8tFwQegHosVAYmRXnG6s8wNIXlDIRGC+AjH0jSiH0fzbo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705974356; c=relaxed/simple; bh=5wJqw6fsHvp4O9WelU3s/ixWebkH+5Q8cXzsNQzdElI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=W1opSwAOhxd2tNEzR55ZCwt2qcD4XWL7eRERnMYHDJ3DLk8sK4K7nXfAXv6KTCXAVBVcssKsfqFs9V04cwCxvEZV+KMaxW929DAPyq2fGHRbEDhFA5yJvUNhZIdtEY3mKIhHO8GoZA3MCTRtpqDZ2XNBf8433xIJECDRGJ27QUM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=ZS8RQLbg; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="ZS8RQLbg" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 37B99C433F1; Tue, 23 Jan 2024 01:45:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1705974356; bh=5wJqw6fsHvp4O9WelU3s/ixWebkH+5Q8cXzsNQzdElI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ZS8RQLbgid+o1X47aU/8il0Kzu0LhZ4zu+XhDruOWTJNlH1+5S3AE8cHZPb3wfu8L 8KiIkyZ3wBmzheAfyzKwsfMqfxGlP5th/4O412LxmBQ02JjX4jGPSFXBT2aJq1bOOn 3yRzqQ49ta3qMsjRFBJ3L5fXwQMDHfgE6txV32c4= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Charles Keepax , Linus Walleij , Mark Brown , Sasha Levin Subject: [PATCH 5.15 190/374] ASoC: cs35l33: Fix GPIO name and drop legacy include Date: Mon, 22 Jan 2024 15:57:26 -0800 Message-ID: <20240122235751.222785292@linuxfoundation.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240122235744.598274724@linuxfoundation.org> References: <20240122235744.598274724@linuxfoundation.org> User-Agent: quilt/0.67 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 5.15-stable review patch. If anyone has any objections, please let me know. ------------------ From: Linus Walleij [ Upstream commit 50678d339d670a92658e5538ebee30447c88ccb3 ] This driver includes the legacy GPIO APIs and but does not use any symbols from any of them. Drop the includes. Further the driver is requesting "reset-gpios" rather than just "reset" from the GPIO framework. This is wrong because the gpiolib core will add "-gpios" before processing the request from e.g. device tree. Drop the suffix. The last problem means that the optional RESET GPIO has never been properly retrieved and used even if it existed, but nobody noticed. Fixes: 3333cb7187b9 ("ASoC: cs35l33: Initial commit of the cs35l33 CODEC driver.") Acked-by: Charles Keepax Signed-off-by: Linus Walleij Link: https://lore.kernel.org/r/20231201-descriptors-sound-cirrus-v2-2-ee9f9d4655eb@linaro.org Signed-off-by: Mark Brown Signed-off-by: Sasha Levin --- sound/soc/codecs/cs35l33.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sound/soc/codecs/cs35l33.c b/sound/soc/codecs/cs35l33.c index 2a6f5e46d031..3ea311fda98f 100644 --- a/sound/soc/codecs/cs35l33.c +++ b/sound/soc/codecs/cs35l33.c @@ -22,13 +22,11 @@ #include #include #include -#include #include #include #include #include #include -#include #include #include #include @@ -1169,7 +1167,7 @@ static int cs35l33_i2c_probe(struct i2c_client *i2c_client, /* We could issue !RST or skip it based on AMP topology */ cs35l33->reset_gpio = devm_gpiod_get_optional(&i2c_client->dev, - "reset-gpios", GPIOD_OUT_HIGH); + "reset", GPIOD_OUT_HIGH); if (IS_ERR(cs35l33->reset_gpio)) { dev_err(&i2c_client->dev, "%s ERROR: Can't get reset GPIO\n", __func__); -- 2.43.0