gigaset: patches for 2.6.29

PATCH Applied Tilman Schmidt 3 participants 7 messages Nov 29, 2008 – Nov 30, 2008

Subsystems (2)

Networking Network drivers

lore

Share & tools

lore

Tilman Schmidt

author

2008-11-29 22:41 UTC #0 lore raw

cover gigaset: patches for 2.6.29

Dave,

I gather you are willing to take ISDN subsystem patches through your networking tree. Would you please merge the following two patches for the Gigaset driver.

Thanks,
Tilman

Tilman Schmidt

author

2008-11-29 22:41 UTC #1 lore raw

PATCH 1/2 gigaset: remove unnecessary poll method

The N_GIGASET_M101 line discipline implemented by the ser_gigaset driver does not transfer any data from/to userspace through the tty interface. Therefore a poll method is not needed.

Signed-off-by Tilman Schmidt <[email protected]>

---
 drivers/isdn/gigaset/ser-gigaset.c |   15 ---------------
 1 files changed, 0 insertions(+), 15 deletions(-)

drivers/isdn/gigaset/ser-gigaset.c +0 ~~-15~~

index 07052ed..0b8c8f2 100644
@@ -16,7 +16,6 @@
 #include <linux/moduleparam.h>
 #include <linux/platform_device.h>
 #include <linux/tty.h>
-#include <linux/poll.h>
 #include <linux/completion.h>
 
 /* Version Information */

@@ -571,7 +570,6 @@ gigaset_tty_close(struct tty_struct *tty)
 	}
 
 	/* prevent other callers from entering ldisc methods */
-	/* FIXME: should use the tty state flags */
 	tty->disc_data = NULL;
 
 	if (!cs->hw.ser)

@@ -673,18 +671,6 @@ gigaset_tty_ioctl(struct tty_struct *tty, struct file *file,
 }
 
 /*
- * Poll on the tty.
- * Unused, always return zero.
- *
- * FIXME: should probably return an exception - especially on hangup
- */
-static unsigned int
-gigaset_tty_poll(struct tty_struct *tty, struct file *file, poll_table *wait)
-{
-	return 0;
-}
-
-/*
  * Called by the tty driver when a block of data has been received.
  * Will not be re-entered while running but other ldisc functions
  * may be called in parallel.

@@ -773,7 +759,6 @@ static struct tty_ldisc_ops gigaset_ldisc = {
 	.read		= gigaset_tty_read,
 	.write		= gigaset_tty_write,
 	.ioctl		= gigaset_tty_ioctl,
-	.poll		= gigaset_tty_poll,
 	.receive_buf	= gigaset_tty_receive,
 	.write_wakeup	= gigaset_tty_wakeup,
 };

Signature · 1 line

1.6.0.1.100.gcb138

David Miller

maintainer Applied

2008-11-30 05:39 UTC #2 lore raw

Re: [PATCH 1/2] gigaset: remove unnecessary poll method

From: Tilman Schmidt <[email protected]>
Date: Sat, 29 Nov 2008 23:41:07 +0100 (CET)

Quoted 5 lines

The N_GIGASET_M101 line discipline implemented by the ser_gigaset driver does not transfer any data from/to userspace through the tty interface. Therefore a poll method is not needed.
Signed-off-by Tilman Schmidt <[email protected]>

Applied to net-next-2.6

Tilman Schmidt

author

2008-11-29 22:41 UTC #3 lore raw

PATCH 2/2 gigaset: get rid of info() and warn() macros

In reply to Tilman Schmidt

Join the move away from the obsolete info() macro, opencoding the remaining uses. While we're at it, also get rid of the warn() macro by promoting the three remaining uses to err().

Signed-off-by Tilman Schmidt <[email protected]>

---
 drivers/isdn/gigaset/bas-gigaset.c |    3 +--
 drivers/isdn/gigaset/common.c      |    5 ++---
 drivers/isdn/gigaset/gigaset.h     |    6 ------
 drivers/isdn/gigaset/interface.c   |    4 ++--
 drivers/isdn/gigaset/ser-gigaset.c |    3 +--
 drivers/isdn/gigaset/usb-gigaset.c |    3 +--
 6 files changed, 7 insertions(+), 17 deletions(-)

drivers/isdn/gigaset/bas-gigaset.c +1 -2

index 3f11910..3eca9c8 100644
@@ -2507,8 +2507,7 @@ static int __init bas_gigaset_init(void)
 		goto error;
 	}
 
-	info(DRIVER_AUTHOR);
-	info(DRIVER_DESC);
+	printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_DESC "\n");
 	return 0;
 
 error:

drivers/isdn/gigaset/common.c +2 -3

index 9d3ce77..1664e31 100644
@@ -580,7 +580,7 @@ static struct bc_state *gigaset_initbcs(struct bc_state *bcs,
 	} else if ((bcs->skb = dev_alloc_skb(SBUFSIZE + HW_HDR_LEN)) != NULL)
 		skb_reserve(bcs->skb, HW_HDR_LEN);
 	else {
-		warn("could not allocate skb");
+		err("could not allocate skb");
 		bcs->inputstate |= INS_skip_frame;
 	}

@@ -1120,8 +1120,7 @@ static int __init gigaset_init_module(void)
 	if (gigaset_debuglevel == 1)
 		gigaset_debuglevel = DEBUG_DEFAULT;
 
-	info(DRIVER_AUTHOR);
-	info(DRIVER_DESC);
+	printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_DESC "\n");
 	return 0;
 }

drivers/isdn/gigaset/gigaset.h +0 -6

index 0037529..901ff43 100644
@@ -104,15 +104,9 @@ enum debuglevel {
  * removed rather than fixed anyway.
  */
 #undef err
