From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] WM97xx AC97 codec controls Date: Thu, 28 Jul 2005 18:02:01 +0200 Message-ID: References: <1121769565.6237.32.camel@cearnarfon> <1121855229.14604.44.camel@cearnarfon> <1122565385.5371.55.camel@localhost.localdomain> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: <1122565385.5371.55.camel@localhost.localdomain> 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: Liam Girdwood Cc: Thierry Vignaud , alsa-devel@lists.sourceforge.net List-Id: alsa-devel@alsa-project.org At Thu, 28 Jul 2005 16:43:05 +0100, Liam Girdwood wrote: > > I've now made the suggested changes and have used the suffix naming > convention for the controls. I've also added AC97_HAS_NO_STD_PCM, since > the wm9713 uses this register for another purpose and replaced some > double mono controls with stereo equivalents. Thanks! Still I see minor problems: - The standard control name for PCM volume is "PCM Playback Volume". Ditto for Front, Rear, etc. But I don't think it's necessary for the controls like "Aux To Headphone" - the direction is not obvious in such a case. Similarly, PCM switch should be "PCM Playback Switch". Also, you don't have to be too nervous about "Switch" suffix. It's mandatory only if the volume control with same component exists, or if its name is not obvious as a switch. - PCM switch is supposed to be reversed from mute switch. If it's 1, PCM is turned ON. Thus the last argument (invert) of AC97_SINGLE() should be 0. Care to fix the above? thanks, Takashi ------------------------------------------------------- SF.Net email is Sponsored by the Better Software Conference & EXPO September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf