Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752743AbZCJFa1 (ORCPT ); Tue, 10 Mar 2009 01:30:27 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751881AbZCJFaO (ORCPT ); Tue, 10 Mar 2009 01:30:14 -0400 Received: from india.einfochips.com ([203.88.139.151]:4730 "EHLO ahmedabad.einfochips.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751905AbZCJFaN (ORCPT ); Tue, 10 Mar 2009 01:30:13 -0400 Message-ID: <49B5F8F5.8040100@einfochips.com> Date: Tue, 10 Mar 2009 10:51:57 +0530 From: Viral Mehta Organization: eIbfochips Ltd. User-Agent: Thunderbird 2.0.0.19 (X11/20081209) MIME-Version: 1.0 To: linux-kernel@vger.kernel.org Subject: [PATCH] [SOUND] recording gain control Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2662 Lines: 61 Hi, I am trying to do gain control while RECORDING for my Audio Mic device. My application opens /dev/mixer device and calls ioctl(fdmixer, MIXER_WRITE(SOUND_MIXER_MIC), ...) But, the call fails. It traces to sound/core/oss/mixer_oss.c file and snd_mixer_oss_put_volume1() function. It never falls in "if (slot->present & SNDRV_MIXER_OSS_PRESENT_CVOLUME)" as condition "if (slot->present & SNDRV_MIXER_OSS_PRESENT_PVOLUME) " comes as false. I think there is a bug in kernel and I think it should be like as below, --- sound/core/oss/mixer_oss.c 2008-12-03 13:24:02.000000000 +0530 +++ sound/core/oss/mixer_oss.c 2009-03-09 16:22:06.548766896 +0530 @@ -688,7 +688,7 @@ static int snd_mixer_oss_put_volume1(str if (slot->present & SNDRV_MIXER_OSS_PRESENT_PVOLUME) { snd_mixer_oss_put_volume1_vol(fmixer, pslot, slot->numid[SNDRV_MIXER_OSS_ITEM_PVOLUME], left, right); - if (slot->present & SNDRV_MIXER_OSS_PRESENT_CVOLUME) + } else if (slot->present & SNDRV_MIXER_OSS_PRESENT_CVOLUME) { snd_mixer_oss_put_volume1_vol(fmixer, pslot, slot->numid[SNDRV_MIXER_OSS_ITEM_CVOLUME], left, right); } else if (slot->present & SNDRV_MIXER_OSS_PRESENT_GVOLUME) { snd_mixer_oss_put_volume1_vol(fmixer, pslot, slot->numid[SNDRV_MIXER_OSS_ITEM_GVOLUME], left, right); Thanks, Viral -- _____________________________________________________________________ Disclaimer: This e-mail message and all attachments transmitted with it are intended solely for the use of the addressee and may contain legally privileged and confidential information. If the reader of this message is not the intended recipient, or an employee or agent responsible for delivering this message to the intended recipient, you are hereby notified that any dissemination, distribution, copying, or other use of this message or its attachments is strictly prohibited. If you have received this message in error, please notify the sender immediately by replying to this message and please delete it from your computer. Any views expressed in this message are those of the individual sender unless otherwise stated.Company has taken enough precautions to prevent the spread of viruses. However the company accepts no liability for any damage caused by any virus transmitted by this email. __________________________________________________________________________ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/