From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liam Girdwood Subject: Re: [PATCH 1/7] ASoC: Intel: Make ADSP memory block allocation more generic Date: Mon, 27 Oct 2014 10:09:54 +0000 Message-ID: <1414404594.6917.7.camel@loki> References: <1413814012-5619-1-git-send-email-liam.r.girdwood@linux.intel.com> <20141021230224.GF2344@sirena.org.uk> <5449FE1B.9000201@linux.intel.com> <5449FF2A.9070200@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by alsa0.perex.cz (Postfix) with ESMTP id C1E87260859 for ; Mon, 27 Oct 2014 11:10:17 +0100 (CET) In-Reply-To: <5449FF2A.9070200@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: Jarkko Nikula Cc: alsa-devel@alsa-project.org, Mark Brown List-Id: alsa-devel@alsa-project.org On Fri, 2014-10-24 at 10:26 +0300, Jarkko Nikula wrote: > On 10/24/2014 10:22 AM, Jarkko Nikula wrote: > > Hi Liam > > > > On 10/22/2014 02:02 AM, Mark Brown wrote: > >> On Mon, Oct 20, 2014 at 03:06:46PM +0100, Liam Girdwood wrote: > >>> Current block allocation is tied to block type and requestor type. > >>> Make the > >>> allocation more generic by removing the struct module parameter and > >>> adding > >>> a generic block allocator structure. Also pass in the list that the > >>> blocks > >>> have to be added too in order to remove dependence on block > >>> requestor type. > >> I'm having trouble apply this - patch 1 manages to apply with some fuzz > >> but patch 5 is failing. Probably needs refreshing against Vinod's > >> recent changes? I applied everything up to patch 4 (ASoC: Intel: Add > >> runtime module support). > > > > For me sound.git for-next head a929d726a507 don't build anymore. I > > have to revert all your four recent patches cd51c82524ff -4. I guess > > some patch order or dependency issue since the first patch doesn't > > build when applied to topic/intel? > > > Ah, nevermind. Mark was dropped these already. I was building at my > local copy of for-next branch before patches were dropped. > Yeah, it looks like it was out of sync with Vinod's latest patches so I'll resend today now that I'm back from holidays. Liam