From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] ALSA: cs553*: Fix dependencies Date: Tue, 11 Mar 2014 10:38:05 +0100 Message-ID: References: <20140310211226.17ad03c6@endymion.delvare> <20140311103201.32b8d8da@endymion.delvare> 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 22DF8261609 for ; Tue, 11 Mar 2014 10:38:06 +0100 (CET) In-Reply-To: <20140311103201.32b8d8da@endymion.delvare> 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: Jean Delvare Cc: alsa-devel@alsa-project.org, Jaya Kumar List-Id: alsa-devel@alsa-project.org At Tue, 11 Mar 2014 10:32:01 +0100, Jean Delvare wrote: > > Hi Takashi, > > On Tue, 11 Mar 2014 07:54:25 +0100, Takashi Iwai wrote: > > At Mon, 10 Mar 2014 21:12:26 +0100, > > Jean Delvare wrote: > > > > > > The CS5530, CS5535 and CS5536 chipsets are companions of the Geode > > > series of processors, which are 32-bit x86 processors. So the > > > snd-cs5530 and snd-cs5535audio drivers are only needed on this > > > architecture, except for build testing purpose. > > > > > > Signed-off-by: Jean Delvare > > > Cc: Jaya Kumar > > > Cc: Jaroslav Kysela > > > Cc: Takashi Iwai > > > > Thanks, applied. > > Hmm, we might have been a bit too quick, sorry. Looking at the > defconfig files, it would seem that the CS5536 was also used on some > MIPS machines and the CS3350 was also used on some PowerPC machines: > > arch/mips/configs/lemote2f_defconfig:CONFIG_SND_CS5535AUDIO=m > arch/powerpc/configs/ppc6xx_defconfig:CONFIG_SND_CS5530=m > > So assuming these default configuration files are correct, the proper > dependencies would be somewhat different. I'll investigate and send an > updated or incremental patch, let me know what you prefer. Ah, I vaguely remember of MIPS stuff, indeed. I don't know of PPC, but it's no surprise even if there are such. So, yes, please send an incremental patch to cover these. thanks, Takashi