All of lore.kernel.org
 help / color / mirror / Atom feed
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

             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.