Browse Source

extcon: arizona: Use DAPM mutex helper functions

We should be using the helper functions to lock the DAPM mutex not
accessing it directly. There are no ill effects of this as the moment
but it is best practice, and the implementation could be changed in the
future.

Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
Charles Keepax 9 years ago
parent
commit
03bf1adbd6
1 changed files with 2 additions and 2 deletions
  1. 2 2
      drivers/extcon/extcon-arizona.c

+ 2 - 2
drivers/extcon/extcon-arizona.c

@@ -185,7 +185,7 @@ static void arizona_extcon_hp_clamp(struct arizona_extcon_info *info,
 		break;
 		break;
 	};
 	};
 
 
-	mutex_lock(&arizona->dapm->card->dapm_mutex);
+	snd_soc_dapm_mutex_lock(arizona->dapm);
 
 
 	arizona->hpdet_clamp = clamp;
 	arizona->hpdet_clamp = clamp;
 
 
@@ -227,7 +227,7 @@ static void arizona_extcon_hp_clamp(struct arizona_extcon_info *info,
 				 ret);
 				 ret);
 	}
 	}
 
 
-	mutex_unlock(&arizona->dapm->card->dapm_mutex);
+	snd_soc_dapm_mutex_unlock(arizona->dapm);
 }
 }
 
 
 static void arizona_extcon_set_mode(struct arizona_extcon_info *info, int mode)
 static void arizona_extcon_set_mode(struct arizona_extcon_info *info, int mode)