From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Salazar Subject: Re: [es18xx.c PATCH] #1/4 for Zoom Video - resolve common vs chipset specific mixer controls Date: Sun, 15 Jan 2006 15:30:06 -0500 Message-ID: <43CAB0CE.2040602@justmyself.net> References: <43B00FF2.6030007@justmyself.net> <1135613234.8293.38.camel@mindpipe> <43B02A0E.20401@justmyself.net> <43C199C1.1050301@justmyself.net> <43C306AA.4030900@justmyself.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: alsa-devel-admin@lists.sourceforge.net Errors-To: alsa-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Takashi Iwai Cc: alsa-devel@lists.sourceforge.net List-Id: alsa-devel@alsa-project.org Takashi Iwai wrote: > >Basically, rewriting the static array and restoring it looks odd, so >I prefer having multiple kcontrol_new arrays. (Besides, you could use >simply NULL for the validity check.) From the readability viewpoint, >separate arrays are better, too. The most important point is that the >code flow is straightforward. > > >Takashi > > > OK, I've tried your version of mod1 and it seems to work fine. Also retested the patches for mods 2 & 3 ... no changes necessary. Revised mod4 patch to follow. ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click