RFC : mmc: mmc_blk_issue_secdiscard_rq() and mmc_blk_issue_discard_rq() effectively merged into one.

[NamJae Jeon <linkinjeon@xxxxxxxxx>]

Hi.

I am wordering why mmc_blk_issue_secdiscard_rq() and
mmc_blk_issue_discard_rq() is seperated.
So I try to make mmc_blk_issue_secdiscard_rq() and
mmc_blk_issue_discard_rq() effectively merged into one.

I want to know your opinion.

Thanks.

----------------------------------------------------------------------------------------------------------------------------------------------------------------------
static int mmc_blk_issue_discard_rq(struct mmc_queue *mq, struct request *req)
{
        struct mmc_blk_data *md = mq->data;
        struct mmc_card *card = md->queue.card;
        unsigned int from, nr, arg;
        int err = 0;
        from = blk_rq_pos(req);
        nr = blk_rq_sectors(req);
        if (req->cmd_flags & REQ_SECURE) {
                if (!mmc_can_secure_erase_trim(card)) {
                        err = -EOPNOTSUPP;
                        goto out;
                }
                if(mmc_can_trim(card) &&
!mmc_erase_group_aligned(card, from, nr))
                        arg = MMC_SECURE_TRIM1_ARG;
                else
                        arg = MMC_SECURE_ERASE_ARG;
        }
        else {
                if (!mmc_can_erase(card)) {
                        err = -EOPNOTSUPP;
                        goto out;
                }
                if (mmc_can_trim(card))
                        arg = MMC_TRIM_ARG;
                else
                        arg = MMC_ERASE_ARG;
        }

        if (card->quirks & MMC_QUIRK_INAND_CMD38) {
                if(req->cmd_flags & REQ_SECURE)
                        err = mmc_switch(card, EXT_CSD_CMD_SET_NORMAL,
                                        NAND_CMD38_ARG_EXT_CSD,
                                        arg == MMC_SECURE_TRIM1_ARG ?
                                        INAND_CMD38_ARG_SECTRIM1 :
                                        INAND_CMD38_ARG_SECERASE,
                                        0);
                else
                        err = mmc_switch(card, EXT_CSD_CMD_SET_NORMAL,
                                        INAND_CMD38_ARG_EXT_CSD,
                                        arg == MMC_TRIM_ARG ?
                                        INAND_CMD38_ARG_TRIM :
                                        INAND_CMD38_ARG_ERASE,
                                        0);
                if (err)
                        goto out;
        }
        err = mmc_erase(card, from, nr, arg);
        if (!err && arg == MMC_SECURE_TRIM1_ARG) {
                if (card->quirks & MMC_QUIRK_INAND_CMD38) {
                        err = mmc_switch(card, EXT_CSD_CMD_SET_NORMAL,
                                         INAND_CMD38_ARG_EXT_CSD,
                                         INAND_CMD38_ARG_SECTRIM2,
                                         0);
                        if (err)
                                goto out;
                }
                err = mmc_erase(card, from, nr, MMC_SECURE_TRIM2_ARG);
        }
out:
        spin_lock_irq(&md->lock);
        __blk_end_request(req, err, blk_rq_bytes(req));
        spin_unlock_irq(&md->lock);
        return err ? 0 : 1;
}


static int mmc_blk_issue_rq(struct mmc_queue *mq, struct request *req)
{
        int ret;
        struct mmc_blk_data *md = mq->data;
        struct mmc_card *card = md->queue.card;
        mmc_claim_host(card->host);
        ret = mmc_blk_part_switch(card, md);
        if (ret) {
                ret = 0;
                goto out;
        }
        if (req->cmd_flags & REQ_DISCARD) {
                ret = mmc_blk_issue_discard_rq(mq, req);
        } else if (req->cmd_flags & REQ_FLUSH) {
                ret = mmc_blk_issue_flush(mq, req);
        } else {
                ret = mmc_blk_issue_rw_rq(mq, req);
        }
out:
        mmc_release_host(card->host);
        return ret;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html