From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Ujfalusi Subject: Re: [PATCH 04/11] MFD: twl4030-audio: Add DT support Date: Wed, 08 Aug 2012 17:16:58 +0300 Message-ID: <502274DA.9020204@ti.com> References: <1344418887-5262-1-git-send-email-peter.ujfalusi@ti.com> <1344418887-5262-5-git-send-email-peter.ujfalusi@ti.com> <20120808131356.GS16861@opensource.wolfsonmicro.com> <50226CF4.1010202@ti.com> <20120808135253.GC16861@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20120808135253.GC16861@opensource.wolfsonmicro.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Mark Brown Cc: alsa-devel@alsa-project.org, Samuel Ortiz , Tony Lindgren , Dmitry Torokhov , devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org, linux-omap@vger.kernel.org, Liam Girdwood , linux-arm-kernel@lists.infradead.org, Benoit Cousson List-Id: linux-input@vger.kernel.org On 08/08/2012 04:52 PM, Mark Brown wrote: >> The hs_extmute is used for reducing pop noise on the headset. The sequen= ce is >> described in the TRM and it need to be done within the codec driver sinc= e it >> is within the sequence. > = > Is the point it's at in the sequence the end? On both ends :o On HS enable: the extmute has to be enabled, run the power on sequence for = the HS, wait for ramp, disable the HS extmute. On stop: enable the extmute, run the sequence, wait for ramp, release the e= xtmute. > We can always add a patch stubbing it out as part of the series, or in a > branch in the OF tree that we can pull in. If you are OK with the hassle I can add a patch to the series for include/linux/of.h to handle this. -- = P=E9ter