* [PATCH] early_port_register
@ 2003-06-12 20:20 Matt Porter
2003-06-12 20:27 ` Christoph Hellwig
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Matt Porter @ 2003-06-12 20:20 UTC (permalink / raw)
To: linux-kernel; +Cc: rmk
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 */
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-12 20:20 [PATCH] early_port_register Matt Porter
@ 2003-06-12 20:27 ` Christoph Hellwig
2003-06-12 20:41 ` Matt Porter
2003-06-12 20:50 ` David Mosberger
2003-06-12 21:44 ` Tom Rini
2 siblings, 1 reply; 8+ messages in thread
From: Christoph Hellwig @ 2003-06-12 20:27 UTC (permalink / raw)
To: Matt Porter; +Cc: linux-kernel, rmk
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..)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-12 20:27 ` Christoph Hellwig
@ 2003-06-12 20:41 ` Matt Porter
2003-06-12 21:53 ` Russell King
0 siblings, 1 reply; 8+ messages in thread
From: Matt Porter @ 2003-06-12 20:41 UTC (permalink / raw)
To: Christoph Hellwig, Matt Porter, linux-kernel, rmk
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
mporter@kernel.crashing.org
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-12 20:20 [PATCH] early_port_register Matt Porter
2003-06-12 20:27 ` Christoph Hellwig
@ 2003-06-12 20:50 ` David Mosberger
2003-06-12 21:44 ` Tom Rini
2 siblings, 0 replies; 8+ messages in thread
From: David Mosberger @ 2003-06-12 20:50 UTC (permalink / raw)
To: Matt Porter; +Cc: linux-kernel, rmk
>>>>> On Thu, 12 Jun 2003 13:20:01 -0700, Matt Porter <mporter@kernel.crashing.org> 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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-12 20:20 [PATCH] early_port_register Matt Porter
2003-06-12 20:27 ` Christoph Hellwig
2003-06-12 20:50 ` David Mosberger
@ 2003-06-12 21:44 ` Tom Rini
2 siblings, 0 replies; 8+ messages in thread
From: Tom Rini @ 2003-06-12 21:44 UTC (permalink / raw)
To: Matt Porter; +Cc: linux-kernel, rmk
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/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-12 20:41 ` Matt Porter
@ 2003-06-12 21:53 ` Russell King
2003-06-13 9:52 ` Russell King
0 siblings, 1 reply; 8+ messages in thread
From: Russell King @ 2003-06-12 21:53 UTC (permalink / raw)
To: Matt Porter; +Cc: Christoph Hellwig, linux-kernel
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:
ChangeSet@1.1113.2.1, 2003-05-14 16:50:47+02:00, hch@lab343.munich.sgi.com
acpi serial stuff
I'll request Linus pulls this tonight.
--
Russell King (rmk@arm.linux.org.uk) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-12 21:53 ` Russell King
@ 2003-06-13 9:52 ` Russell King
2003-06-14 21:02 ` Russell King
0 siblings, 1 reply; 8+ messages in thread
From: Russell King @ 2003-06-13 9:52 UTC (permalink / raw)
To: Matt Porter, Christoph Hellwig, linux-kernel
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:
>
> ChangeSet@1.1113.2.1, 2003-05-14 16:50:47+02:00, hch@lab343.munich.sgi.com
> acpi serial stuff
>
> I'll request Linus pulls this tonight.
FYI, its still sitting here...
--
Russell King (rmk@arm.linux.org.uk) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] early_port_register
2003-06-13 9:52 ` Russell King
@ 2003-06-14 21:02 ` Russell King
0 siblings, 0 replies; 8+ messages in thread
From: Russell King @ 2003-06-14 21:02 UTC (permalink / raw)
To: Matt Porter, Christoph Hellwig, linux-kernel
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:
> >
> > ChangeSet@1.1113.2.1, 2003-05-14 16:50:47+02:00, hch@lab343.munich.sgi.com
> > 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 (rmk@arm.linux.org.uk) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2003-06-14 20:48 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-06-12 20:20 [PATCH] early_port_register Matt Porter
2003-06-12 20:27 ` Christoph Hellwig
2003-06-12 20:41 ` Matt Porter
2003-06-12 21:53 ` Russell King
2003-06-13 9:52 ` Russell King
2003-06-14 21:02 ` Russell King
2003-06-12 20:50 ` David Mosberger
2003-06-12 21:44 ` Tom Rini
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).