Date: Wed, 29 Apr 2015 14:12:45 +0100
From: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
To: Lee Jones <lee.jones@linaro.org>
Cc: Inha Song <ideal.song@samsung.com>, devicetree@vger.kernel.org,
        alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
        patches@opensource.wolfsonmicro.com, cw00.choi@samsung.com,
        broonie@kernel.org, rf@opensource.wolfsonmicro.com, robh+dt@kernel.org,
        pawel.moll@arm.com, mark.rutland@arm.com,
        ijc+devicetree@hellion.org.uk
Subject: Re: [alsa-devel] [PATCH v2 2/2] mfd: arizona: Update DT binding to
	support hpdet channel
Message-ID: <20150429131245.GL3480@opensource.wolfsonmicro.com>
References: <1429589505-11954-1-git-send-email-ideal.song@samsung.com> <1429589505-11954-3-git-send-email-ideal.song@samsung.com> <20150429105737.GZ9169@x1>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20150429105737.GZ9169@x1>
User-Agent: Mutt/1.5.17+20080114 (2008-01-14)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1200
Lines: 30

On Wed, Apr 29, 2015 at 11:57:37AM +0100, Lee Jones wrote:
> Does this looks okay to you Charles?
> 
> > This patch add device tree bindings for the pdata needed to configure
> > the Accessory Detect Mode select when Headphone detection.
> 
> Isn't the idea of headphone detection that it headphones are detected
> i.e. automatically?
> 

Yeah, pretty much the chip can run an impedance measurement on
either the left or the right headphone pin and to complicate
things a little further you don't necessarily have to wire that
to the actual headphones in hardware. So people tend to wire
up all sorts of odd things that they thought sounded awesome. So
you do really need a way to inform the driver which pin is
actually going to be useful for measuring the headphone impedance
on a specific system.

In an ideal world we would just measure both and take the
average, but in practice it is very dependent on how the hardware
is wired up.

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