lkml.org 
[lkml]   [2005]   [Jun]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] Add removal schedule of register_serial/unregister_serial to appropriate file
On Sat, Jun 25, 2005 at 10:47:25AM +0100, Russell King wrote:
> On Thu, Jun 23, 2005 at 02:23:35PM +0100, Russell King wrote:
> > +What: register_serial/unregister_serial
> > +When: December 2005
> > +Why: This interface does not allow serial ports to be registered against
> > + a struct device, and as such does not allow correct power management
> > + of such ports. 8250-based ports should use serial8250_register_port
> > + and serial8250_unregister_port instead.
> > +Who: Russell King <rmk@arm.linux.org.uk>
>
> Other places appear to have a prototype for these functions but do not
> use them:
>
> arch/xtensa/platform-iss/console.c:int register_serial(struct serial_struct*);
> arch/xtensa/platform-iss/console.c:void unregister_serial(int);
> drivers/char/amiserial.c:int register_serial(struct serial_struct *req);
> drivers/char/amiserial.c:void unregister_serial(int line);
>
> The patch which follows this message will remove these.
>
>
> In terms of the use of these functions, this is the state of play as of
> yesterday:
>
> arch/frv/kernel/setup.c:// register_serial(&__frv_uart0);
> arch/frv/kernel/setup.c:// register_serial(&__frv_uart1);

I've also removed this. The following is mostly for comment only.
I'm intending to commit this change by Monday at the latest.

[PATCH] Serial: remove unnecessary register_serial/unregister_serial

A couple of drivers declare register_serial/unregister_serial
prototypes but don't use them. FRV contains a commented out
call to register_serial. Since these are deprecated, remove
these unnecessary references.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>

Index: arch/frv/kernel/setup.c
===================================================================
--- 39040c7a05edd69381c0a25636a1a328d856cb9c/arch/frv/kernel/setup.c (mode:100644)
+++ uncommitted/arch/frv/kernel/setup.c (mode:100644)
@@ -790,12 +790,10 @@
#ifndef CONFIG_GDBSTUB_UART0
__reg(UART0_BASE + UART_IER * 8) = 0;
early_serial_setup(&__frv_uart0);
-// register_serial(&__frv_uart0);
#endif
#ifndef CONFIG_GDBSTUB_UART1
__reg(UART1_BASE + UART_IER * 8) = 0;
early_serial_setup(&__frv_uart1);
-// register_serial(&__frv_uart1);
#endif

#if defined(CONFIG_CHR_DEV_FLASH) || defined(CONFIG_BLK_DEV_FLASH)
Index: arch/xtensa/platform-iss/console.c
===================================================================
--- 39040c7a05edd69381c0a25636a1a328d856cb9c/arch/xtensa/platform-iss/console.c (mode:100644)
+++ uncommitted/arch/xtensa/platform-iss/console.c (mode:100644)
@@ -198,9 +198,6 @@
}


-int register_serial(struct serial_struct*);
-void unregister_serial(int);
-
static struct tty_operations serial_ops = {
.open = rs_open,
.close = rs_close,
Index: drivers/char/amiserial.c
===================================================================
--- 39040c7a05edd69381c0a25636a1a328d856cb9c/drivers/char/amiserial.c (mode:100644)
+++ uncommitted/drivers/char/amiserial.c (mode:100644)
@@ -1973,10 +1973,6 @@
}


-int register_serial(struct serial_struct *req);
-void unregister_serial(int line);
-
-
static struct tty_operations serial_ops = {
.open = rs_open,
.close = rs_close,

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-06-25 12:01    [W:0.128 / U:1.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site