sdio: optimize sdio tuning print [1/1]

PD#SWPL-14443

Problem:
sdio tuning print info is too much.

Solution:
optimize tuning print.

Verify:
tl1

Change-Id: Ia0a0315eb30480b0014aa1e35d12b50576221f2e
Signed-off-by: Nan Li <nan.li@amlogic.com>
This commit is contained in:
Nan Li
2019-10-12 21:14:53 +08:00
committed by Luke Go
parent 098bcdb77b
commit d54d5f30d7
3 changed files with 43 additions and 8 deletions

View File

@@ -596,7 +596,7 @@ u32 aml_sd_emmc_tuning_transfer(struct mmc_host *mmc,
break;
}
} else {
pr_err("Tuning transfer error: nmatch=%d tuning_err:0x%x\n",
pr_debug("Tuning transfer error: nmatch=%d tuning_err:0x%x\n",
nmatch, tuning_err);
break;
}
@@ -617,6 +617,8 @@ static int aml_tuning_adj(struct mmc_host *mmc, u32 opcode,
u32 vctrl;
struct sd_emmc_config *ctrl = (struct sd_emmc_config *)&vctrl;
u32 clk_rate = 1000000000, clock, clk_div, nmatch = 0;
u8 *adj_print = host->adj_win;
u32 len = 0;
int adj_delay = 0;
const u8 *blk_pattern = tuning_data->blk_pattern;
unsigned int blksz = tuning_data->blksz;
@@ -638,6 +640,8 @@ static int aml_tuning_adj(struct mmc_host *mmc, u32 opcode,
pr_info("%s: clk %d %s tuning start\n",
mmc_hostname(mmc), (ctrl->ddr ? (clock / 2) : clock),
(ctrl->ddr ? "DDR mode" : "SDR mode"));
memset(adj_print, 0, sizeof(u8) * ADJ_WIN_PRINT_MAXLEN);
len += sprintf(adj_print + len, "%s: adj_win: < ", pdata->pinname);
for (adj_delay = 0; adj_delay < clk_div; adj_delay++) {
adjust = readl(host->base + SD_EMMC_ADJUST);
gadjust->adj_delay = adj_delay;
@@ -656,7 +660,9 @@ static int aml_tuning_adj(struct mmc_host *mmc, u32 opcode,
if (curr_win_start < 0)
curr_win_start = adj_delay;
curr_win_size++;
pr_info("%s: rx_tuning_result[%d] = %d\n",
len += sprintf(adj_print + len,
"%d ", adj_delay);
pr_debug("%s: rx_tuning_result[%d] = %d\n",
mmc_hostname(host->mmc), adj_delay, nmatch);
} else {
if (curr_win_start >= 0) {
@@ -675,6 +681,9 @@ static int aml_tuning_adj(struct mmc_host *mmc, u32 opcode,
}
}
}
len += sprintf(adj_print + len, ">\n");
pr_info("%s", host->adj_win);
/* last point is ok! */
if (curr_win_start >= 0) {
if (best_win_start < 0) {
@@ -3118,6 +3127,11 @@ static int meson_mmc_probe(struct platform_device *pdev)
ret = -ENOMEM;
goto fail_init_host;
}
host->adj_win = kzalloc(sizeof(u8) * ADJ_WIN_PRINT_MAXLEN, GFP_KERNEL);
if (host->adj_win == NULL) {
ret = -ENOMEM;
goto fail_init_host;
}
spin_lock_init(&host->mrq_lock);
mutex_init(&host->pinmux_lock);
@@ -3353,6 +3367,7 @@ free_cali:
kfree(host->desc_bn);
#endif
kfree(host->blk_test);
kfree(host->adj_win);
fail_init_host:
kfree(host);
pr_err("%s() fail!\n", __func__);
@@ -3384,6 +3399,7 @@ static int meson_mmc_remove(struct platform_device *pdev)
clk_disable_unprepare(host->core_clk);
kfree(host->blk_test);
kfree(host->adj_win);
mmc_free_host(host->mmc);
kfree(pdata);
kfree(host);

View File

@@ -1542,10 +1542,10 @@ static void pr_adj_info(char *name,
{
int i;
pr_info("[%s] fixed_adj_win_map:%lu\n",
pr_debug("[%s] fixed_adj_win_map:%lu\n",
name, x);
for (i = 0; i < div; i++)
pr_info("[%d]=%d\n", (fir_adj + i) % div,
pr_debug("[%d]=%d\n", (fir_adj + i) % div,
((x >> i) & 0x1) ? 1 : 0);
}
@@ -1555,6 +1555,8 @@ static unsigned long _test_fixed_adj(struct amlsd_platform *pdata,
{
struct amlsd_host *host = pdata->host;
int i = 0;
u8 *adj_print = host->adj_win;
u32 len = 0;
u32 nmatch = 0;
u32 adjust = readl(host->base + SD_EMMC_ADJUST_V3);
struct sd_emmc_adjust_v3 *gadjust =
@@ -1563,6 +1565,8 @@ static unsigned long _test_fixed_adj(struct amlsd_platform *pdata,
unsigned int blksz = tuning_data->blksz;
DECLARE_BITMAP(fixed_adj_map, div);
memset(adj_print, 0, sizeof(u8) * ADJ_WIN_PRINT_MAXLEN);
len += sprintf(adj_print + len, "%s: adj_win: < ", pdata->pinname);
bitmap_zero(fixed_adj_map, div);
for (i = 0; i < div; i++) {
gadjust->adj_delay = adj + i;
@@ -1576,10 +1580,15 @@ static unsigned long _test_fixed_adj(struct amlsd_platform *pdata,
/*get a ok adjust point!*/
if (nmatch == TUNING_NUM_PER_POINT) {
set_bit(adj+i, fixed_adj_map);
pr_info("%s: rx_tuning_result[%d] = %d\n",
len += sprintf(adj_print + len,
"%d ", gadjust->adj_delay);
pr_debug("%s: rx_tuning_result[%d] = %d\n",
mmc_hostname(host->mmc), adj+i, nmatch);
}
}
len += sprintf(adj_print + len, ">\n");
pr_info("%s", host->adj_win);
return *fixed_adj_map;
}
@@ -1647,7 +1656,7 @@ static u32 _find_fixed_adj_valid_win(struct amlsd_platform *pdata,
*prev_map = *fixed_adj_map;
pr_adj_info("prev_map", *prev_map, 0, div);
for (; step <= 63;) {
pr_info("[%s]retry test fixed adj...\n", __func__);
pr_debug("[%s]retry test fixed adj...\n", __func__);
step += AML_FIXED_ADJ_STEP;
set_fixed_adj_line_delay(step, pdata);
*cur_map = _test_fixed_adj(pdata, tuning_data, opcode, 0, div);
@@ -1657,7 +1666,7 @@ static u32 _find_fixed_adj_valid_win(struct amlsd_platform *pdata,
if (*dst != 0) {
fir_adj = find_first_bit(dst, div);
pr_adj_info(">>>>>>>>bitmap_xor_dst", *dst, 0, div);
pr_info("[%s] fir_adj:%u\n", __func__, fir_adj);
pr_debug("[%s] fir_adj:%u\n", __func__, fir_adj);
*prev_map = _swap_fixed_adj_win(*prev_map,
fir_adj, div);
@@ -1713,6 +1722,8 @@ static int _aml_sd_emmc_execute_tuning(struct mmc_host *mmc, u32 opcode,
u32 old_dly, d1_dly, dly;
unsigned long fixed_adj_map[1];
bool all_flag = false;
u8 *adj_print = host->adj_win;
u32 len = 0;
if ((host->mem->start == host->data->port_b_base)
&& host->data->tdma_f)
@@ -1732,6 +1743,8 @@ tunning:
curr_win_start = -1;
curr_win_size = 0;
memset(adj_print, 0, sizeof(u8) * ADJ_WIN_PRINT_MAXLEN);
len += sprintf(adj_print + len, "%s: adj_win: < ", pdata->pinname);
spin_lock_irqsave(&host->mrq_lock, flags);
pdata->need_retuning = false;
spin_unlock_irqrestore(&host->mrq_lock, flags);
@@ -1765,7 +1778,8 @@ tunning:
curr_win_start = adj_delay;
curr_win_size++;
pr_info("%s: rx_tuning_result[%d] = %d\n",
len += sprintf(adj_print + len, "%d ", adj_delay);
pr_debug("%s: rx_tuning_result[%d] = %d\n",
mmc_hostname(host->mmc), adj_delay, nmatch);
if (clk_div <= AML_FIXED_ADJ_MAX)
set_bit(adj_delay, fixed_adj_map);
@@ -1787,6 +1801,9 @@ tunning:
}
}
}
len += sprintf(adj_print + len, ">\n");
pr_info("%s", host->adj_win);
/* last point is ok! */
if (curr_win_start >= 0) {
if (best_win_start < 0) {

View File

@@ -45,6 +45,7 @@
#define SD_EMMC_MANUAL_CMD23
#define MAX_TUNING_RETRY 4
#define TUNING_NUM_PER_POINT 40
#define ADJ_WIN_PRINT_MAXLEN 256
#define CALI_PATTERN_OFFSET ((SZ_1M * (36 + 3)) / 512)
/* #define AML_RESP_WR_EXT */
/* pio to transfer data */
@@ -478,6 +479,7 @@ struct amlsd_host {
unsigned long clk_rate;
u8 *blk_test;
u8 *adj_win;
char *desc_buf;
#ifdef CFG_SDEMMC_PIO
/* bounce buffer to accomplish 32bit apb access */