Commit 58d784a5 authored by Martin Habets's avatar Martin Habets Committed by David S. Miller
Browse files

[SERIAL] sparc: Infrastructure to fix section mismatch bugs. 



This patch against 2.6.23 sparc-2.6.git contains a number of minor
cleanups of the sparc serial drivers.  Initially I fixed this build
warning:

WARNING: vmlinux.o(.text+0x107a2c): Section mismatch: reference to .init.text:add_preferred_console (between 'sunserial_console_match' and 'sunserial_console_termios')

which is done by declaring sunserial_console_match() as __init.  This
resulted in build warnings on sunserial_current_minor.  To resolve
these the variable was changed so it is no longer global, and to hide
operations on it inside 2 new functions. These functions handle the
UART minor handling code that is common to all sparc serial drivers.

These changes allowed to clean up the uart counters in all the sparc
serial drivers, and the administration of minor device numbers.

Lastly, sunserial_console_termios() does not need to be exported since
it is only called from non-modular code.

Sadly, the following build warning still exists:

WARNING: vmlinux.o(__ksymtab+0x2910): Section mismatch: reference to .init.text:sunserial_console_match (between '__ksymtab_sunserial_console_match' and '__ksymtab_sunserial_unregister_minors')

This could be resolved by not exporting sunserial_console_match(), but
this is not possible at the moment because it is being called from
modular code. On the other hand, this is a bogus warning since it
comes from a ksymtab section.

Signed-off-by: default avatarMartin Habets <errandir_news@mph.eclipse.co.uk>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 4af75653

drivers/serial/suncore.c

+28 −5
Original line number Diff line number Diff line
@@ -23,11 +23,36 @@ 


#include "suncore.h"



int sunserial_current_minor = 64;

static int sunserial_current_minor = 64;



EXPORT_SYMBOL(sunserial_current_minor);

int sunserial_register_minors(struct uart_driver *drv, int count)

{

	int err = 0;



	drv->minor = sunserial_current_minor;

	drv->nr += count;

	/* Register the driver on the first call */

	if (drv->nr == count)

		err = uart_register_driver(drv);

	if (err == 0) {

		sunserial_current_minor += count;

		drv->tty_driver->name_base = drv->minor - 64;

	}

	return err;

}

EXPORT_SYMBOL(sunserial_register_minors);



void sunserial_unregister_minors(struct uart_driver *drv, int count)

{

	drv->nr -= count;

	sunserial_current_minor -= count;



	if (drv->nr == 0)

		uart_unregister_driver(drv);

}

EXPORT_SYMBOL(sunserial_unregister_minors);



