linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [2.6 patch] drivers/isdn/gigaset/common.c: small cleanups
@ 2006-04-04 16:29 Adrian Bunk
  2006-04-05  9:38 ` Tilman Schmidt
  0 siblings, 1 reply; 2+ messages in thread
From: Adrian Bunk @ 2006-04-04 16:29 UTC (permalink / raw)
  To: hjlipp, tilman; +Cc: gigaset307x-common, linux-kernel, kkeil, isdn4linux

This patch contains the following cleanups:
- makethe needlessly global gigaset_get_cs_by_tty() static
- remove the unused EXPORT_SYMBOL_GPL(gigaset_debugdrivers)

Signed-off-by: Adrian Bunk <bunk@stusta.de>

---

 drivers/isdn/gigaset/common.c  |   17 ++++++++---------
 drivers/isdn/gigaset/gigaset.h |    1 -
 2 files changed, 8 insertions(+), 10 deletions(-)

--- linux-2.6.16-mm2-full/drivers/isdn/gigaset/gigaset.h.old	2006-04-04 00:15:36.000000000 +0200
+++ linux-2.6.16-mm2-full/drivers/isdn/gigaset/gigaset.h	2006-04-04 00:15:42.000000000 +0200
@@ -754,7 +754,6 @@
 /* Deallocate driver structure. */
 void gigaset_freedriver(struct gigaset_driver *drv);
 void gigaset_debugdrivers(void);
-struct cardstate *gigaset_get_cs_by_minor(unsigned minor);
 struct cardstate *gigaset_get_cs_by_tty(struct tty_struct *tty);
 struct cardstate *gigaset_get_cs_by_id(int id);
 
--- linux-2.6.16-mm2-full/drivers/isdn/gigaset/common.c.old	2006-04-04 00:06:34.000000000 +0200
+++ linux-2.6.16-mm2-full/drivers/isdn/gigaset/common.c	2006-04-04 00:16:45.000000000 +0200
@@ -962,16 +962,8 @@
 	}
 	spin_unlock_irqrestore(&driver_lock, flags);
 }
-EXPORT_SYMBOL_GPL(gigaset_debugdrivers);
 
-struct cardstate *gigaset_get_cs_by_tty(struct tty_struct *tty)
-{
-	if (tty->index < 0 || tty->index >= tty->driver->num)
-		return NULL;
-	return gigaset_get_cs_by_minor(tty->index + tty->driver->minor_start);
-}
-
-struct cardstate *gigaset_get_cs_by_minor(unsigned minor)
+static struct cardstate *gigaset_get_cs_by_minor(unsigned minor)
 {
 	unsigned long flags;
 	static struct cardstate *ret = NULL;
@@ -994,6 +986,13 @@
 	return ret;
 }
 
+struct cardstate *gigaset_get_cs_by_tty(struct tty_struct *tty)
+{
+	if (tty->index < 0 || tty->index >= tty->driver->num)
+		return NULL;
+	return gigaset_get_cs_by_minor(tty->index + tty->driver->minor_start);
+}
+
 void gigaset_freedriver(struct gigaset_driver *drv)
 {
 	unsigned long flags;


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [2.6 patch] drivers/isdn/gigaset/common.c: small cleanups
  2006-04-04 16:29 [2.6 patch] drivers/isdn/gigaset/common.c: small cleanups Adrian Bunk
@ 2006-04-05  9:38 ` Tilman Schmidt
  0 siblings, 0 replies; 2+ messages in thread
From: Tilman Schmidt @ 2006-04-05  9:38 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: hjlipp, gigaset307x-common, linux-kernel, kkeil, isdn4linux

[-- Attachment #1: Type: text/plain, Size: 2311 bytes --]

Adrian Bunk wrote:

> This patch contains the following cleanups:
> - makethe needlessly global gigaset_get_cs_by_tty() static
> - remove the unused EXPORT_SYMBOL_GPL(gigaset_debugdrivers)

Thanks for catching these.

> Signed-off-by: Adrian Bunk <bunk@stusta.de>

Acked-by: Tilman Schmidt <tilman@imap.cc>

>
> ---
>
>  drivers/isdn/gigaset/common.c  |   17 ++++++++---------
>  drivers/isdn/gigaset/gigaset.h |    1 -
>  2 files changed, 8 insertions(+), 10 deletions(-)
>
> --- linux-2.6.16-mm2-full/drivers/isdn/gigaset/gigaset.h.old	2006-04-04 00:15:36.000000000 +0200
> +++ linux-2.6.16-mm2-full/drivers/isdn/gigaset/gigaset.h	2006-04-04 00:15:42.000000000 +0200
> @@ -754,7 +754,6 @@
>  /* Deallocate driver structure. */
>  void gigaset_freedriver(struct gigaset_driver *drv);
>  void gigaset_debugdrivers(void);
> -struct cardstate *gigaset_get_cs_by_minor(unsigned minor);
>  struct cardstate *gigaset_get_cs_by_tty(struct tty_struct *tty);
>  struct cardstate *gigaset_get_cs_by_id(int id);
>
> --- linux-2.6.16-mm2-full/drivers/isdn/gigaset/common.c.old	2006-04-04 00:06:34.000000000 +0200
> +++ linux-2.6.16-mm2-full/drivers/isdn/gigaset/common.c	2006-04-04 00:16:45.000000000 +0200
> @@ -962,16 +962,8 @@
>  	}
>  	spin_unlock_irqrestore(&driver_lock, flags);
>  }
> -EXPORT_SYMBOL_GPL(gigaset_debugdrivers);
>
> -struct cardstate *gigaset_get_cs_by_tty(struct tty_struct *tty)
> -{
> -	if (tty->index < 0 || tty->index >= tty->driver->num)
> -		return NULL;
> -	return gigaset_get_cs_by_minor(tty->index + tty->driver->minor_start);
> -}
> -
> -struct cardstate *gigaset_get_cs_by_minor(unsigned minor)
> +static struct cardstate *gigaset_get_cs_by_minor(unsigned minor)
>  {
>  	unsigned long flags;
>  	static struct cardstate *ret = NULL;
> @@ -994,6 +986,13 @@
>  	return ret;
>  }
>
> +struct cardstate *gigaset_get_cs_by_tty(struct tty_struct *tty)
> +{
> +	if (tty->index < 0 || tty->index >= tty->driver->num)
> +		return NULL;
> +	return gigaset_get_cs_by_minor(tty->index + tty->driver->minor_start);
> +}
> +
>  void gigaset_freedriver(struct gigaset_driver *drv)
>  {
>  	unsigned long flags;
>


--
Tilman Schmidt                    E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 250 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-04-05  9:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-04-04 16:29 [2.6 patch] drivers/isdn/gigaset/common.c: small cleanups Adrian Bunk
2006-04-05  9:38 ` Tilman Schmidt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).