Re: [PATCH v3 1/6] mfd: arizona: Add TST_CAP bits for headphone detection

From: Chanwoo Choi
Date: Tue Sep 15 2015 - 00:05:39 EST


Hi Charles,

Your following patch-set[1][2] receive the ack message about almost patches.
But, a few patches[3][4][5] must need the ack message from each maintainer.
[1] http://www.spinics.net/lists/devicetree/msg93383.html
[2] https://lkml.org/lkml/2015/8/11/145
[3] patch1 is included in patch-set[1]
: [PATCH v3 1/6] mfd: arizona: Add TST_CAP bits for headphone detection
[4] patch6 is included in patch-set[1]
: [PATCH v2 6/6] mfd: arizona: Update DT binding documentation for jack detection
[5] patch5 is included in patch-set[2]
[PATCH RESEND 5/5] mfd: arizona: Update DT doc for new mic detection bindings

So, if all patches receive the ack message, I'll create the immutable branch
for both EXTCON and MFD. And then I'll send the pull request to the Lee Jones(MFD maintainer)

Thanks,
Chanwoo Choi

On 2015ë 09ì 11ì 21:37, Charles Keepax wrote:
> On Florida some additional settings are required to get accurate
> measurements at the top end of the headphone detection range. This patch
> adds the bits required for this.
>
> Signed-off-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> ---
>
> Changes since v2:
> - Removed accidental addition of Chanwoo's ack.
>
> Thanks,
> Charles
>
> drivers/mfd/wm5110-tables.c | 2 ++
> include/linux/mfd/arizona/registers.h | 8 ++++++++
> 2 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mfd/wm5110-tables.c b/drivers/mfd/wm5110-tables.c
> index acb3bb2..78032e8 100644
> --- a/drivers/mfd/wm5110-tables.c
> +++ b/drivers/mfd/wm5110-tables.c
> @@ -1908,6 +1908,7 @@ static bool wm5110_readable_register(struct device *dev, unsigned int reg)
> case ARIZONA_HP1_SHORT_CIRCUIT_CTRL:
> case ARIZONA_HP2_SHORT_CIRCUIT_CTRL:
> case ARIZONA_HP3_SHORT_CIRCUIT_CTRL:
> + case ARIZONA_HP_TEST_CTRL_1:
> case ARIZONA_AIF1_BCLK_CTRL:
> case ARIZONA_AIF1_TX_PIN_CTRL:
> case ARIZONA_AIF1_RX_PIN_CTRL:
> @@ -2853,6 +2854,7 @@ static bool wm5110_volatile_register(struct device *dev, unsigned int reg)
> case ARIZONA_INPUT_ENABLES_STATUS:
> case ARIZONA_OUTPUT_STATUS_1:
> case ARIZONA_RAW_OUTPUT_STATUS_1:
> + case ARIZONA_HP_TEST_CTRL_1:
> case ARIZONA_SLIMBUS_RX_PORT_STATUS:
> case ARIZONA_SLIMBUS_TX_PORT_STATUS:
> case ARIZONA_INTERRUPT_STATUS_1:
> diff --git a/include/linux/mfd/arizona/registers.h b/include/linux/mfd/arizona/registers.h
> index e96644c..fe1b5d0 100644
> --- a/include/linux/mfd/arizona/registers.h
> +++ b/include/linux/mfd/arizona/registers.h
> @@ -237,6 +237,7 @@
> #define ARIZONA_HP1_SHORT_CIRCUIT_CTRL 0x4A0
> #define ARIZONA_HP2_SHORT_CIRCUIT_CTRL 0x4A1
> #define ARIZONA_HP3_SHORT_CIRCUIT_CTRL 0x4A2
> +#define ARIZONA_HP_TEST_CTRL_1 0x4A4
> #define ARIZONA_SPK_CTRL_2 0x4B5
> #define ARIZONA_SPK_CTRL_3 0x4B6
> #define ARIZONA_DAC_COMP_1 0x4DC
> @@ -3548,6 +3549,13 @@
> #define ARIZONA_HP3_SC_ENA_WIDTH 1 /* HP3_SC_ENA */
>
> /*
> + * R1188 (0x4A4) HP Test Ctrl 1
> + */
> +#define ARIZONA_HP1_TST_CAP_SEL_MASK 0x0003 /* HP1_TST_CAP_SEL - [1:0] */
> +#define ARIZONA_HP1_TST_CAP_SEL_SHIFT 0 /* HP1_TST_CAP_SEL - [1:0] */
> +#define ARIZONA_HP1_TST_CAP_SEL_WIDTH 2 /* HP1_TST_CAP_SEL - [1:0] */
> +
> +/*
> * R1244 (0x4DC) - DAC comp 1
> */
> #define ARIZONA_OUT_COMP_COEFF_MASK 0xFFFF /* OUT_COMP_COEFF - [15:0] */
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/