From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH v2 1/2] ASoC: soc-compress: add a config item for soc-compress Date: Tue, 16 Jun 2015 18:17:12 +0200 Message-ID: References: <1434338445-31992-1-git-send-email-yang.jie@intel.com> <20150615150556.GQ18309@sirena.org.uk> <20150616105429.GI18309@sirena.org.uk> <20150616160607.GT18309@sirena.org.uk> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 4FC2426585A for ; Tue, 16 Jun 2015 18:17:12 +0200 (CEST) In-Reply-To: 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: "ramesh.babu@linux.intel.com" , "alsa-devel@alsa-project.org" , "Koul, Vinod" , "Zhang, Vivian" , "Girdwood, Liam R" List-Id: alsa-devel@alsa-project.org At Tue, 16 Jun 2015 18:14:40 +0200, Takashi Iwai wrote: > > At Tue, 16 Jun 2015 17:06:07 +0100, > Mark Brown wrote: > > > > On Tue, Jun 16, 2015 at 12:36:16PM +0000, Jie, Yang wrote: > > > > > > OK, that should have been in the commit message. > > > > > OK, so let me add it to the commit message and resend? > > > > Well, there's still the stubs to consider. Should we really be > > returning an error or should we silently ignore the error and hide the > > DAI if the user deconfigured compressed audio? Or rearrange things so > > we don't need stubs? Given that the machine driver has to select > > compressed support it's not something that should ever really hit the > > stubs, it's not truly a runtime thing... > > Yes, I guess that leaving without dummy function will give unresolved > symbol errors at module link time, so the user can catch before > actually running it. Of course, this should be checked actually. Oh, scratch this. It won't work well in the current code. Possibly with weak linking, but... Takashi