Linus, please apply.
This has been discussed in a previous thread originated by David
Mosberger. This removed early_serial_setup() in favor of a
working early_port_register() call. Many PPC systems rely on
this functionality and are currently hacking around it in the
PPC devel tree. Last I looked, IA64 still had this in their
devel tree too.
-Matt
===== drivers/serial/8250.c 1.32 vs edited =====
--- 1.32/drivers/serial/8250.c Thu Jun 5 23:36:47 2003
+++ edited/drivers/serial/8250.c Thu Jun 12 11:43:54 2003
@@ -2061,9 +2061,15 @@
return __register_serial(req, -1);
}
-int __init early_serial_setup(struct serial_struct *req)
+int __init early_register_port(struct uart_port *port)
{
- __register_serial(req, req->line);
+ if (port->line >= ARRAY_SIZE(serial8250_ports))
+ return -ENODEV;
+
+ serial8250_isa_init_ports(); /* force ISA defaults */
+ serial8250_ports[port->line].port = *port;
+ serial8250_ports[port->line].port.ops = &serial8250_pops;
+
return 0;
}
===== include/linux/serial.h 1.8 vs edited =====
--- 1.8/include/linux/serial.h Sun Feb 16 12:59:58 2003
+++ edited/include/linux/serial.h Thu Jun 12 11:43:59 2003
@@ -180,8 +180,10 @@
extern int register_serial(struct serial_struct *req);
extern void unregister_serial(int line);
+struct uart_port;
+
/* Allow complicated architectures to specify rs_table[] at run time */
-extern int early_serial_setup(struct serial_struct *req);
+extern int early_register_port(struct uart_port *port);
#ifdef CONFIG_ACPI
/* tty ports reserved for the ACPI serial console port and debug port */
On Thu, Jun 12, 2003 at 01:20:01PM -0700, Matt Porter wrote:
> Linus, please apply.
Umm, isn't Russell the serial maintainer? I think you should
discuss changes in that area with him.. (And I remember having
sent him a similar patch..)
On Thu, Jun 12, 2003 at 09:27:23PM +0100, Christoph Hellwig wrote:
> On Thu, Jun 12, 2003 at 01:20:01PM -0700, Matt Porter wrote:
> > Linus, please apply.
>
> Umm, isn't Russell the serial maintainer? I think you should
> discuss changes in that area with him.. (And I remember having
> sent him a similar patch..)
Well, I didn't mention that I first sent this to him more than two
months ago. He hasn't had time to reply about it so it may be that
he's not actively maintaining the serial driver. Queried a couple
more times with no reply. This is the next stop.
BTW, in the original thread, David M. seemed to indicate that
Russell was in support of this FWIW.
Regards,
--
Matt Porter
[email protected]
>>>>> On Thu, 12 Jun 2003 13:20:01 -0700, Matt Porter <[email protected]> said:
Matt> Linus, please apply.
Matt> This has been discussed in a previous thread originated by David
Matt> Mosberger. This removed early_serial_setup() in favor of a
Matt> working early_port_register() call. Many PPC systems rely on
Matt> this functionality and are currently hacking around it in the
Matt> PPC devel tree. Last I looked, IA64 still had this in their
Matt> devel tree too.
Absolutely. I was planning to resubmit it, but you beat me to it (and
I'm happy you did; one less thing to worry about... ;-).
--david
On Thu, Jun 12, 2003 at 01:20:01PM -0700, Matt Porter wrote:
> This has been discussed in a previous thread originated by David
> Mosberger. This removed early_serial_setup() in favor of a
> working early_port_register() call. Many PPC systems rely on
> this functionality and are currently hacking around it in the
> PPC devel tree. Last I looked, IA64 still had this in their
> devel tree too.
I'm not sure if this is related to recent TTY changes or not, but this
does bring up a small 'issue'. To get at uart_port, you need
<linux/serial_core.h>, which currently requires but does not #include
<linux/tty.h>. But I haven't been following close enough to know if
this is new breakage due to TTY changes or just no one had done
linux/serial_core.h w/o linux/tty.h before..
--
Tom Rini
http://gate.crashing.org/~trini/
On Thu, Jun 12, 2003 at 01:41:13PM -0700, Matt Porter wrote:
> On Thu, Jun 12, 2003 at 09:27:23PM +0100, Christoph Hellwig wrote:
> > On Thu, Jun 12, 2003 at 01:20:01PM -0700, Matt Porter wrote:
> > > Linus, please apply.
> >
> > Umm, isn't Russell the serial maintainer? I think you should
> > discuss changes in that area with him.. (And I remember having
> > sent him a similar patch..)
>
> Well, I didn't mention that I first sent this to him more than two
> months ago. He hasn't had time to reply about it so it may be that
> he's not actively maintaining the serial driver. Queried a couple
> more times with no reply. This is the next stop.
Sigh, there seems to be one cset still pending in my serial BK tree:
[email protected], 2003-05-14 16:50:47+02:00, [email protected]
acpi serial stuff
I'll request Linus pulls this tonight.
--
Russell King ([email protected]) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html
On Thu, Jun 12, 2003 at 10:53:10PM +0100, Russell King wrote:
> Sigh, there seems to be one cset still pending in my serial BK tree:
>
> [email protected], 2003-05-14 16:50:47+02:00, [email protected]
> acpi serial stuff
>
> I'll request Linus pulls this tonight.
FYI, its still sitting here...
--
Russell King ([email protected]) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html
On Fri, Jun 13, 2003 at 10:52:38AM +0100, Russell King wrote:
> On Thu, Jun 12, 2003 at 10:53:10PM +0100, Russell King wrote:
> > Sigh, there seems to be one cset still pending in my serial BK tree:
> >
> > [email protected], 2003-05-14 16:50:47+02:00, [email protected]
> > acpi serial stuff
> >
> > I'll request Linus pulls this tonight.
>
> FYI, its still sitting here...
Linus pulled it, but added an extra cset:
1.1311 Remove strange and broken ACPI rule from serial Makefile
diff -Nru a/drivers/serial/Makefile b/drivers/serial/Makefile
--- a/drivers/serial/Makefile Wed May 14 07:50:37 2003
+++ b/drivers/serial/Makefile Sat Jun 14 12:07:41 2003
@@ -8,7 +8,6 @@
serial-8250-$(CONFIG_GSC) += 8250_gsc.o
serial-8250-$(CONFIG_PCI) += 8250_pci.o
serial-8250-$(CONFIG_PNP) += 8250_pnp.o
-serial-8250-$(CONFIG_ACPI) += acpi.o
serial-8250-$(CONFIG_SERIAL_HCDP) += 8250_hcdp.o
obj-$(CONFIG_SERIAL_CORE) += core.o
The file is actually called "8250_acpi.o".
Can you guys ensure that the patch and code works and send an update
direct to Linus please?
Unfortunately, its too late for 2.5.71.
--
Russell King ([email protected]) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html