From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933113AbdKOQob (ORCPT ); Wed, 15 Nov 2017 11:44:31 -0500 Received: from omzsmtpe02.verizonbusiness.com ([199.249.25.209]:31189 "EHLO omzsmtpe02.verizonbusiness.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933092AbdKOQoO (ORCPT ); Wed, 15 Nov 2017 11:44:14 -0500 From: alexander.levin@verizon.com X-Host: surveyor.tdc.vzwcorp.com To: Mark Brown CC: "linux-kernel@vger.kernel.org" , "stable@vger.kernel.org" , Adam Thomson Subject: Re: [PATCH AUTOSEL for 4.9 31/56] ASoC: da7218: Set DAI output pin high impedance when not in use Thread-Topic: [PATCH AUTOSEL for 4.9 31/56] ASoC: da7218: Set DAI output pin high impedance when not in use Thread-Index: AQHTXjDRFgbbjmWn70amyySLUiqwQw== Date: Wed, 15 Nov 2017 16:43:08 +0000 Message-ID: <20171115164304.23pieave2vze6wcq@sasha-lappy> References: <20171115024521.5884-1-alexander.levin@verizon.com> <20171115024521.5884-31-alexander.levin@verizon.com> <20171115121918.4zf7enwijbmsow6s@sirena.org.uk> In-Reply-To: <20171115121918.4zf7enwijbmsow6s@sirena.org.uk> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: NeoMutt/20170113 (1.7.2) x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.144.60.250] Content-Type: text/plain; charset="us-ascii" Content-ID: <54C9F2CE956C24459C1602CD891D7997@vzwcorp.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id vAFGiad7009330 On Wed, Nov 15, 2017 at 12:19:18PM +0000, Mark Brown wrote: >On Wed, Nov 15, 2017 at 02:45:42AM +0000, alexander.levin@verizon.com wrote: > >> This patch updates the AIF_OUT widget to set the DAI output pin of >> the device as high impedance when not in use. > >This will leave the line floating which EEs don't always like. It's >probably OK and hasn't lead to any reports in mainline yet but still... The idea I got from the patch is that output will not be working if there are multiple codecs? -- Thanks, Sasha