-#undef info
-#undef warn
 
 #define err(format, arg...) printk(KERN_ERR KBUILD_MODNAME ": " \
 	format "\n" , ## arg)
-#define info(format, arg...) printk(KERN_INFO KBUILD_MODNAME ": " \
-	format "\n" , ## arg)
-#define warn(format, arg...) printk(KERN_WARNING KBUILD_MODNAME ": " \
-	format "\n" , ## arg)
 
 #ifdef CONFIG_GIGASET_DEBUG

drivers/isdn/gigaset/interface.c +2 -2

index 521951a..956381c 100644
@@ -701,7 +701,7 @@ void gigaset_if_initdriver(struct gigaset_driver *drv, const char *procname,
 
 	ret = tty_register_driver(tty);
 	if (ret < 0) {
-		warn("failed to register tty driver (error %d)", ret);
+		err("failed to register tty driver (error %d)", ret);
 		goto error;
 	}
 	gig_dbg(DEBUG_IF, "tty driver initialized");

@@ -709,7 +709,7 @@ void gigaset_if_initdriver(struct gigaset_driver *drv, const char *procname,
 	return;
 
 enomem:
-	warn("could not allocate tty structures");
+	err("could not allocate tty structures");
 error:
 	if (drv->tty)
 		put_tty_driver(drv->tty);

drivers/isdn/gigaset/ser-gigaset.c +1 -2

index 0b8c8f2..b306a2f 100644
@@ -513,8 +513,7 @@ gigaset_tty_open(struct tty_struct *tty)
 
 	gig_dbg(DEBUG_INIT, "Starting HLL for Gigaset M101");
 
-	info(DRIVER_AUTHOR);
-	info(DRIVER_DESC);
+	printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_DESC "\n");
 
 	if (!driver) {
 		err("%s: no driver structure", __func__);

drivers/isdn/gigaset/usb-gigaset.c +1 -2

index 4661830..e847c6e 100644
@@ -941,8 +941,7 @@ static int __init usb_gigaset_init(void)
 		goto error;
 	}
 
-	info(DRIVER_AUTHOR);
-	info(DRIVER_DESC);
+	printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_DESC "\n");
 	return 0;
 
 error:

Signature · 1 line

1.6.0.1.100.gcb138

David Miller

maintainer

2008-11-30 05:39 UTC #4 lore raw

Re: [PATCH 2/2] gigaset: get rid of info() and warn() macros

From: Tilman Schmidt <[email protected]>
Date: Sat, 29 Nov 2008 23:41:07 +0100 (CET)

Quoted 5 lines

Join the move away from the obsolete info() macro, opencoding the remaining uses. While we're at it, also get rid of the warn() macro by promoting the three remaining uses to err().
Signed-off-by Tilman Schmidt <[email protected]>

Also applied to net-next-2.6

Thanks

Alan Cox

2008-11-29 23:43 UTC #5 lore raw

In reply to Tilman Schmidt

Quoted 5 lines · On Sat, 29 Nov 2008 23:41:07 +0100 (CET) Tilman Schmidt <[email protected]> wrote:

Dave,
I gather you are willing to take ISDN subsystem patches through your networking tree. Would you please merge the following two patches for the Gigaset driver.

Or via the ttydev tree - either.

Alan

David Miller

maintainer

2008-11-30 05:37 UTC #6 lore raw

From: Alan Cox <[email protected]>
Date: Sat, 29 Nov 2008 23:44:02 +0000

Quoted 10 lines

Quoted 5 lines · On Sat, 29 Nov 2008 23:41:07 +0100 (CET) Tilman Schmidt <[email protected]> wrote:
Dave,
I gather you are willing to take ISDN subsystem patches through your networking tree. Would you please merge the following two patches for the Gigaset driver.
Or via the ttydev tree - either.

Yep either way is fine.

I'll take these.