Hi,
On 22-02-18 11:52, Mark Brown wrote:
On Wed, Feb 21, 2018 at 08:43:47PM +0100, Hans de Goede wrote:
On 21-02-18 12:40, Mark Brown wrote:
This is now only enabling the LDO and bias when the jack is inserted - is that enough?
Yes, we only need to do OVCD on an insertion event to differ between headset vs headphones.
Is there not a fault protection aspect to this? The naming and the separate control makes this seem like it's got some wider purpose.
That is a good question, my patch series ends up always enabling this because my testing has shown that it works better (*) when enabled before the supplies feeding it are enabled, and there is no harm in leaving it enabled while the supplies are disabled.
This will automatically also give us the fault protection when micbias1 is on because we're actually recording from the mic, rather then that it is forced on for a short while for the headset vs headphones detection.
So after this series we are good wrt its function for fault protection too. Note that the datasheet is not clear on if it is really necessary for this, but I think it makes sense to have this enabled at all times.
Regards,
Hans