From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: fine sample rate control for RME hdsp 9632 Date: Fri, 09 Mar 2007 17:41:11 +0100 Message-ID: References: <557316.48905.qm@web35603.mail.mud.yahoo.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <557316.48905.qm@web35603.mail.mud.yahoo.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@lists.sourceforge.net Errors-To: alsa-devel-bounces@lists.sourceforge.net To: Julian Cable Cc: alsa-devel@lists.sourceforge.net List-Id: alsa-devel@alsa-project.org At Thu, 8 Mar 2007 12:33:56 -0800 (PST), Julian Cable wrote: > > Hi, > > I've added a control to the hdsp driver to control the dds register on the 9632. > This allows fine tuning of the sample rate as is supported by the windows driver. The > feature is implemented as an offset to the current sample rate, since most of the code > assumes only discrete sample rates are supported. So the sample rate will always be > selected and indicated as one of the standard supported sample rates but may actually > be a few % higher or lower. It seems to work very well. > > One possible application is to discipline the sample rate from the system clock or a > GPS receiver to provide a long term accurate precise frequency source without > expensive reference clocks. > > The patch is against alsa-driver-1.0.14rc3 Could you provide a patch as a unified diff (diff -u)? At a first glance, the change looks almost OK. One suggestion is to avoid special characters like parentesis in the control name. Also, please provide 'Signed-off-by: xxx' line, too. Thanks, Takashi ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV