Re: [PATCH] of_serial: Add UPF_FIXED_TYPE flag

2009-06-19 Thread Michael Ellerman
On Tue, 2009-06-09 at 18:39 -0500, Dave Mitchell wrote:
 This patch adds the UPF_FIXED_TYPE flag which will bypass the
 8250's autoconfig probe for uart type. The uart type identified
 by the of_serial's parse of the flat device tree will be utilized
 as defined.
 
 Signed-off-by: Dave Mitchell dmitch...@amcc.com
 ---
  drivers/serial/of_serial.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 
 diff --git a/drivers/serial/of_serial.c b/drivers/serial/of_serial.c
 index 14f8fa9..3f2027c 100644
 --- a/drivers/serial/of_serial.c
 +++ b/drivers/serial/of_serial.c
 @@ -67,7 +67,7 @@ static int __devinit of_platform_serial_setup(struct 
 of_device *ofdev,
   port-type = type;
   port-uartclk = *clk;
   port-flags = UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF | UPF_IOREMAP
 - | UPF_FIXED_PORT;
 + | UPF_FIXED_PORT | UPF_FIXED_TYPE;

I think we /probably/ want a similar change in legacy_serial.c, for soc
ports at least.

Or do people have things that are compatible ns16550 but aren't?

cheers


signature.asc
Description: This is a digitally signed message part
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH] of_serial: Add UPF_FIXED_TYPE flag

2009-06-19 Thread Benjamin Herrenschmidt
On Fri, 2009-06-19 at 18:24 +1000, Michael Ellerman wrote:
 I think we /probably/ want a similar change in legacy_serial.c, for
 soc
 ports at least.
 
 Or do people have things that are compatible ns16550 but aren't?
 
Do we really want to keep of_serial.c ?

I fail to see the point in keeping -both- the of_serial (of_device
variant) and the legacy serial stuff which shims the device-tree into
platform devices.

They tend to clash with each other too, which is annoying.

Cheers,
Ben

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


[PATCH] of_serial: Add UPF_FIXED_TYPE flag

2009-06-09 Thread Dave Mitchell
This patch adds the UPF_FIXED_TYPE flag which will bypass the
8250's autoconfig probe for uart type. The uart type identified
by the of_serial's parse of the flat device tree will be utilized
as defined.

Signed-off-by: Dave Mitchell dmitch...@amcc.com
---
 drivers/serial/of_serial.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/of_serial.c b/drivers/serial/of_serial.c
index 14f8fa9..3f2027c 100644
--- a/drivers/serial/of_serial.c
+++ b/drivers/serial/of_serial.c
@@ -67,7 +67,7 @@ static int __devinit of_platform_serial_setup(struct 
of_device *ofdev,
port-type = type;
port-uartclk = *clk;
port-flags = UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF | UPF_IOREMAP
-   | UPF_FIXED_PORT;
+   | UPF_FIXED_PORT | UPF_FIXED_TYPE;
port-dev = ofdev-dev;
/* If current-speed was set, then try not to change it. */
if (spd)
-- 
1.6.3.2

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev