Hi Charles, On 02/17/2015 01:48 AM, Chanwoo Choi wrote: > Hi Charles, > > > On Tue, Feb 17, 2015 at 12:41 AM, Charles Keepax > <[email protected]> wrote: >> arizona_extcon_do_magic does not lend a lot of clarity to the purpose >> of the function, and as all the registers used are described in the >> datasheet there is no need to obfuscate the code. This patch renames the >> function to arizona_extcon_hp_clamp, as it controls clamping on the >> headphone output. >> >> Signed-off-by: Charles Keepax <[email protected]> >> --- >> drivers/extcon/extcon-arizona.c | 36 ++++++++++++++++++++---------------- >> include/linux/mfd/arizona/core.h | 2 +- >> sound/soc/codecs/arizona.c | 4 ++-- >> 3 files changed, 23 insertions(+), 19 deletions(-) > > > Looks good to me. > But, you need more ack message by MFD maintainer. > > For extcon part: > Acked-by: Chanwoo Choi <[email protected]>
Applied it. Thanks, Chanwoo Choi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
