From: Pierre Ossman <drzeus-list@drzeus.cx>
To: LKML <linux-kernel@vger.kernel.org>
Cc: Russell King <rmk+lkml@arm.linux.org.uk>,
Alan Cox <alan@lxorguk.ukuu.org.uk>,
Linus Torvalds <torvalds@osdl.org>
Subject: Re: ISA DMA controller hangs
Date: Thu, 30 Jun 2005 10:00:24 +0200 [thread overview]
Message-ID: <42C3A698.9020404@drzeus.cx> (raw)
In-Reply-To: <20050616170622.A1712@flint.arm.linux.org.uk>
[-- Attachment #1: Type: text/plain, Size: 374 bytes --]
Reset the ISA DMA controller into a known state after a suspend. Primary
concern was reenabling the cascading DMA channel (4).
Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
If this patch seems ok then I'll make another one for x86_64 (the only
other platform with ISA DMA AFAIK).
I couldn't find who was responsible for the i386 arch so I've cc:ed Linus.
Rgds
Pierre
[-- Attachment #2: i8237-i386.patch --]
[-- Type: text/x-patch, Size: 2134 bytes --]
Index: linux-wbsd/arch/i386/kernel/i8237.c
===================================================================
--- linux-wbsd/arch/i386/kernel/i8237.c (revision 0)
+++ linux-wbsd/arch/i386/kernel/i8237.c (revision 0)
@@ -0,0 +1,67 @@
+/*
+ * i8237.c: 8237A DMA controller suspend functions.
+ *
+ * Written by Pierre Ossman, 2005.
+ */
+
+#include <linux/init.h>
+#include <linux/sysdev.h>
+
+#include <asm/dma.h>
+
+/*
+ * This module just handles suspend/resume issues with the
+ * 8237A DMA controller (used for ISA and LPC).
+ * Allocation is handled in kernel/dma.c and normal usage is
+ * in asm/dma.h.
+ */
+
+static int i8237A_resume(struct sys_device *dev)
+{
+ unsigned long flags;
+ int i;
+
+ flags = claim_dma_lock();
+
+ dma_outb(DMA1_RESET_REG, 0);
+ dma_outb(DMA2_RESET_REG, 0);
+
+ for (i = 0;i < 8;i++) {
+ set_dma_addr(i, 0x000000);
+ /* DMA count is a bit wierd so this is not 0 */
+ set_dma_count(i, 1);
+ }
+
+ /* Enable cascade DMA or channel 0-3 won't work */
+ enable_dma(4);
+
+ release_dma_lock(flags);
+
+ return 0;
+}
+
+static int i8237A_suspend(struct sys_device *dev, pm_message_t state)
+{
+ return 0;
+}
+
+static struct sysdev_class i8237_sysdev_class = {
+ set_kset_name("i8237"),
+ .suspend = i8237A_suspend,
+ .resume = i8237A_resume,
+};
+
+static struct sys_device device_i8237A = {
+ .id = 0,
+ .cls = &i8237_sysdev_class,
+};
+
+static int __init i8237A_init_sysfs(void)
+{
+ int error = sysdev_class_register(&i8237_sysdev_class);
+ if (!error)
+ error = sysdev_register(&device_i8237A);
+ return error;
+}
+
+device_initcall(i8237A_init_sysfs);
Index: linux-wbsd/arch/i386/kernel/Makefile
===================================================================
--- linux-wbsd/arch/i386/kernel/Makefile (revision 153)
+++ linux-wbsd/arch/i386/kernel/Makefile (working copy)
@@ -7,7 +7,7 @@
obj-y := process.o semaphore.o signal.o entry.o traps.o irq.o vm86.o \
ptrace.o time.o ioport.o ldt.o setup.o i8259.o sys_i386.o \
pci-dma.o i386_ksyms.o i387.o dmi_scan.o bootflag.o \
- doublefault.o quirks.o
+ doublefault.o quirks.o i8237.o
obj-y += cpu/
obj-y += timers/
next prev parent reply other threads:[~2005-06-30 8:01 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-28 13:38 ISA DMA controller hangs Pierre Ossman
2005-05-28 13:51 ` Alan Cox
2005-06-05 8:21 ` Pierre Ossman
2005-06-05 14:03 ` Pierre Ossman
2005-06-16 15:53 ` Pierre Ossman
2005-06-16 16:06 ` Russell King
2005-06-18 8:15 ` Pierre Ossman
2005-06-19 15:45 ` Russell King
2005-06-30 8:00 ` Pierre Ossman [this message]
2005-06-30 11:28 ` Alan Cox
2005-06-30 12:20 ` Pierre Ossman
2005-06-30 17:55 ` [PATCH 1/2] ISA DMA suspend for i386 Pierre Ossman
2005-06-30 17:58 ` [PATCH 2/2] ISA DMA suspend for x86_64 Pierre Ossman
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=42C3A698.9020404@drzeus.cx \
--to=drzeus-list@drzeus.cx \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=rmk+lkml@arm.linux.org.uk \
--cc=torvalds@osdl.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox