[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [gulliver] compilation / webcam / incompatible


From Thomas Petazzoni <thomas dot petazzoni at enix dot org>
Subject Re: [gulliver] compilation / webcam / incompatible
Date Tue, 2 Oct 2007 09:22:21 +0200

Salut,

Le Mon, 01 Oct 2007 20:04:52 +0200,
pascal coupel <pascal dot coupel at free dot fr> a écrit :

> Désolé je n'ai pas pensé qu'il fallait te la préciser :)
> C'est celle là:
> urb->complete = ov51x_isoc_irq;

La fonction ov51x_isoc_irq() a le prototype suivant dans le code du
driver:

 ov51x_isoc_irq(struct urb *urb, struct pt_regs *regs)

or, depuis le noyau 2.6.19, le paramètre "struct pt_regs *regs" a été
supprimé. Voir http://lwn.net/Articles/2.6-kernel-api/:

«
The prototype for interrupt handler functions has changed. In short,
the regs argument has been removed, since almost nobody used it. Any
interrupt handler which needs the pre-interrupt register state can use
get_irq_regs() to obtain it.
»

J'ai rapidement fait un patch pour corriger ce problème, il est joint à
ce mail. Pour l'appliquer, à la racine des sources du driver, tu fais:

 cat ov51x-jpeg-1.5.2-pt-regs-fix.patch | patch -p1

Bonne journée,

Thomas
-- 
Thomas Petazzoni - thomas dot petazzoni at enix dot org
http://{thomas,sos,kos}.enix.org - http://www.toulibre.org
http://www.{livret,agenda}dulibre.org
diff -ru ov51x-jpeg-1.5.2/ov51x-jpeg-core.c ov51x-jpeg-1.5.2.new/ov51x-jpeg-core.c
--- ov51x-jpeg-1.5.2/ov51x-jpeg-core.c  2007-07-21 14:25:00.000000000 +0200
+++ ov51x-jpeg-1.5.2.new/ov51x-jpeg-core.c      2007-10-02 09:19:53.000000000 +0200
@@ -5273,7 +5273,7 @@
 }
 
 static void
-ov51x_isoc_irq(struct urb *urb, struct pt_regs *regs)
+ov51x_isoc_irq(struct urb *urb)
 {
        int i;
        struct usb_ov511 *ov;