int sunserial_console_match(struct console *con, struct device_node *dp,

int __init sunserial_console_match(struct console *con, struct device_node *dp,

			    struct uart_driver *drv, int line)

{

	int off;
@@ -133,8 +158,6 @@ sunserial_console_termios(struct console *con) 
	con->cflag = cflag;

}



EXPORT_SYMBOL(sunserial_console_termios);



/* Sun serial MOUSE auto baud rate detection.  */

static struct mouse_baud_cflag {

	int baud;

drivers/serial/suncore.h

+2 −1
Original line number Diff line number Diff line
@@ -22,7 +22,8 @@ 
extern unsigned int suncore_mouse_baud_cflag_next(unsigned int, int *);

extern int suncore_mouse_baud_detection(unsigned char, int);



extern int sunserial_current_minor;

extern int sunserial_register_minors(struct uart_driver *, int);

extern void sunserial_unregister_minors(struct uart_driver *, int);



extern int sunserial_console_match(struct console *, struct device_node *,

				   struct uart_driver *, int);

drivers/serial/sunhv.c

+3 −11
Original line number Diff line number Diff line
@@ -562,16 +562,10 @@ static int __devinit hv_probe(struct of_device *op, const struct of_device_id *m 


	port->dev = &op->dev;



	sunhv_reg.minor = sunserial_current_minor;

	sunhv_reg.nr = 1;



	err = uart_register_driver(&sunhv_reg);

	err = sunserial_register_minors(&sunhv_reg, 1);

	if (err)

		goto out_free_con_read_page;



	sunhv_reg.tty_driver->name_base = sunhv_reg.minor - 64;

	sunserial_current_minor += 1;



	sunserial_console_match(&sunhv_console, op->node,

				&sunhv_reg, port->line);
@@ -591,8 +585,7 @@ out_remove_port: 
	uart_remove_one_port(&sunhv_reg, port);



out_unregister_driver:

	sunserial_current_minor -= 1;

	uart_unregister_driver(&sunhv_reg);

	sunserial_unregister_minors(&sunhv_reg, 1);



out_free_con_read_page:

	kfree(con_read_page);
@@ -614,8 +607,7 @@ static int __devexit hv_remove(struct of_device *dev) 


	uart_remove_one_port(&sunhv_reg, port);



	sunserial_current_minor -= 1;

	uart_unregister_driver(&sunhv_reg);

	sunserial_unregister_minors(&sunhv_reg, 1);



	kfree(port);

	sunhv_port = NULL;

drivers/serial/sunsab.c

+4 −12
Original line number Diff line number Diff line
@@ -832,7 +832,6 @@ static struct uart_driver sunsab_reg = { 
};



static struct uart_sunsab_port *sunsab_ports;

static int num_channels;



#ifdef CONFIG_SERIAL_SUNSAB_CONSOLE
@@ -1102,8 +1101,8 @@ static int __init sunsab_init(void) 
{

	struct device_node *dp;

	int err;

	int num_channels = 0;



	num_channels = 0;

	for_each_node_by_name(dp, "se")

		num_channels += 2;

	for_each_node_by_name(dp, "serial") {
@@ -1117,20 +1116,14 @@ static int __init sunsab_init(void) 
		if (!sunsab_ports)

			return -ENOMEM;



		sunsab_reg.minor = sunserial_current_minor;

		sunsab_reg.nr = num_channels;

		sunsab_reg.cons = SUNSAB_CONSOLE();



		err = uart_register_driver(&sunsab_reg);

		err = sunserial_register_minors(&sunsab_reg, num_channels);

		if (err) {

			kfree(sunsab_ports);

			sunsab_ports = NULL;



			return err;

		}



		sunsab_reg.tty_driver->name_base = sunsab_reg.minor - 64;

		sunserial_current_minor += num_channels;

	}



	return of_register_driver(&sab_driver, &of_bus_type);
@@ -1139,9 +1132,8 @@ static int __init sunsab_init(void) 
static void __exit sunsab_exit(void)

{

	of_unregister_driver(&sab_driver);

	if (num_channels) {

		sunserial_current_minor -= num_channels;

		uart_unregister_driver(&sunsab_reg);

	if (sunsab_reg.nr) {

		sunserial_unregister_minors(&sunsab_reg, sunsab_reg.nr);

	}



	kfree(sunsab_ports);

drivers/serial/sunsu.c

+5 −11
Original line number Diff line number Diff line
@@ -1528,14 +1528,12 @@ static struct of_platform_driver su_driver = { 
	.remove		= __devexit_p(su_remove),

};



static int num_uart;



static int __init sunsu_init(void)

{

	struct device_node *dp;

	int err;

	int num_uart = 0;



	num_uart = 0;

	for_each_node_by_name(dp, "su") {

		if (su_get_type(dp) == SU_PORT_PORT)

			num_uart++;
@@ -1552,26 +1550,22 @@ static int __init sunsu_init(void) 
	}



	if (num_uart) {

		sunsu_reg.minor = sunserial_current_minor;

		sunsu_reg.nr = num_uart;

		err = uart_register_driver(&sunsu_reg);

		err = sunserial_register_minors(&sunsu_reg, num_uart);

		if (err)

			return err;

		sunsu_reg.tty_driver->name_base = sunsu_reg.minor - 64;

		sunserial_current_minor += num_uart;

	}



	err = of_register_driver(&su_driver, &of_bus_type);

	if (err && num_uart)

		uart_unregister_driver(&sunsu_reg);

		sunserial_unregister_minors(&sunsu_reg, num_uart);



	return err;

}



static void __exit sunsu_exit(void)

{

	if (num_uart)

		uart_unregister_driver(&sunsu_reg);

	if (sunsu_reg.nr)

		sunserial_unregister_minors(&sunsu_reg, sunsu_reg.nr);

}



module_init(sunsu_init);

CRA Git | Maintained and supported by SUSTech CRA and CCSE