From: David Miller <davem@davemloft.net>
To: gregkh@suse.de
Cc: stern@rowland.harvard.edu, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: usb_autopm_*() removal build failure...
Date: Fri, 11 Dec 2009 21:47:14 -0800 (PST) [thread overview]
Message-ID: <20091211.214714.41966763.davem@davemloft.net> (raw)
In-Reply-To: <20091212040524.GA11212@suse.de>
From: Greg KH <gregkh@suse.de>
Date: Fri, 11 Dec 2009 20:05:24 -0800
> Here's the patch, sent back in August:
> http://marc.info/?l=linux-netdev&m=125071045301270&w=2
>
> Can you take it from there, or do you need me to forward it on?
I tossed this into the net-2.6 tree, but...
I would recommend that in the future when you have the
dependency of an API change that's going to break the
build, you take in the necessary change. And you make
sure such a commit is in your tree before the commit
that removes the API.
That posting had all the markings of something that wasn't
my business applying:
1) the subject said "usb" and nothing about networking or
a specific networking driver
2) it was resolving an API issue introduced by another tree
This also means you weren't doing allmodconfig builds on your USB tree
this entire time, because if you had you would have realized that this
fix belonged there not in net-2.6 :-) And if the USB bits were
going in via -next I'm really surprised Stephen Rothwell wasn't
complaining about this too :)
Anyways, next time, please resolve API issues internally when you
do things that will break the build otherwise.
We can merge it all together during the merge window just fine.
Thanks!
next prev parent reply other threads:[~2009-12-12 5:47 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-12 1:45 David Miller
2009-12-12 3:52 ` Greg KH
2009-12-12 4:05 ` Greg KH
2009-12-12 5:47 ` David Miller [this message]
2009-12-12 6:46 ` Greg KH
2009-12-12 6:56 ` Stephen Rothwell
2009-12-12 6:59 ` David Miller
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=20091211.214714.41966763.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=stern@rowland.harvard.edu \
/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
all inboxes | Powered by JetHome®