On Tue, 13 Dec 2011, Benjamin Herrenschmidt wrote:

On Tue, 2011-12-13 at 00:34 +1100, Finn Thain wrote:

quoted

On Mon, 12 Dec 2011, Benjamin Herrenschmidt wrote:

quoted

Any chance you can test this patch ? I would not be surprised if it 
broke m68k since I had to do some of the changes in there "blind", 
so let me know... with this, I can again suspend/resume properly on 
a Pismo while using the internal modem among other things.

The patch works on a PowerBook 520 given a few changes (below). This 
PowerBook only has one serial port that I can test (the internal modem 
is not supported on 68k Macs).

Interesting. The modem is a soft-modem "geoport" or a hw serial modem ? 

It's the latter.

In the later case it's probably just a matter of finding the right GPIO 
bit in Apple ASIC to turn the power on :-)

Surely feasible, but not high on the list of missing hardware support.

quoted

 Can you test a machine with two ports? The rest of my Mac hardware is 
in storage since I moved house last week.

I tried on 2 port powermacs, but I only have one adapter, so I've 
basically been running with one serial port open and shooting irda frame 
on the other (with nothing to check wether I got the frames on the other 
hand), oh well ...

I'll apply your patch and commit via my tree.

I forgot to include this fix for your logging change.

Finn

Index: linux-git/drivers/tty/serial/pmac_zilog.c
===================================================================

--- linux-git.orig/drivers/tty/serial/pmac_zilog.c	2011-12-13 12:12:05.000000000 +1100
+++ linux-git/drivers/tty/serial/pmac_zilog.c	2011-12-13 12:13:29.000000000 +1100

@@ -99,6 +99,10 @@ MODULE_LICENSE("GPL");
 #define PMACZILOG_NAME		"ttyPZ"
 #endif
 
+#define pmz_debug(fmt, arg...)	pr_debug(PMACZILOG_NAME "%d: " fmt, uap->port.line, ## arg)
+#define pmz_error(fmt, arg...)	pr_err(PMACZILOG_NAME "%d: " fmt, uap->port.line, ## arg)
+#define pmz_info(fmt, arg...)	pr_info(PMACZILOG_NAME "%d: " fmt, uap->port.line, ## arg)
+
 
 /*
  * For the sake of early serial console, we can do a pre-probe

Index: linux-git/drivers/tty/serial/pmac_zilog.h
===================================================================

--- linux-git.orig/drivers/tty/serial/pmac_zilog.h	2011-12-13 12:12:05.000000000 +1100
+++ linux-git/drivers/tty/serial/pmac_zilog.h	2011-12-13 12:12:28.000000000 +1100

@@ -1,10 +1,6 @@
 #ifndef __PMAC_ZILOG_H__
 #define __PMAC_ZILOG_H__
 
-#define pmz_debug(fmt, arg...)	pr_debug("ttyPZ%d: " fmt, uap->port.line, ## arg)
-#define pmz_error(fmt, arg...)	pr_err("ttyPZ%d: " fmt, uap->port.line, ## arg)
-#define pmz_info(fmt, arg...)	pr_info("ttyPZ%d: " fmt, uap->port.line, ## arg)
-
 /*
  * At most 2 ESCCs with 2 ports each
  */