[U-Boot] [PATCH] mmc: dw_mmc: remove the exynos specific code in dw-mmc.c

[Minkyu Kang]

On 29/11/13 21:01, Pantelis Antoniou wrote:
> Hi Jaehoon,
> 
> Looks good
> 
> On Nov 29, 2013, at 1:08 PM, Jaehoon Chung wrote:
> 
>> dw-mmc.c is the general driver file.
>> So, remove the exynos specific code at dw-mmc.c.
>> Instead, exynos specific cod can be move into exynos-dw_mmc.c.
>>
>> Signed-off-by: Jaehoon Chung <jh80.chung at samsung.com>
>> ---
>> drivers/mmc/dw_mmc.c        |   12 ++----------
>> drivers/mmc/exynos_dw_mmc.c |   14 ++++++++++++++
>> include/dwmmc.h             |    1 +
>> 3 files changed, 17 insertions(+), 10 deletions(-)
>>
>> diff --git a/drivers/mmc/dw_mmc.c b/drivers/mmc/dw_mmc.c
>> index 1e0f72b..38f50ce 100644
>> --- a/drivers/mmc/dw_mmc.c
>> +++ b/drivers/mmc/dw_mmc.c
>> @@ -11,7 +11,6 @@
>> #include <mmc.h>
>> #include <dwmmc.h>
>> #include <asm-generic/errno.h>
>> -#include <asm/arch/dwmmc.h>
>>
>> #define PAGE_SIZE 4096
>>
>> @@ -302,15 +301,8 @@ static int dwmci_init(struct mmc *mmc)
>> 	struct dwmci_host *host = (struct dwmci_host *)mmc->priv;
>> 	u32 fifo_size;
>>
>> -	if (host->quirks & DWMCI_QUIRK_DISABLE_SMU) {
>> -		dwmci_writel(host, EMMCP_MPSBEGIN0, 0);
>> -		dwmci_writel(host, EMMCP_SEND0, 0);
>> -		dwmci_writel(host, EMMCP_CTRL0,
>> -			     MPSCTRL_SECURE_READ_BIT |
>> -			     MPSCTRL_SECURE_WRITE_BIT |
>> -			     MPSCTRL_NON_SECURE_READ_BIT |
>> -			     MPSCTRL_NON_SECURE_WRITE_BIT | MPSCTRL_VALID);
>> -	}
>> +	if (host->board_init)
>> +		host->board_init(host);
>>
>> 	dwmci_writel(host, DWMCI_PWREN, 1);
>>
>> diff --git a/drivers/mmc/exynos_dw_mmc.c b/drivers/mmc/exynos_dw_mmc.c
>> index a0f1511..b3e5c5e 100644
>> --- a/drivers/mmc/exynos_dw_mmc.c
>> +++ b/drivers/mmc/exynos_dw_mmc.c
>> @@ -34,6 +34,19 @@ unsigned int exynos_dwmci_get_clk(int dev_index)
>> 	return get_mmc_clk(dev_index);
>> }
>>
>> +static void exynos_dwmci_board_init(struct dwmci_host *host)
>> +{
>> +	if (host->quirks & DWMCI_QUIRK_DISABLE_SMU) {
>> +		dwmci_writel(host, EMMCP_MPSBEGIN0, 0);
>> +		dwmci_writel(host, EMMCP_SEND0, 0);
>> +		dwmci_writel(host, EMMCP_CTRL0,
>> +			     MPSCTRL_SECURE_READ_BIT |
>> +			     MPSCTRL_SECURE_WRITE_BIT |
>> +			     MPSCTRL_NON_SECURE_READ_BIT |
>> +			     MPSCTRL_NON_SECURE_WRITE_BIT | MPSCTRL_VALID);
>> +	}
>> +}
>> +
>> /*
>>  * This function adds the mmc channel to be registered with mmc core.
>>  * index -	mmc channel number.
>> @@ -65,6 +78,7 @@ int exynos_dwmci_add_port(int index, u32 regbase, int bus_width, u32 clksel)
>> #ifdef CONFIG_EXYNOS5420
>> 	host->quirks = DWMCI_QUIRK_DISABLE_SMU;
>> #endif
>> +	host->board_init = exynos_dwmci_board_init;
>>
>> 	if (clksel) {
>> 		host->clksel_val = clksel;
>> diff --git a/include/dwmmc.h b/include/dwmmc.h
>> index 6c91143..a02dd67 100644
>> --- a/include/dwmmc.h
>> +++ b/include/dwmmc.h
>> @@ -141,6 +141,7 @@ struct dwmci_host {
>> 	struct mmc *mmc;
>>
>> 	void (*clksel)(struct dwmci_host *host);
>> +	void (*board_init)(struct dwmci_host *host);
>> 	unsigned int (*get_mmc_clk)(int dev_index);
>> };
>>
>> -- 
>> 1.7.9.5
> 
> Acked-by: Pantelis Antoniou <panto at antoniou-consulting.com>
> 

Acked-by: Minkyu Kang <mk7.kang at samsung.com>

Thanks,
Minkyu Kang.