Re: linux-next: ttydev tree build failure

From: Aleksey Gorelov <hidden>
Date: 2008-07-18 21:41:39


----- Original Message ----
quoted hunk
From: Stephen Rothwell <redacted>
To: Alan Cox <redacted>
Cc: linux-next@vger.kernel.org; Greg KH <redacted>; Aleksey Gorelov <redacted>
Sent: Friday, July 18, 2008 1:59:45 AM
Subject: linux-next: ttydev tree build failure

Hi Alan,

Today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/usb/serial/usb_debug.c: In function 'usb_debug_open':
drivers/usb/serial/usb_debug.c:37: warning: passing argument 1 of 
'usb_serial_generic_open' from incompatible pointer type
drivers/usb/serial/usb_debug.c:37: warning: passing argument 2 of 
'usb_serial_generic_open' from incompatible pointer type
drivers/usb/serial/usb_debug.c:37: error: too few arguments to function 
'usb_serial_generic_open'
drivers/usb/serial/usb_debug.c: At top level:
drivers/usb/serial/usb_debug.c:47: warning: initialization from incompatible 
pointer type

usb_debug_open was added in commit
1d051a41474c386498bc37e3ae89f7937222cfe7 ("USB: debug port converter does
not accept more than 8 byte packets") from the usb tree.

I applied the patch below and will do so until it can be resolved.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

From 36676f026d56652cf4d42c13651373dc6a232141 Mon Sep 17 00:00:00 2001
From: Stephen Rothwell 
Date: Fri, 18 Jul 2008 18:54:09 +1000
Subject: [PATCH] usb: tty API change fallout

Signed-off-by: Stephen Rothwell 
---
drivers/usb/serial/usb_debug.c |    4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/serial/usb_debug.c b/drivers/usb/serial/usb_debug.c
index 9ca4d4d..06a272b 100644
--- a/drivers/usb/serial/usb_debug.c
+++ b/drivers/usb/serial/usb_debug.c
@@ -31,10 +31,10 @@ static struct usb_driver debug_driver = {
    .no_dynamic_id =     1,
};

-int usb_debug_open(struct usb_serial_port *port, struct file *filp)
+int usb_debug_open(struct tty_struct *tty, struct usb_serial_port *port, struct 
file *filp)
{
    port->bulk_out_size = USB_DEBUG_MAX_PACKET_SIZE;
-    return usb_serial_generic_open(port, filp);
+    return usb_serial_generic_open(tty, port, filp);
}

static struct usb_serial_driver debug_device = {
-- 
1.5.6.2
original patch was against 2.6.26 git which still has
int usb_serial_generic_open (struct usb_serial_port *port, struct file *filp)

Aleks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help