diff mbox

[-next] mmc: wmt-sdmmc: remove unused variable in wmt_complete_data_request()

Message ID CAPgLHd-7nVD3=83wpNtw1cBmkr+xs5iD7tuxZM+PdxWPLtwZsQ@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Wei Yongjun March 11, 2013, 2:36 p.m. UTC
From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>

The variable 'mmc' is initialized but never used
otherwise, so remove the unused variable.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
---
 drivers/mmc/host/wmt-sdmmc.c | 2 --
 1 file changed, 2 deletions(-)



--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Tony Prisk March 12, 2013, 4:03 a.m. UTC | #1
On Mon, 2013-03-11 at 22:36 +0800, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
> 
> The variable 'mmc' is initialized but never used
> otherwise, so remove the unused variable.
> 
> Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
> ---
>  drivers/mmc/host/wmt-sdmmc.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
> index 154f0e8..41d5802 100644
> --- a/drivers/mmc/host/wmt-sdmmc.c
> +++ b/drivers/mmc/host/wmt-sdmmc.c
> @@ -348,13 +348,11 @@ static void wmt_complete_data_request(struct wmt_mci_priv *priv)
>  
>  static irqreturn_t wmt_mci_dma_isr(int irq_num, void *data)
>  {
> -	struct mmc_host *mmc;
>  	struct wmt_mci_priv *priv;
>  
>  	int status;
>  
>  	priv = (struct wmt_mci_priv *)data;
> -	mmc = priv->mmc;
>  
>  	status = readl(priv->sdmmc_base + SDDMA_CCR) & 0x0F;
>  
> 

Acked-by: Tony Prisk <linux@prisktech.co.nz>

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Chris Ball March 22, 2013, 4:38 p.m. UTC | #2
Hi,

On Tue, Mar 12 2013, Tony Prisk wrote:
> On Mon, 2013-03-11 at 22:36 +0800, Wei Yongjun wrote:
>> From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
>> 
>> The variable 'mmc' is initialized but never used
>> otherwise, so remove the unused variable.
>> 
>> Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
>> ---
>>  drivers/mmc/host/wmt-sdmmc.c | 2 --
>>  1 file changed, 2 deletions(-)
>> 
>> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
>> index 154f0e8..41d5802 100644
>> --- a/drivers/mmc/host/wmt-sdmmc.c
>> +++ b/drivers/mmc/host/wmt-sdmmc.c
>> @@ -348,13 +348,11 @@ static void wmt_complete_data_request(struct wmt_mci_priv *priv)
>>  
>>  static irqreturn_t wmt_mci_dma_isr(int irq_num, void *data)
>>  {
>> -	struct mmc_host *mmc;
>>  	struct wmt_mci_priv *priv;
>>  
>>  	int status;
>>  
>>  	priv = (struct wmt_mci_priv *)data;
>> -	mmc = priv->mmc;
>>  
>>  	status = readl(priv->sdmmc_base + SDDMA_CCR) & 0x0F;
>>  
>> 
>
> Acked-by: Tony Prisk <linux@prisktech.co.nz>

Thanks, pushed to mmc-next for 3.10.

- Chris.
diff mbox

Patch

diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
index 154f0e8..41d5802 100644
--- a/drivers/mmc/host/wmt-sdmmc.c
+++ b/drivers/mmc/host/wmt-sdmmc.c
@@ -348,13 +348,11 @@  static void wmt_complete_data_request(struct wmt_mci_priv *priv)
 
 static irqreturn_t wmt_mci_dma_isr(int irq_num, void *data)
 {
-	struct mmc_host *mmc;
 	struct wmt_mci_priv *priv;
 
 	int status;
 
 	priv = (struct wmt_mci_priv *)data;
-	mmc = priv->mmc;
 
 	status = readl(priv->sdmmc_base + SDDMA_CCR) & 0x0F;