diff mbox

[5/5] ALSA: mixart: Remove unused variable

Message ID 1404214378-24669-5-git-send-email-sachin.kamat@samsung.com (mailing list archive)
State Accepted
Commit 330fb10df832b0008892964601d9d142c14a906f
Delegated to: Takashi Iwai
Headers show

Commit Message

Sachin Kamat July 1, 2014, 11:32 a.m. UTC 
'err' is not used in the function. Remove it.

Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
---
 sound/pci/mixart/mixart_core.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Takashi Iwai July 1, 2014, 12:06 p.m. UTC | #1 
At Tue,  1 Jul 2014 17:02:58 +0530,
Sachin Kamat wrote:
> 
> 'err' is not used in the function. Remove it.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>

Applied, thanks.


Takashi

> ---
>  sound/pci/mixart/mixart_core.c |    4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/sound/pci/mixart/mixart_core.c b/sound/pci/mixart/mixart_core.c
> index 71f4bdcc4055..84f67450924e 100644
> --- a/sound/pci/mixart/mixart_core.c
> +++ b/sound/pci/mixart/mixart_core.c
> @@ -151,13 +151,11 @@ static int send_msg( struct mixart_mgr *mgr,
>  {
>  	u32 headptr, tailptr;
>  	u32 msg_frame_address;
> -	int err, i;
> +	int i;
>  
>  	if (snd_BUG_ON(msg->size % 4))
>  		return -EINVAL;
>  
> -	err = 0;
> -
>  	/* get message frame address */
>  	tailptr = readl_be(MIXART_MEM(mgr, MSG_INBOUND_FREE_TAIL));
>  	headptr = readl_be(MIXART_MEM(mgr, MSG_INBOUND_FREE_HEAD));
> -- 
> 1.7.9.5
>
diff mbox

Patch

diff --git a/sound/pci/mixart/mixart_core.c b/sound/pci/mixart/mixart_core.c
index 71f4bdcc4055..84f67450924e 100644
--- a/sound/pci/mixart/mixart_core.c
+++ b/sound/pci/mixart/mixart_core.c
@@ -151,13 +151,11 @@  static int send_msg( struct mixart_mgr *mgr,
 {
 	u32 headptr, tailptr;
 	u32 msg_frame_address;
-	int err, i;
+	int i;
 
 	if (snd_BUG_ON(msg->size % 4))
 		return -EINVAL;
 
-	err = 0;
-
 	/* get message frame address */
 	tailptr = readl_be(MIXART_MEM(mgr, MSG_INBOUND_FREE_TAIL));
 	headptr = readl_be(MIXART_MEM(mgr, MSG_INBOUND_FREE_HEAD));