From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH v2 0/9] Baytrail-Cherrytrail updates Date: Fri, 13 Oct 2017 19:23:24 +0300 Message-ID: <1507911804.16112.455.camel@linux.intel.com> References: <20171012233805.30055-1-pierre-louis.bossart@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by alsa0.perex.cz (Postfix) with ESMTP id 079E0266E90 for ; Fri, 13 Oct 2017 18:23:28 +0200 (CEST) In-Reply-To: <20171012233805.30055-1-pierre-louis.bossart@linux.intel.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: Pierre-Louis Bossart , alsa-devel@alsa-project.org Cc: tiwai@suse.de, vinod.koul@intel.com, broonie@kernel.org, liam.r.girdwood@linux.intel.com List-Id: alsa-devel@alsa-project.org On Thu, 2017-10-12 at 18:37 -0500, Pierre-Louis Bossart wrote: > As discussed on the alsa-devel mailing list, here is a set of patches > to fix > > 1. previous feedback on bytcht_rt5651 - no new functionality beyond > what was already reviewed > 2. cosmetic issues (variables in reverse xmas tree order, etc) > 3. unnecessary clock tests > 4. addition of a helper to remove duplicate code across machine > drivers > > The last two patches were not included in the first pass, > they correct an off-by-one issue which has no functional impact > and remove the hard-codec compressed dailinks which don't currently > work with the mainline code+publicly available firmware. > Patch 6 can be folded into 5, but it's up to you. Either choice: Reviewed-by: Andy Shevchenko > Pierre-Louis Bossart (9): > ASoC: Intel: bytcr_rt5651: add MCLK, quirks and cleanups > ASoC: Intel: bytcr_rt5640: cosmetic fixes > ASoC: Intel: cht_bsw_rt5645: cosmetic fixes > ASoC: Intel: bytcht_da7213: cosmetic fixes > ASoC: Intel: bytcht_es8316: remove useless code > ASoC: Add helper to find codec_dai from dai_name > ASoC: Intel: boards: use helper to get codec_dai > ASoC: Intel: boards: fix off-by-one dailink id > ASoC: Intel: boards: remove hard-coded compressed dailinks > > include/sound/soc.h | 14 ++ > sound/soc/intel/boards/bxt_da7219_max98357a.c | 16 +- > sound/soc/intel/boards/bytcht_da7213.c | 17 +- > sound/soc/intel/boards/bytcht_es8316.c | 25 +-- > sound/soc/intel/boards/bytcht_nocodec.c | 10 +- > sound/soc/intel/boards/bytcr_rt5640.c | 105 ++++------- > sound/soc/intel/boards/bytcr_rt5651.c | 236 > +++++++++++++++++++++--- > sound/soc/intel/boards/cht_bsw_max98090_ti.c | 24 +-- > sound/soc/intel/boards/cht_bsw_rt5645.c | 103 ++++------- > sound/soc/intel/boards/cht_bsw_rt5672.c | 24 +-- > sound/soc/intel/boards/skl_nau88l25_max98357a.c | 16 +- > sound/soc/intel/boards/skl_nau88l25_ssm4567.c | 16 +- > 12 files changed, 318 insertions(+), 288 deletions(-) > -- Andy Shevchenko Intel Finland Oy