From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugtrack@alsa-project.org Subject: [ALSA - driver 0001588]: alsa1.0.10 driver compilation error in adriver.h Date: Thu, 24 Nov 2005 23:43:52 +0100 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from bugtrack.alsa-project.org (gate.perex.cz [85.132.177.35]) by alsa.jcu.cz (ALSA's E-mail Delivery System) with ESMTP id 535A01C1 for ; Thu, 24 Nov 2005 23:43:52 +0100 (MET) 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: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org A NOTE has been added to this issue. ====================================================================== ====================================================================== Reported By: alanmann Assigned To: perex ====================================================================== Project: ALSA - driver Issue ID: 1588 Category: 0_compilation problem_!!! Reproducibility: always Severity: major Priority: normal Status: assigned Distribution: Kernel Version: 2.4.99 ====================================================================== Date Submitted: 11-22-2005 15:30 CET Last Modified: 11-24-2005 23:43 CET ====================================================================== Summary: alsa1.0.10 driver compilation error in adriver.h Description: Mainstream alsa1.0.10 downloaded this morning. Make fails in adriver.h lines 675-713 (added since 1.0.9b); the syntax is wrong? Has this already been noticed and fixed? The section is below:- static inline unsigned int jiffies_to_msecs(const unsigned long j) { #ifndef HAVE_VARIABLE_HZ #if HZ <= 1000 && !(1000 % HZ) return (1000 / HZ) * j; #elif HZ > 1000 && !(HZ % 1000) return (j + (HZ / 1000) - 1)/(HZ / 1000); #else return (j * 1000) / HZ; #endif #else if (HZ <= 1000 && !(1000 % HZ)) return (1000 / HZ) * j; else if (HZ > 1000 && !(HZ % 1000)) return (j + (HZ / 1000) - 1)/(HZ / 1000); else return (j * 1000) / HZ; #endif } static inline unsigned long msecs_to_jiffies(const unsigned int m) { if (m > jiffies_to_msecs(MAX_JIFFY_OFFSET)) return MAX_JIFFY_OFFSET; #ifndef HAVE_VARIABLE_HZ #if HZ <= 1000 && !(1000 % HZ) return (m + (1000 / HZ) - 1) / (1000 / HZ); #elif HZ > 1000 && !(HZ % 1000) return m * (HZ / 1000); #else return (m * HZ + 999) / 1000; #endif #else if (HZ <= 1000 && !(1000 % HZ)) return (m + (1000 / HZ) - 1) / (1000 / HZ); else if (HZ > 1000 && !(HZ % 1000)) return m * (HZ / 1000); else return (m * HZ + 999) / 1000; #endif ====================================================================== ---------------------------------------------------------------------- tiwai - 11-23-05 17:20 ---------------------------------------------------------------------- The older version caused errors on other kernels without variable HZ. The uploaded patch should fix the problem. ---------------------------------------------------------------------- alanmann - 11-24-05 23:43 ---------------------------------------------------------------------- The patch seemed to do the trick, thanks. Compile successful. Issue History Date Modified Username Field Change ====================================================================== 11-22-05 15:30 alanmann New Issue 11-22-05 15:30 alanmann Kernel Version => 2.4.99 11-22-05 17:28 Clemens LadischNote Added: 0006788 11-22-05 18:06 alanmann Note Added: 0006789 11-22-05 18:08 alanmann Note Edited: 0006789 11-22-05 20:01 alanmann Note Edited: 0006789 11-23-05 08:15 Clemens LadischNote Added: 0006808 11-23-05 12:52 alanmann Note Added: 0006809 11-23-05 17:20 tiwai Note Added: 0006812 11-23-05 17:20 tiwai File Added: suse-2.4-fix.diff 11-24-05 23:43 alanmann Note Added: 0006846 ====================================================================== ------------------------------------------------------- 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