From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pierre-Louis Bossart Subject: Re: [PATCH] ASoC: Intel: Boards: Add CNL RT274 I2S machine driver Date: Thu, 17 May 2018 13:17:48 -0500 Message-ID: References: <20171127124437.8441-1-guneshwor.o.singh@intel.com> <5bdd77e1-fb83-0626-c377-a457294ca38d@linux.intel.com> <20171128162415.GV3187@localhost> <20180517063725.GM20254@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by alsa0.perex.cz (Postfix) with ESMTP id 87296266DF0 for ; Thu, 17 May 2018 20:17:51 +0200 (CEST) In-Reply-To: <20180517063725.GM20254@sirena.org.uk> Content-Language: en-US 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: Takashi Iwai , Liam Girdwood , ALSA ML , Guneshwor Singh , Patches Audio List-Id: alsa-devel@alsa-project.org On 5/17/18 1:37 AM, Mark Brown wrote: > On Tue, Nov 28, 2017 at 10:44:50AM -0600, Pierre-Louis Bossart wrote: >> On 11/28/2017 10:24 AM, Vinod Koul wrote: > >>> Since you are still working on the fixes, I guess this can sneak in before. >>> The machine drivers are typically low hanging in terms on complexity so we >>> can merge this first followed by the rework you are doing. I think that part >>> will still take a bit to stabilize and cleanup. > >> the update goes out this afternoon, working on it, does this patch really >> need to 'sneak in'? > > This seems to have got buried with all the Kconfig fixes and reworks? I > was expecting a resend based on those given this discussion but that > didn't seem to ever happen. We have a small set of corrections to enable SOF to use this machine driver (mainly to remove hard-coded routes that can be handled by topology). The plan as I understand it is to first have Liam provide the SOF core, and in a second step provide all the machine driver updates - including this one.