Commit ee698f50 authored by Ido Yariv's avatar Ido Yariv Committed by Chris Ball

mmc: davinci: Poll status for small size transfers

For small size non-dma sdio transactions, it is sometimes better to poll
the mmc host and avoid interrupts altogether. Polling lowers the number
of interrupts and context switches. Tests have shown that for small
transactions, only a few polling iterations are needed, so this is worth
while.
Signed-off-by: default avatarIdo Yariv <ido@wizery.com>
Tested-by: default avatarRajashekhara, Sudhakar <sudhakar.raj@ti.com>
Signed-off-by: default avatarChris Ball <cjb@laptop.org>
parent be7b5622
...@@ -160,6 +160,16 @@ module_param(rw_threshold, uint, S_IRUGO); ...@@ -160,6 +160,16 @@ module_param(rw_threshold, uint, S_IRUGO);
MODULE_PARM_DESC(rw_threshold, MODULE_PARM_DESC(rw_threshold,
"Read/Write threshold. Default = 32"); "Read/Write threshold. Default = 32");
static unsigned poll_threshold = 128;
module_param(poll_threshold, uint, S_IRUGO);
MODULE_PARM_DESC(poll_threshold,
"Polling transaction size threshold. Default = 128");
static unsigned poll_loopcount = 32;
module_param(poll_loopcount, uint, S_IRUGO);
MODULE_PARM_DESC(poll_loopcount,
"Maximum polling loop count. Default = 32");
static unsigned __initdata use_dma = 1; static unsigned __initdata use_dma = 1;
module_param(use_dma, uint, 0); module_param(use_dma, uint, 0);
MODULE_PARM_DESC(use_dma, "Whether to use DMA or not. Default = 1"); MODULE_PARM_DESC(use_dma, "Whether to use DMA or not. Default = 1");
...@@ -193,6 +203,7 @@ struct mmc_davinci_host { ...@@ -193,6 +203,7 @@ struct mmc_davinci_host {
bool use_dma; bool use_dma;
bool do_dma; bool do_dma;
bool sdio_int; bool sdio_int;
bool active_request;
/* Scatterlist DMA uses one or more parameter RAM entries: /* Scatterlist DMA uses one or more parameter RAM entries:
* the main one (associated with rxdma or txdma) plus zero or * the main one (associated with rxdma or txdma) plus zero or
...@@ -219,6 +230,7 @@ struct mmc_davinci_host { ...@@ -219,6 +230,7 @@ struct mmc_davinci_host {
#endif #endif
}; };
static irqreturn_t mmc_davinci_irq(int irq, void *dev_id);
/* PIO only */ /* PIO only */
static void mmc_davinci_sg_to_buf(struct mmc_davinci_host *host) static void mmc_davinci_sg_to_buf(struct mmc_davinci_host *host)
...@@ -376,6 +388,19 @@ static void mmc_davinci_start_command(struct mmc_davinci_host *host, ...@@ -376,6 +388,19 @@ static void mmc_davinci_start_command(struct mmc_davinci_host *host,
writel(cmd->arg, host->base + DAVINCI_MMCARGHL); writel(cmd->arg, host->base + DAVINCI_MMCARGHL);
writel(cmd_reg, host->base + DAVINCI_MMCCMD); writel(cmd_reg, host->base + DAVINCI_MMCCMD);
host->active_request = true;
if (!host->do_dma && host->bytes_left <= poll_threshold) {
u32 count = poll_loopcount;
while (host->active_request && count--) {
mmc_davinci_irq(0, host);
cpu_relax();
}
}
if (host->active_request)
writel(im_val, host->base + DAVINCI_MMCIM); writel(im_val, host->base + DAVINCI_MMCIM);
} }
...@@ -915,6 +940,7 @@ mmc_davinci_xfer_done(struct mmc_davinci_host *host, struct mmc_data *data) ...@@ -915,6 +940,7 @@ mmc_davinci_xfer_done(struct mmc_davinci_host *host, struct mmc_data *data)
if (!data->stop || (host->cmd && host->cmd->error)) { if (!data->stop || (host->cmd && host->cmd->error)) {
mmc_request_done(host->mmc, data->mrq); mmc_request_done(host->mmc, data->mrq);
writel(0, host->base + DAVINCI_MMCIM); writel(0, host->base + DAVINCI_MMCIM);
host->active_request = false;
} else } else
mmc_davinci_start_command(host, data->stop); mmc_davinci_start_command(host, data->stop);
} }
...@@ -942,6 +968,7 @@ static void mmc_davinci_cmd_done(struct mmc_davinci_host *host, ...@@ -942,6 +968,7 @@ static void mmc_davinci_cmd_done(struct mmc_davinci_host *host,
cmd->mrq->cmd->retries = 0; cmd->mrq->cmd->retries = 0;
mmc_request_done(host->mmc, cmd->mrq); mmc_request_done(host->mmc, cmd->mrq);
writel(0, host->base + DAVINCI_MMCIM); writel(0, host->base + DAVINCI_MMCIM);
host->active_request = false;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment