From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pierre-Louis Bossart Subject: Re: [RFC 01/10] ASoC: Intel: Boards: Machine driver for Intel platforms Date: Thu, 7 Dec 2017 10:33:51 -0600 Message-ID: <0e9da474-c259-65a5-e217-e8b7d3042694@linux.intel.com> References: <1512119648-2700-1-git-send-email-rakesh.a.ughreja@intel.com> <1512119648-2700-2-git-send-email-rakesh.a.ughreja@intel.com> <67a5002e-9671-f6cd-f37b-e8c1b6d630ce@linux.intel.com> <85DFEED57DC57344B2483EF7BF8CB60579AC5C24@BGSMSX104.gar.corp.intel.com> <85DFEED57DC57344B2483EF7BF8CB60579AC5D64@BGSMSX104.gar.corp.intel.com> <20171206161755.GD32417@localhost> <85DFEED57DC57344B2483EF7BF8CB60579AC9A2A@BGSMSX104.gar.corp.intel.com> <7a1aabda-d883-d644-5476-81f73cbd1509@linux.intel.com> <85DFEED57DC57344B2483EF7BF8CB60579AC9E12@BGSMSX104.gar.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by alsa0.perex.cz (Postfix) with ESMTP id 478AA266B67 for ; Thu, 7 Dec 2017 17:33:55 +0100 (CET) In-Reply-To: <85DFEED57DC57344B2483EF7BF8CB60579AC9E12@BGSMSX104.gar.corp.intel.com> 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: "Ughreja, Rakesh A" , "Koul, Vinod" Cc: "tiwai@suse.de" , "liam.r.girdwood@linux.intel.com" , "alsa-devel@alsa-project.org" , "broonie@kernel.org" , Patches Audio List-Id: alsa-devel@alsa-project.org On 12/7/17 9:21 AM, Ughreja, Rakesh A wrote: > > >> -----Original Message----- >> From: Pierre-Louis Bossart [mailto:pierre-louis.bossart@linux.intel.com] >> Sent: Thursday, December 7, 2017 6:36 PM >> To: Ughreja, Rakesh A ; Koul, Vinod >> >> Cc: alsa-devel@alsa-project.org; broonie@kernel.org; tiwai@suse.de; >> liam.r.girdwood@linux.intel.com; Patches Audio >> Subject: Re: [alsa-devel] [RFC 01/10] ASoC: Intel: Boards: Machine driver for Intel >> platforms >> >> On 12/7/17 6:27 AM, Ughreja, Rakesh A wrote: >>> >>> >>>> -----Original Message----- >>>> From: Koul, Vinod >>>> Sent: Wednesday, December 6, 2017 9:48 PM >>>> To: Pierre-Louis Bossart >>>> Cc: Ughreja, Rakesh A ; alsa-devel@alsa- >> project.org; >>>> broonie@kernel.org; tiwai@suse.de; liam.r.girdwood@linux.intel.com; Patches >>>> Audio >>>> Subject: Re: [alsa-devel] [RFC 01/10] ASoC: Intel: Boards: Machine driver for >> Intel >>>> platforms >>>> >>>> On Mon, Dec 04, 2017 at 09:37:37AM -0600, Pierre-Louis Bossart wrote: >>>>> On 12/4/17 9:10 AM, Ughreja, Rakesh A wrote: >>>> >>>>>>>>>> sound/soc/intel/boards/Kconfig | 10 ++ >>>>>>>>>> sound/soc/intel/boards/Makefile | 2 + >>>>>>>>>> sound/soc/intel/boards/skl_hda_generic.c | 276 >>>>>>>>> +++++++++++++++++++++++++++++++ >>>>>>>>> >>>>>>>>> can we drop the Skylake reference? It's become a catch-all term to >> mean >>>>>>>>> both the platform, the IP and the driver. >>>>>>>> >>>>>>>> Suggest some name. I have no problem. >>>>>>> >>>>>>> HiFi3 ? >>>>>>> iDisp ? >>>>>>> HDAudio-DSP ? >>>>>> >>>>>> hda_dsp_generic.c -- For the main file >>>>>> hda_dsp_common.c -- for common functions >>>>>> >>>>>> Does it look fine ? >>>>> >>>>> works for me. >>>> >>>> Sorry not for me. hda_dsp_xxx doesnt tie it to anything. HDA and DSP are too >>>> generic terms. But yes I don't have a better alternate than skl_generic. >>>> Here this solution is tied to a very specfic IP which is present in SKL >>>> onwards platforms.. >>>> >>>> Yes SKL is become an IP as well as platform. Maybe we should have a >> codename >>>> for this like azx :) >>> >>> We do have a code name "sst". >>> >>> So does this sound okay for you ? >>> >>> sst_hda_dsp_generic.c -- For main file >>> sst_hda_dsp_common.c -- for common functions >> >> SST is also aliased to platform drivers with closed-source firmware, not >> a good idea for machine drivers. > > Then I have to go back to skl_hda_dsp_generic.c and > skl_hda_dsp_common.c as Vinod suggested. Fine for now. you will have to do this work of allowing for more granularity in the 'SKL' platform driver in the near future, to e.g. use it for the Skylake and Kabylake platforms and not for APL, CNL and followups, so we'll have to have this naming context again for the second coat of paint on the bikeshed.