From: Ondrej Zary <linux@rainbow-software.org>
To: Linux Kernel <linux-kernel@vger.kernel.org>
Cc: alsa-devel@alsa-project.org
Subject: PNP_DRIVER_RES_DISABLE breaks swsusp at least with snd_cs4236
Date: Wed, 9 Jan 2008 23:43:47 +0100 [thread overview]
Message-ID: <200801092343.48726.linux@rainbow-software.org> (raw)
Hello,
as hibernation (swsusp) started to work with my CPU, I found that my Turtle
Beach Malibu stops working after resume from hibernation. It's caused by fact
that the card is not enabled on the pnp layer during resume - and thus card
registers are inaccessible (reads return FFs, writes go nowhere).
During resume, pnp_bus_resume() in drivers/pnp/driver.c is called for each pnp
device. This function calls pnp_start_dev() only when the
PNP_DRIVER_RES_DO_NOT_CHANGE bit is NOT seting pnp_drv->flags. But the cs4236
driver in sound/isa/cs423x/cs4236.c explicitly sets the .flags to
PNP_DRIVER_RES_DISABLE - it's value is 3 and that includes
PNP_DRIVER_RES_DO_NOT_CHANGE bit.
The same .flags value is present in many of the ALSA ISA sound drivers.
Removing that .flags line caused this to appear inlog when loading snd_cs4236
module:
CS4236+ WSS PnP manual resources are invalid, using auto config
CS4236+ CTRL PnP manual resources are invalid, using auto config
CS4236+ MPU401 PnP manual resources are invalid, using auto config
and the sound now works after resume!
So the question is: why is this line present?
Is this a bug? What's the correct fix?
--
Ondrej Zary
next reply other threads:[~2008-01-09 22:43 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-09 22:43 Ondrej Zary [this message]
2008-01-10 1:53 ` PNP_DRIVER_RES_DISABLE breaks swsusp at least with snd_cs4236 Rene Herman
2008-01-10 1:53 ` [alsa-devel] " Rene Herman
2008-01-10 7:58 ` Jaroslav Kysela
2008-01-10 7:58 ` [alsa-devel] " Jaroslav Kysela
2008-01-11 1:19 ` Rene Herman
2008-01-11 1:19 ` [alsa-devel] " Rene Herman
2008-01-11 7:01 ` Pierre Ossman
2008-01-11 14:21 ` Rene Herman
2008-01-11 14:21 ` [alsa-devel] " Rene Herman
2008-01-11 18:40 ` Ondrej Zary
2008-01-11 18:40 ` [alsa-devel] " Ondrej Zary
2008-01-12 1:23 ` Rene Herman
2008-01-12 1:23 ` Rene Herman
2008-01-12 1:23 ` [alsa-devel] " Rene Herman
2008-01-12 11:12 ` Pierre Ossman
2008-01-12 11:12 ` Pierre Ossman
2008-01-12 11:12 ` [alsa-devel] " Pierre Ossman
2008-01-12 13:39 ` Rene Herman
2008-01-12 13:39 ` [alsa-devel] " Rene Herman
2008-01-12 15:21 ` Pierre Ossman
2008-01-12 15:21 ` Pierre Ossman
2008-01-12 15:21 ` [alsa-devel] " Pierre Ossman
2008-01-12 16:46 ` Ondrej Zary
2008-01-12 16:46 ` Ondrej Zary
2008-01-12 16:46 ` [alsa-devel] " Ondrej Zary
2008-01-12 17:00 ` Rene Herman
2008-01-12 17:00 ` [alsa-devel] " Rene Herman
2008-01-12 19:08 ` Rafael J. Wysocki
2008-01-12 19:08 ` Rafael J. Wysocki
2008-01-12 20:08 ` -mm: pnp-do-not-stop-start-devices-in-suspend-resume-path.patch breaks resuming isapnp cards Rene Herman
2008-01-12 20:08 ` Rene Herman
2008-01-12 20:08 ` Rene Herman
2008-01-13 5:50 ` Bjorn Helgaas
2008-01-13 5:50 ` Bjorn Helgaas
2008-01-13 6:13 ` Rene Herman
2008-01-13 6:13 ` Rene Herman
2008-01-14 22:26 ` Bjorn Helgaas
2008-01-14 22:26 ` Bjorn Helgaas
2008-01-14 23:46 ` Rene Herman
2008-01-14 23:46 ` Rene Herman
2008-01-14 23:46 ` Rene Herman
2008-01-15 7:51 ` Jaroslav Kysela
2008-01-15 7:51 ` Jaroslav Kysela
2008-01-16 17:46 ` Bjorn Helgaas
2008-01-16 17:46 ` Bjorn Helgaas
2008-01-16 18:03 ` Ondrej Zary
2008-01-16 18:03 ` Ondrej Zary
2008-01-16 18:16 ` Rene Herman
2008-01-16 18:16 ` Rene Herman
2008-01-16 18:16 ` Rene Herman
2008-01-16 17:46 ` Bjorn Helgaas
2008-01-15 7:51 ` Jaroslav Kysela
2008-01-14 22:26 ` Bjorn Helgaas
2008-01-13 6:13 ` Rene Herman
2008-01-13 5:50 ` Bjorn Helgaas
2008-01-12 17:00 ` [alsa-devel] PNP_DRIVER_RES_DISABLE breaks swsusp at least with snd_cs4236 Rene Herman
2008-01-12 19:01 ` Rafael J. Wysocki
2008-01-12 19:01 ` Rafael J. Wysocki
2008-01-12 13:39 ` Rene Herman
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=200801092343.48726.linux@rainbow-software.org \
--to=linux@rainbow-software.org \
--cc=alsa-devel@alsa-project.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.