2015-04-30 14:51:41

by Charles Keepax

[permalink] [raw]
Subject: [PATCH] extcon: arizona: Apply HP clamps correctly for WM8280

The headphone clamping is not set correctly currently, this was missed
because the wm8280 patches and the patch fixing the clamping for wm5110
went upstream at very similar times. This patch sets the headphone
clamping correctly for wm8280.

Signed-off-by: Charles Keepax <[email protected]>
---
drivers/extcon/extcon-arizona.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/extcon/extcon-arizona.c b/drivers/extcon/extcon-arizona.c
index 830c5f9..0342ba0 100644
--- a/drivers/extcon/extcon-arizona.c
+++ b/drivers/extcon/extcon-arizona.c
@@ -145,6 +145,7 @@ static void arizona_extcon_hp_clamp(struct arizona_extcon_info *info,

switch (arizona->type) {
case WM5110:
+ case WM8280:
mask = ARIZONA_HP1L_SHRTO | ARIZONA_HP1L_FLWR |
ARIZONA_HP1L_SHRTI;
if (clamp)
--
1.7.2.5


2015-05-03 05:49:39

by Chanwoo Choi

[permalink] [raw]
Subject: Re: [PATCH] extcon: arizona: Apply HP clamps correctly for WM8280

Hi Charles,

On Thu, Apr 30, 2015 at 11:43 PM, Charles Keepax
<[email protected]> wrote:
> The headphone clamping is not set correctly currently, this was missed
> because the wm8280 patches and the patch fixing the clamping for wm5110
> went upstream at very similar times. This patch sets the headphone
> clamping correctly for wm8280.
>
> Signed-off-by: Charles Keepax <[email protected]>
> ---
> drivers/extcon/extcon-arizona.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>

Applied it.

Thanks,
Chanwoo Choi