From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932199Ab2ISP20 (ORCPT ); Wed, 19 Sep 2012 11:28:26 -0400 Received: from cantor2.suse.de ([195.135.220.15]:59845 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754790Ab2ISP2Y (ORCPT ); Wed, 19 Sep 2012 11:28:24 -0400 Date: Wed, 19 Sep 2012 17:28:21 +0200 Message-ID: From: Takashi Iwai To: Ondrej Zary Cc: alsa-devel@alsa-project.org, broonie@opensource.wolfsonmicro.com, linux-kernel@vger.kernel.org Subject: Re: [alsa-devel] [RFC PATCH v2 0/5] snd-ice1712: Add Philips PSC724 Ultimate Edge In-Reply-To: <1347994716-10251-1-git-send-email-linux@rainbow-software.org> References: <1347994716-10251-1-git-send-email-linux@rainbow-software.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.1 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Tue, 18 Sep 2012 20:58:31 +0200, Ondrej Zary wrote: > > > > Hello, > this is second attempt to add Philips PSC724 Ultimate Edge sound card > support to snd-ice1712 driver. Unlike other snd-ice1712 subdrivers, the codec > code is splitted into separate modules and thus reusable by other ice1712 > subdrivers. Well, the problem is that most of ice17xx boards use only a part of channels on codecs. Thus you don't want to expose controls for all channels, too. Otherwise user will get confusing dead controls. If these new modules are supposed to be used by other ice1712/ice1724 sub drivers, they should provide some good binding mechanism, too. (And that's one of the good reasons why ASoC has been nominated.) thanks, Takashi > Working: all analog outputs (front, rear, center+lfe, headphone) and inputs > (front mic, rear mic, line, cd, aux), mixer, headphone jack detection, > suspend/resume > > Currently untested: SPDIF > > Buggy: switching AGC mode in mixer causes alsamixer to crash - looks like > it does not like notifications about control activating/deactivating > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel >