From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751954AbaB1Gno (ORCPT ); Fri, 28 Feb 2014 01:43:44 -0500 Received: from mailhost.informatik.uni-hamburg.de ([134.100.9.70]:62900 "EHLO mailhost.informatik.uni-hamburg.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750750AbaB1Gnm (ORCPT ); Fri, 28 Feb 2014 01:43:42 -0500 Message-ID: <531030C7.2060703@metafoo.de> Date: Fri, 28 Feb 2014 07:46:31 +0100 From: Lars-Peter Clausen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20131103 Icedove/17.0.10 MIME-Version: 1.0 To: "Li.Xiubo@freescale.com" CC: Mark Brown , "alsa-devel@alsa-project.org" , "shawn.guo@linaro.org" , "linux-kernel@vger.kernel.org" Subject: Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. References: <1393494593-26310-1-git-send-email-Li.Xiubo@freescale.com> <1393494593-26310-3-git-send-email-Li.Xiubo@freescale.com> <20140228035244.GJ9383@sirena.org.uk> <20140228052904.GP9383@sirena.org.uk> <1b89016263c84e0297655034a648032f@BY2PR03MB505.namprd03.prod.outlook.com> <53102EE4.80702@metafoo.de> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/28/2014 07:40 AM, Li.Xiubo@freescale.com wrote: > >> Subject: Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try >> regmap. >> >> On 02/28/2014 06:56 AM, Li.Xiubo@freescale.com wrote: >>>> Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. >>>> >>>> On Fri, Feb 28, 2014 at 04:00:38AM +0000, Li.Xiubo@freescale.com wrote: >>>> >>>>> I'll send another patches to applied to use this for another CODEC drivers. >>>>> And there almost 80 files, Should I send them in one patch or split them >>>> into >>>>> individual patch for each CODEC driver ? >>>> >>>> I'd suggest doing one patch that covers the boring drivers where the >>>> first thing they do is call set_cache_io() but split out the others into >>>> one patch per driver since the need more examination. >>> >>> Got it. >> >> Btw. be careful, just removing the set_cache_io() call will not work for all >> drivers. There are some MFD child devices which use regmap from the parent >> device. So dev_get_regmap() will return NULL for those. >> > > @Lars, > > Do you mean the CODEC drivers like wm5110 and wm8997 ? > Yes.