On 10-12-2011 00:25, Devin Heitmueller wrote:
Hello Andreas,
On Fri, Dec 9, 2011 at 9:06 PM, Andreas Oberritter<obi@xxxxxxxxxxx> wrote:
WTF, Devin, you again? I haven't asked anyone to upstream it. Feel free
to analyze the code and resubmit it.
1. It's marked with a subject line that starts with "[PATCH]"
2. It has your SIgned-Off-By line.
3. it was sent to the mailing list.
4. It doesn't have any keywords like "RFC" or "proposed".
Devin,
You're over-reacting. This patch were a reply from Andreas to a thread,
and not a separate patch submission.
Patches like are generally handled as RFC, especially since it doesn't
contain a description.
If you don't intend for it to go upstream then don't do all of the above.
I'm not sure if your "WTF, Devin, you again?" is some indication that
I'm annoying you. The last patch you submitted that touches the
threading in dvb_frontend.c had a host of problems and was clearly not
well researched (i.e. "DVB: dvb_frontend: convert semaphore to
mutex"). As in this case, there is no background indicating that this
patch has been fully thought out and due diligence has been done.
Maybe you have thoroughly researched the change, taken the time to
fully understand its effects, and tested it with a variety of boards
and scenarios. Without a good patch description, there is no way to
know.
I apologize if you're inconvenienced if I'm making an active effort to
prevent poorly documented changes from getting merged (which often
result in regressions). Oh wait, I'm not sorry at all. Nevermind.
Devin
Regards,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
