diff options
author | Ulf Hansson <ulf.hansson@linaro.org> | 2025-06-10 13:16:25 +0200 |
---|---|---|
committer | Ulf Hansson <ulf.hansson@linaro.org> | 2025-06-24 12:43:26 +0200 |
commit | 55131d685ebee43d7f7260f34459201734ee6a44 (patch) | |
tree | 01c9617c35abdf5f78e55c90bc6a02a360296def | |
parent | f12b55b9782017edb0533463be6215e23f4b4f8f (diff) |
mmc: rtsx_usb_sdmmc: Convert sd_set_power_mode() into void
The sdmmc_set_ios() is the only caller of sd_set_power_mode() and it
ignores the return code. Let's therefore convert sd_set_power_mode() into a
void function instead.
Reviewed-by: Avri Altman <avri.altman@sandisk.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Acked-by: Ricky Wu <ricky_wu@realtek.com>
Link: https://lore.kernel.org/r/20250610111633.504366-4-ulf.hansson@linaro.org
-rw-r--r-- | drivers/mmc/host/rtsx_usb_sdmmc.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/mmc/host/rtsx_usb_sdmmc.c b/drivers/mmc/host/rtsx_usb_sdmmc.c index 0e2aac8dd59e..676f08f71ec0 100644 --- a/drivers/mmc/host/rtsx_usb_sdmmc.c +++ b/drivers/mmc/host/rtsx_usb_sdmmc.c @@ -1010,7 +1010,7 @@ static int sd_power_off(struct rtsx_usb_sdmmc *host) return sd_pull_ctl_disable_qfn24(ucr); } -static int sd_set_power_mode(struct rtsx_usb_sdmmc *host, +static void sd_set_power_mode(struct rtsx_usb_sdmmc *host, unsigned char power_mode) { int err; @@ -1019,7 +1019,7 @@ static int sd_set_power_mode(struct rtsx_usb_sdmmc *host, power_mode = MMC_POWER_ON; if (power_mode == host->power_mode) - return 0; + return; if (power_mode == MMC_POWER_OFF) { err = sd_power_off(host); @@ -1034,7 +1034,6 @@ static int sd_set_power_mode(struct rtsx_usb_sdmmc *host, } host->power_mode = power_mode; - return err; } static int sd_set_timing(struct rtsx_usb_sdmmc *host, |