All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Ritz <daniel.ritz@gmx.ch>
To: Greg KH <greg@kroah.com>
Cc: David Brownell <david-b@pacbell.net>,
	linux-kernel@vger.kernel.org,
	linux-usb-devel@lists.sourceforge.net, rjw@sisk.pl,
	akpm@osdl.org, Linus Torvalds <torvalds@osdl.org>
Subject: Re: [PATCH] usb/core/hcd-pci.c: don't free_irq() on suspend
Date: Sat, 1 Oct 2005 22:46:56 +0200	[thread overview]
Message-ID: <200510012246.57783.daniel.ritz@gmx.ch> (raw)
In-Reply-To: <20050930233833.GA19471@kroah.com>

[ restored some of the cc: list of the original thread ]

On Saturday 01 October 2005 01.38, Greg KH wrote:
> <top-post on purpose...>
> 
> Daniel, are you sure about this patch (the second part specifically)?

well, the first hunk doesn't make sense without the second and vice versa,
isn't it? to answer your question: yes.

> It directly conflicts with a set of patches in my current tree in this
> area that fix all of the reported suspend/resume issues with usb host
> controllers (that patch series written by David Brownell.)
> 
> Yeah, I see that we shouldn't have been dropping the irq on suspend and
> getting a new one on resume, that's not good and could have caused
> problems for people.
> 
> But could you at least drop the linux-usb-devel mailing list a note that
> you are having issues, and post the proposed patch?  Directly sending it

i don't have problems, it was rafael...i just happend to look at it because
yenta_socket was involved...see the following link for more background:
	http://marc.theaimsgroup.com/?t=112275164900002&r=1&w=4

> to Linus is a bit rude, it's not like the USB developers aren't

sorry for that, but i actually asked for a round in -mm. it just happend
that linus was on to: and the rest on cc: by pressing reply-to-all in kmail

the original thread
	http://marc.theaimsgroup.com/?t=112618280600003&r=1&w=4

> responsive to emails (yeah, I've been a bit slow at times these past few
> weeks, but my traveling all over the place for the past month is now
> over, and I'm not going anywhere for a long time...)
> 
> David, this conflicts with your usb/usb-pm-06.patch in my quilt tree.
> I'll try to resolve the merge with my best guess, but you should check
> that I got it right...
> 
> thanks,
> 
> greg k-h
> 

rgds
-daniel

      parent reply	other threads:[~2005-10-01 20:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200509302101.j8UL1Htj026067@hera.kernel.org>
2005-09-30 23:38 ` [PATCH] usb/core/hcd-pci.c: don't free_irq() on suspend Greg KH
2005-10-01  1:15   ` David Brownell
2005-10-01 20:46   ` Daniel Ritz [this message]

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=200510012246.57783.daniel.ritz@gmx.ch \
    --to=daniel.ritz@gmx.ch \
    --cc=akpm@osdl.org \
    --cc=david-b@pacbell.net \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=rjw@sisk.pl \
    --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 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.