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
prev 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.