From: David Eger <eger@havoc.gtf.org>
To: benh@kernel.crashing.org
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH] pmac_zilog: initialize port spinlock on all init paths
Date: Mon, 12 Jul 2004 20:39:35 -0400 [thread overview]
Message-ID: <20040713003935.GA1050@havoc.gtf.org> (raw)
In-Reply-To: <20040712220935.GA20049@havoc.gtf.org>
Dear Ben,
This patch fixes the Zilog driver so it doesn't freak on my TiBook.
( of course, it still spews diahrea of 'IN from bad port XXXXXXXX'
but then, I don't have the hardware.... still, seems weird that OF
would report that I do have said hardware :-/ )
Please look it over and send it on.
-dte
pmac_zilog: initialize the serial ports' spinlocks even if console over
serial is not enabled; disable this driver by default for ppc
This lock not being initialized was Oopsing my TiBook :-P
Signed-off-by: David Eger <eger@havoc.gtf.org>
diff -Nru a/arch/ppc/defconfig b/arch/ppc/defconfig
--- a/arch/ppc/defconfig 2004-07-13 02:30:37 +02:00
+++ b/arch/ppc/defconfig 2004-07-13 02:30:37 +02:00
@@ -689,7 +689,7 @@
# Input Device Drivers
#
CONFIG_INPUT_KEYBOARD=y
-CONFIG_KEYBOARD_ATKBD=y
+# CONFIG_KEYBOARD_ATKBD is not set
# CONFIG_KEYBOARD_SUNKBD is not set
# CONFIG_KEYBOARD_LKKBD is not set
# CONFIG_KEYBOARD_XTKBD is not set
@@ -724,8 +724,8 @@
#
# Non-8250 serial port support
#
-CONFIG_SERIAL_CORE=y
-CONFIG_SERIAL_PMACZILOG=y
+# CONFIG_SERIAL_CORE is not set
+# CONFIG_SERIAL_PMACZILOG is not set
# CONFIG_SERIAL_PMACZILOG_CONSOLE is not set
CONFIG_UNIX98_PTYS=y
CONFIG_LEGACY_PTYS=y
diff -Nru a/drivers/serial/pmac_zilog.c b/drivers/serial/pmac_zilog.c
--- a/drivers/serial/pmac_zilog.c 2004-07-13 02:30:37 +02:00
+++ b/drivers/serial/pmac_zilog.c 2004-07-13 02:30:37 +02:00
@@ -1490,6 +1490,7 @@
uap->port.ops = &pmz_pops;
uap->port.type = PORT_PMAC_ZILOG;
uap->port.flags = 0;
+ spin_lock_init(&uap->port.lock);
/* Setup some valid baud rate information in the register
* shadows so we don't write crap there before baud rate is
@@ -1985,8 +1986,6 @@
/* Probe ports */
pmz_probe();
-#ifdef CONFIG_SERIAL_PMACZILOG_CONSOLE
-#endif
/* TODO: Autoprobe console based on OF */
/* pmz_console.index = i; */
register_console(&pmz_console);
next prev parent reply other threads:[~2004-07-13 0:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-07-12 7:51 pmac oops: devfs versus power management - fight! David Eger
2004-07-12 8:21 ` oops: pmac power management David Eger
2004-07-12 22:09 ` pmac_zilog: driver loads (and crashes) without hardware David Eger
2004-07-13 0:39 ` David Eger [this message]
2004-07-13 4:16 ` [PATCH] pmac_zilog: initialize port spinlock on all init paths Benjamin Herrenschmidt
2004-07-14 4:04 ` David Eger
2004-07-14 23:39 ` Tom Rini
2004-07-16 20:15 ` David Eger
2004-07-16 20:50 ` Tom Rini
2004-07-19 15:51 ` Anton Blanchard
2004-07-16 18:58 ` [PATCH] pmac_zilog: insert correct failure path for device numbers being taken David Eger
2004-07-16 19:01 ` Benjamin Herrenschmidt
2004-07-17 13:07 ` [PATCH] pmac_zilog: fix patch drain bramage David Eger
[not found] <200407141709.i6EH9EYW029131@hera.kernel.org>
2004-07-14 19:27 ` [PATCH] pmac_zilog: initialize port spinlock on all init paths Hollis Blanchard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20040713003935.GA1050@havoc.gtf.org \
--to=eger@havoc.gtf.org \
--cc=benh@kernel.crashing.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.