mmc: rename mmc_can_trim() to mmc_card_can_trim()
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Tue, 1 Apr 2025 09:58:45 +0000 (11:58 +0200)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 14 May 2025 14:59:17 +0000 (16:59 +0200)
mmc_can_* functions sometimes relate to the card and sometimes to the host.
Make it obvious by renaming this function to include 'card'. Also, convert to
proper bool type while we are here.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Link: https://lore.kernel.org/r/20250401095847.29271-10-wsa+renesas@sang-engineering.com
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/core/block.c
drivers/mmc/core/core.c
drivers/mmc/core/core.h
drivers/mmc/core/mmc.c
drivers/mmc/core/mmc_test.c
drivers/mmc/core/queue.c

index 02a11a7..63320cc 100644 (file)
@@ -1284,7 +1284,7 @@ static void mmc_blk_issue_secdiscard_rq(struct mmc_queue *mq,
        from = blk_rq_pos(req);
        nr = blk_rq_sectors(req);
 
-       if (mmc_can_trim(card) && !mmc_erase_group_aligned(card, from, nr))
+       if (mmc_card_can_trim(card) && !mmc_erase_group_aligned(card, from, nr))
                arg = MMC_SECURE_TRIM1_ARG;
        else
                arg = MMC_SECURE_ERASE_ARG;
index f451903..a0e2dce 100644 (file)
@@ -1843,14 +1843,12 @@ bool mmc_card_can_erase(struct mmc_card *card)
 }
 EXPORT_SYMBOL(mmc_card_can_erase);
 
-int mmc_can_trim(struct mmc_card *card)
+bool mmc_card_can_trim(struct mmc_card *card)
 {
-       if ((card->ext_csd.sec_feature_support & EXT_CSD_SEC_GB_CL_EN) &&
-           (!(card->quirks & MMC_QUIRK_TRIM_BROKEN)))
-               return 1;
-       return 0;
+       return ((card->ext_csd.sec_feature_support & EXT_CSD_SEC_GB_CL_EN) &&
+               (!(card->quirks & MMC_QUIRK_TRIM_BROKEN)));
 }
-EXPORT_SYMBOL(mmc_can_trim);
+EXPORT_SYMBOL(mmc_card_can_trim);
 
 bool mmc_card_can_discard(struct mmc_card *card)
 {
@@ -1864,7 +1862,7 @@ EXPORT_SYMBOL(mmc_card_can_discard);
 
 bool mmc_card_can_sanitize(struct mmc_card *card)
 {
-       if (!mmc_can_trim(card) && !mmc_card_can_erase(card))
+       if (!mmc_card_can_trim(card) && !mmc_card_can_erase(card))
                return false;
        if (card->ext_csd.sec_feature_support & EXT_CSD_SEC_SANITIZE)
                return true;
@@ -1981,7 +1979,7 @@ unsigned int mmc_calc_max_discard(struct mmc_card *card)
                return card->pref_erase;
 
        max_discard = mmc_do_calc_max_discard(card, MMC_ERASE_ARG);
-       if (mmc_can_trim(card)) {
+       if (mmc_card_can_trim(card)) {
                max_trim = mmc_do_calc_max_discard(card, MMC_TRIM_ARG);
                if (max_trim < max_discard || max_discard == 0)
                        max_discard = max_trim;
index 3f7bd25..622085c 100644 (file)
@@ -119,7 +119,7 @@ int mmc_start_request(struct mmc_host *host, struct mmc_request *mrq);
 
 int mmc_erase(struct mmc_card *card, sector_t from, unsigned int nr, unsigned int arg);
 bool mmc_card_can_erase(struct mmc_card *card);
-int mmc_can_trim(struct mmc_card *card);
+bool mmc_card_can_trim(struct mmc_card *card);
 bool mmc_card_can_discard(struct mmc_card *card);
 bool mmc_card_can_sanitize(struct mmc_card *card);
 bool mmc_card_can_secure_erase_trim(struct mmc_card *card);
index 29970f1..e499835 100644 (file)
@@ -1806,7 +1806,7 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
        /* set erase_arg */
        if (mmc_card_can_discard(card))
                card->erase_arg = MMC_DISCARD_ARG;
-       else if (mmc_can_trim(card))
+       else if (mmc_card_can_trim(card))
                card->erase_arg = MMC_TRIM_ARG;
        else
                card->erase_arg = MMC_ERASE_ARG;
index 190ae17..be2d289 100644 (file)
@@ -1746,7 +1746,7 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test)
        struct timespec64 ts1, ts2;
        int ret;
 
-       if (!mmc_can_trim(test->card))
+       if (!mmc_card_can_trim(test->card))
                return RESULT_UNSUP_CARD;
 
        if (!mmc_card_can_erase(test->card))
@@ -1863,7 +1863,7 @@ static int mmc_test_profile_seq_trim_perf(struct mmc_test_card *test)
        struct timespec64 ts1, ts2;
        int ret;
 
-       if (!mmc_can_trim(test->card))
+       if (!mmc_card_can_trim(test->card))
                return RESULT_UNSUP_CARD;
 
        if (!mmc_card_can_erase(test->card))
index f65bf17..284856c 100644 (file)
@@ -186,7 +186,7 @@ static void mmc_queue_setup_discard(struct mmc_card *card,
        lim->max_hw_discard_sectors = max_discard;
        if (mmc_card_can_secure_erase_trim(card))
                lim->max_secure_erase_sectors = max_discard;
-       if (mmc_can_trim(card) && card->erased_byte == 0)
+       if (mmc_card_can_trim(card) && card->erased_byte == 0)
                lim->max_write_zeroes_sectors = max_discard;
 
        /* granularity must not be greater than max. discard */