linux-kernel - [PATCH 1/3] mmc: core: Simplify sd_uhs2_power

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20241029131752.226764-2-ulf.hansson@linaro.org>
Date: Tue, 29 Oct 2024 14:17:47 +0100
From: Ulf Hansson <ulf.hansson@...aro.org>
To: linux-mmc@...r.kernel.org,
	Ulf Hansson <ulf.hansson@...aro.org>
Cc: Adrian Hunter <adrian.hunter@...el.com>,
	Victor Shih <victor.shih@...esyslogic.com.tw>,
	Victor Shih <victorshihgli@...il.com>,
	linux-kernel@...r.kernel.org
Subject: [PATCH 1/3] mmc: core: Simplify sd_uhs2_power_up()

Drop the redundant err-parameter and just return the result from
host->ops->uhs2_control() instead.

Signed-off-by: Ulf Hansson <ulf.hansson@...aro.org>
---
 drivers/mmc/core/sd_uhs2.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/mmc/core/sd_uhs2.c b/drivers/mmc/core/sd_uhs2.c
index ddd2291ad7c4..06857e1bbdb0 100644
--- a/drivers/mmc/core/sd_uhs2.c
+++ b/drivers/mmc/core/sd_uhs2.c
@@ -48,8 +48,6 @@ struct sd_uhs2_wait_active_state_data {
 
 static int sd_uhs2_power_up(struct mmc_host *host)
 {
-	int err;
-
 	if (host->ios.power_mode == MMC_POWER_ON)
 		return 0;
 
@@ -58,9 +56,7 @@ static int sd_uhs2_power_up(struct mmc_host *host)
 	host->ios.timing = MMC_TIMING_UHS2_SPEED_A;
 	host->ios.power_mode = MMC_POWER_ON;
 
-	err = host->ops->uhs2_control(host, UHS2_SET_IOS);
-
-	return err;
+	return host->ops->uhs2_control(host, UHS2_SET_IOS);
 }
 
 static int sd_uhs2_power_off(struct mmc_host *host)
-- 
2.43.0