public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Oliver Neukum <oliver@neukum.org>
Cc: Marcel Holtmann <marcel@holtmann.org>,
	Mario Limonciello <mario_limonciello@dell.com>,
	linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Explicitly disable BT radio using rfkill interface on suspend
Date: Tue, 19 May 2009 13:58:53 +0100	[thread overview]
Message-ID: <20090519125853.GA12483@srcf.ucam.org> (raw)
In-Reply-To: <200905191447.34000.oliver@neukum.org>

On Tue, May 19, 2009 at 02:47:33PM +0200, Oliver Neukum wrote:
> Am Dienstag, 19. Mai 2009 11:06:12 schrieb Matthew Garrett:
> > On Mon, May 18, 2009 at 10:14:24PM -0700, Marcel Holtmann wrote:
> > > Hi Matthew,
> > >
> > > > Ok. I don't think the right way to implement this is forcing
> > > > dell-laptop to simulate an unplug. The HCI driver should be responsible
> > > > for saving and restoring state of the device in its suspend/resume
> > > > method.
> > >
> > > what do you expect the generic USB driver to do? I don't see anything we
> > > can do about it, if the device gets removed from underneath us.
> >
> > If the device is getting removed from underneath you, why isn't hid2hci
> > being run again when it reappears as the hid device?
> 
> udev triggers exactly that.

It doesn't seem to for Mario, otherwise this patch wouldn't change 
anything.

-- 
Matthew Garrett | mjg59@srcf.ucam.org

  reply	other threads:[~2009-05-19 12:59 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-18 20:12 [PATCH] Explicitly disable BT radio using rfkill interface on suspend Mario Limonciello
2009-05-18 21:02 ` Matthew Garrett
2009-05-18 21:36   ` Mario Limonciello
2009-05-19  5:03     ` Matthew Garrett
2009-05-19  5:14       ` Marcel Holtmann
2009-05-19  6:27         ` Oliver Neukum
2009-05-19  6:45           ` Marcel Holtmann
2009-05-19  7:36             ` Oliver Neukum
2009-05-19  9:06         ` Matthew Garrett
2009-05-19 12:47           ` Oliver Neukum
2009-05-19 12:58             ` Matthew Garrett [this message]
2009-05-19 16:30               ` Mario Limonciello
2009-05-19 16:33                 ` Matthew Garrett
2009-05-19 16:52                   ` Mario Limonciello
2009-05-19 19:58                     ` Oliver Neukum
2009-05-19 20:36                       ` Mario Limonciello
2009-05-19 21:26                         ` Oliver Neukum
2009-05-19 21:47                           ` Mario Limonciello
2009-05-19 22:16                             ` Oliver Neukum
2009-05-19 22:19                               ` Mario Limonciello
2009-05-20  2:17                               ` Alan Stern
2009-05-20 12:29                                 ` Oliver Neukum
2009-05-20 14:17                                   ` Alan Stern
2009-05-20 17:52                                     ` Mario Limonciello
2009-05-20 18:31                                       ` Alan Stern

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=20090519125853.GA12483@srcf.ucam.org \
    --to=mjg59@srcf.ucam.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcel@holtmann.org \
    --cc=mario_limonciello@dell.com \
    --cc=oliver@neukum.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