From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH 06/10] ALSA: Add firmware caching to CA0132 codec. Date: Mon, 08 Oct 2012 10:07:14 +0200 Message-ID: References: <1348198161-9256-1-git-send-email-ian_minett@creativelabs.com> <1348198161-9256-7-git-send-email-ian_minett@creativelabs.com> <20120925112634.GB32685@sirena.org.uk> <20121008080059.GA10604@opensource.wolfsonmicro.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 52D9426167A for ; Mon, 8 Oct 2012 10:07:14 +0200 (CEST) In-Reply-To: <20121008080059.GA10604@opensource.wolfsonmicro.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: Mark Brown Cc: alsa-devel@alsa-project.org, Ian Minett List-Id: alsa-devel@alsa-project.org At Mon, 8 Oct 2012 09:01:00 +0100, Mark Brown wrote: > > On Mon, Oct 08, 2012 at 09:49:57AM +0200, Takashi Iwai wrote: > > Mark Brown wrote: > > > > More generally, why is this driver different to other Linux drivers > > > which don't normally cache the firmware? > > > They usually do, if it's needed in the normal resume path. > > It's not universally agreed that this is a wonderful idea, though... Yeah, the situation around the firmware loader (still) sucks. It's getting better slowly for now, but a few big steps are still needed, e.g. for resolving the call of request_firmware() in the module (or built-in) init code. Takashi