linux/drivers/extcon, branch v5.6

extcon: sm5502: Remove unneeded semicolon

2019-12-16T01:23:19Z

Remove unneeded semicolon reported by coccinelle. Signed-off-by: Xu Wang [cw00.choi: Edit patch title and description] Signed-off-by: Chanwoo Choi

extcon: arizona: Factor out microphone and button detection

2019-12-09T11:18:15Z

Continue refactoring the microphone detect handling by factoring out the handling for microphone detection and button detection into separate functions. This both makes the code a little clearer and prepares for some planned future refactoring to make the state handling in the driver more explicit. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Factor out microphone impedance into a function

2019-12-09T11:18:15Z

The microphone detection handler is very long, start breaking it up by factoring out the actual reading of the impedance value into a separate functions. Additionally, this also fixes a minor bug and ensures that the microphone timeout will be rescheduled in all error cases. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Invert logic of check in arizona_hpdet_do_id

2019-12-09T11:18:15Z

Invert the check of hpdet_acc_id at the top of arizona_hpdet_do_id to reduce the identation within the function. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Remove excessive WARN_ON

2019-12-09T11:18:14Z

A WARN_ON is very strong for simply finding a button that is out of range, downgrade this to a simple error message in the log. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Remove unnecessary sets of ACCDET_MODE

2019-12-09T11:18:14Z

arizona_start_mic sets ACCDET_MODE as required for the microphone detection as such it is redundant to set this outside of this function. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Tidy up transition from mic to headphone detect

2019-12-09T11:18:14Z

Moving from microphone detection to headphone detection is done fairly haphazardly at the moment, sometimes calling arizona_stop_mic at the call site sometimes relying on a call inside arizona_identify_headphone. Simplify all this and always call arizona_stop_mic at the top of arizona_identify_headphone. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Clear jack status regardless of detection type

2019-12-09T11:18:14Z

It makes sense to clear the internal state of the jack detection regardless of if the headphone detect based accessory detection or the normal microphone detect based flow is used. No issues are currently known because of this but the change makes more logical sense and eases future refactoring of the code. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Move pdata extraction to probe

2019-12-09T11:18:14Z

It makes no sense to be extracting values from pdata for the first time in the jack detection handler function, move this to probe time where it belongs. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi

extcon: arizona: Make rev A register sequences atomic

2019-12-09T11:18:14Z

The special register sequences that are applied for rev A of wm5102 should be applied atomically with respect to any other register writes. Use regmap_multi_reg_write to ensure all writes happen under the regmap lock. Signed-off-by: Charles Keepax Signed-off-by: Chanwoo Choi