From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Courtier-Dutton Subject: Re: [alsa-cvslog] CVS: alsa-kernel/pci/emu10k1 emufx.c,1.59,1.60 Date: Tue, 09 Nov 2004 00:12:19 +0000 Message-ID: <41900B63.30308@superbug.co.uk> References: <418E654E.5000807@superbug.co.uk> 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: Jaroslav Kysela Cc: Jaroslav Kysela , Alsa-devel@lists.sourceforge.net List-Id: alsa-devel@alsa-project.org Jaroslav Kysela wrote: > On Sun, 7 Nov 2004, James Courtier-Dutton wrote: > > >>2) >> > + if (put_user(x, &icode->code[a + 0]) || >> > + put_user(y, &icode->code[a + 1])) >> > + return -EFAULT; >> >>Will always return -EFAULT on my system. I don't think that is what we >>really want. > > > Does this patch help? > I have just tried the CVS, that includes your fixes. Still returns -EFAULT. At least the kernel module loads, but, no DSP code is installed. James ------------------------------------------------------- This SF.Net email is sponsored by: Sybase ASE Linux Express Edition - download now for FREE LinuxWorld Reader's Choice Award Winner for best database on Linux. http://ads.osdn.com/?ad_id=5588&alloc_id=12065&op=click