public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: tmc-+XVqjN4+fIJfbCVcMT5z6Ydd74u8MsAO@public.gmane.org (Tomasz Ciolek)
To: Ilya Bagrak <reason-ss4n/k5RZhaUArH+D2fLrA@public.gmane.org>
Cc: brewt-acpi.sf.net-CDLCgzMZgTDYtjvyW6yDsg@public.gmane.org,
	acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: S3 suspend/resume driver questions
Date: Tue, 20 Apr 2004 11:46:14 +1000	[thread overview]
Message-ID: <20040420014614.GN8823@dreamcraft.com.au> (raw)
In-Reply-To: <32839.66.245.64.93.1082425332.squirrel-2RFepEojUI2yzif+TlFmFpQCsf4PZ8us@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 5062 bytes --]

 Hooray.
 
 If only someone at ATI or Fujitsu finnaly mamanged to cough up
 the magic incantations for the radeon cards so that the POST
 correctly... 

 Regards
 TMC

On Mon, Apr 19, 2004 at 09:42:12PM -0400, Ilya Bagrak wrote:
> Adrian,
> 
> Your patch worked beautifully on my Fujitsu E7010 laptop.  Prior to the
> patch, the system would lock up hard if I tried to load snd_intel8x0 after
> waking up from S3.  If I didn't not unload the sound modules before going
> to sleep, the system would not wake up at all (lock up would happen on
> resume).  In any case, the sound "just works" (tm) now.
> 
> There is more good news.  I adapted your patch to orinoco_pci driver which
> manages my built in Prism2 wireless adapter.  The unpatched driver failed
> to reinit the card upon resume.  Even though that didn't not lock up my
> computer like sound did, the card was not functional.  I was absolutely
> stunned when this 3-liner-of-a-patch returned full functionality to the
> card upon resume.  You don't even have to unload the driver before going
> to sleep.  I have a little signal strength applet running in the toolbar,
> and you can watch the signal strength gradually increase after wakeup and
> the card is back to normal in a couple of seconds.
> 
> The wireless NIC was the last stumbling block in getting S3 to work.  I
> now officially declare Fujitsu E7010 Linux ACPI-friendly.  All major
> devices (have not tested Firewire or IrDA) recover and work properly after
> S3. Thank you all.
> 
> Ilya
> 
> > Hi,
> >
> >> mine isn't a technical reply, but you can post in the 'alsa-devel'
> >> mailing-list, as it seems that Takashi Iwai is working on PM-lacking
> >> drivers. Here some links:
> >>    http://sourceforge.net/mailarchive/forum.php?thread_id=4201761&forum_id=1752
> >
> > Doing a grep through the alsa code, it doesn't look like any driver
> > save/restores pci state.
> >
> >>    http://sourceforge.net/mailarchive/forum.php?thread_id=4107033&forum_id=1752
> >>
> >> BTW-1, could you post your 'intel8x0' patch? I could give it a try
> >> ASAP...
> >
> > That's essentially what I did.  It is a little different though:
> >
> > --- /usr/src/linux-2.6.5-mm6/sound/pci/intel8x0.c       2004-04-05
> > 18:46:17.000000000 -0700 +++ intel8x0.c  2004-04-18 04:03:34.000000000
> > -0700
> > @@ -445,6 +445,7 @@ struct _snd_intel8x0 {
> >
> >  #ifdef CONFIG_PM
> >         int in_suspend;
> > +       u32 pci_state[16];
> >  #endif
> >  };
> >
> > @@ -2223,11 +2224,13 @@ static int snd_intel8x0_suspend(struct p
> >  {
> >         intel8x0_t *chip = snd_magic_cast(intel8x0_t,
> > pci_get_drvdata(dev), return -ENXIO); intel8x0_suspend(chip);
> > +       pci_save_state(dev, chip->pci_state);
> >         return 0;
> >  }
> >  static int snd_intel8x0_resume(struct pci_dev *dev)
> >  {
> >         intel8x0_t *chip = snd_magic_cast(intel8x0_t,
> > pci_get_drvdata(dev), return -ENXIO);
> > +       pci_restore_state(dev, chip->pci_state);
> >         intel8x0_resume(chip);
> >         return 0;
> >  }
> >
> >> BTW-2, in the few tests I made on my ASUS M6842NWH ('intel8x0' based),
> >> I have to remove the sound modules even to have a successfull S1.
> >
> > Actually, I haven't tried out S1 yet, but I don't have to remove the
> > sound modules for S3 to work.  And even if I do, my system will lock up
> > when trying to load the alsa drivers due to the device's configuration
> > state.
> >
> > Adrian
> >
> >
> > -------------------------------------------------------
> > This SF.Net email is sponsored by: IBM Linux Tutorials
> > Free Linux tutorial presented by Daniel Robbins, President and CEO of
> > GenToo technologies. Learn everything from fundamentals to system
> > administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&op?k
> > _______________________________________________
> > Acpi-devel mailing list
> > Acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> > https://lists.sourceforge.net/lists/listinfo/acpi-devel
> 
> 
> 
> 
> 
> -------------------------------------------------------
> This SF.Net email is sponsored by: IBM Linux Tutorials
> Free Linux tutorial presented by Daniel Robbins, President and CEO of
> GenToo technologies. Learn everything from fundamentals to system
> administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&op=click
> _______________________________________________
> Acpi-devel mailing list
> Acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> https://lists.sourceforge.net/lists/listinfo/acpi-devel

-- 
Tomasz M. Ciolek	
*******************************************************************************
 tmc at dreamcraft dot com dot au 
*******************************************************************************
   GPG Key ID:		0x41C4C2F0
   GPG Key Fingerprint: 3883 B308 8256 2246 D3ED  A1FF 3A1D 0EAD 41C4 C2F0
   Key available on www.pgp.net	
*******************************************************************************

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

  parent reply	other threads:[~2004-04-20  1:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-20  1:42 S3 suspend/resume driver questions Ilya Bagrak
     [not found] ` <32839.66.245.64.93.1082425332.squirrel-2RFepEojUI2yzif+TlFmFpQCsf4PZ8us@public.gmane.org>
2004-04-20  1:46   ` Tomasz Ciolek [this message]
     [not found]     ` <20040420014614.GN8823-+XVqjN4+fIJfbCVcMT5z6Ydd74u8MsAO@public.gmane.org>
2004-04-20  8:52       ` ole.rohne-vJEk5272eHo
2004-04-20  2:29   ` Adrian Yee
2004-04-20 14:01   ` Pavel Machek
  -- strict thread matches above, loose matches on Subject: below --
2004-04-20  2:47 Ilya Bagrak
     [not found] ` <33087.66.245.64.93.1082429245.squirrel-2RFepEojUI2yzif+TlFmFpQCsf4PZ8us@public.gmane.org>
2004-04-20  2:52   ` Tomasz Ciolek
     [not found]     ` <20040420025220.GA5577-+XVqjN4+fIJfbCVcMT5z6Ydd74u8MsAO@public.gmane.org>
2004-04-20  3:15       ` Adrian Yee
     [not found]         ` <GMail.1082430934.185927489.4271239555-CDLCgzMZgTDYtjvyW6yDsg@public.gmane.org>
2004-04-20  3:25           ` Tomasz Ciolek
2004-04-20  3:09   ` Adrian Yee
     [not found]     ` <GMail.1082430576.18596822.096463261417-CDLCgzMZgTDYtjvyW6yDsg@public.gmane.org>
2004-04-20  3:16       ` Tomasz Ciolek
2004-04-18 10:06 Adrian Yee
     [not found] ` <GMail.1082282809.299052003.09816679024-CDLCgzMZgTDYtjvyW6yDsg@public.gmane.org>
2004-04-18 10:50   ` Luca Capello
     [not found]     ` <40825D85.1000702-wlebWZzHoyE@public.gmane.org>
2004-04-18 11:16       ` Adrian Yee

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=20040420014614.GN8823@dreamcraft.com.au \
    --to=tmc-+xvqjn4+fijfbcvcmt5z6ydd74u8msao@public.gmane.org \
    --cc=acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=brewt-acpi.sf.net-CDLCgzMZgTDYtjvyW6yDsg@public.gmane.org \
    --cc=reason-ss4n/k5RZhaUArH+D2fLrA@public.gmane.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