Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753069Ab1CNL3a (ORCPT ); Mon, 14 Mar 2011 07:29:30 -0400 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:48433 "EHLO opensource2.wolfsonmicro.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750990Ab1CNL33 (ORCPT ); Mon, 14 Mar 2011 07:29:29 -0400 Date: Mon, 14 Mar 2011 11:29:47 +0000 From: Mark Brown To: "Cai, Cliff" Cc: Cliff Cai , Mike Frysinger , "alsa-devel@alsa-project.org" , "linux-kernel@vger.kernel.org" , "device-drivers-devel@blackfin.uclinux.org" , "akpm@linux-foundation.org" , "lrg@slimlogic.co.uk" Subject: Re: [alsa-devel] [Device-drivers-devel] [PATCH] Add driver for Analog Devices ADAU1701 SigmaDSP Message-ID: <20110314112946.GA2626@opensource.wolfsonmicro.com> References: <20110307121502.GG13471@opensource.wolfsonmicro.com> <20110309100055.GB6923@opensource.wolfsonmicro.com> <20110310134605.GC22195@opensource.wolfsonmicro.com> <20110311115344.GA1760@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Cookie: You love peace. User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 824 Lines: 16 On Sun, Mar 13, 2011 at 10:23:19PM -0400, Cai, Cliff wrote: > I think I need to have a check of the requirement from customer. > Anyway,if I modified the driver according to the comments here, > Would you like to accept it currently or we need to wait until the missing patch > Has been merged in mainline? Please repost, there were quite a few issues raised with this revision even outside the firmware. For the purpose of getting things into mainline I'd recommend splitting out the functionaltiy which allows firmware to be loaded dynamically at runtime as a second patch. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/