From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Rostedt Subject: Re: [PATCH 0/7] Nexus One Support Date: Fri, 21 Jan 2011 20:58:21 -0500 Message-ID: <20110122015821.GA6249@home.goodmis.org> References: <20110121094827.41818a55@jbarnes-desktop> <20110121095658.1ab623fe@jbarnes-desktop> <1295632828.19880.22.camel@m0nster> <20110121100441.06a94482@jbarnes-desktop> <1295633882.19880.31.camel@m0nster> <1295642995.19880.42.camel@m0nster> <20110121234930.GK3043@thunk.org> <1295654593.22882.9.camel@m0nster> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from hrndva-omtalb.mail.rr.com ([71.74.56.123]:35505 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754228Ab1AVB6Z (ORCPT ); Fri, 21 Jan 2011 20:58:25 -0500 Content-Disposition: inline In-Reply-To: <1295654593.22882.9.camel@m0nster> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Daniel Walker Cc: Ted Ts'o , Pekka Enberg , Dima Zavin , Jesse Barnes , Joe Perches , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, davidb@codeaurora.org On Fri, Jan 21, 2011 at 04:03:13PM -0800, Daniel Walker wrote: > > right, but it wasn't a cherrypick which was explain in the thread. So > there's no wrongs here .. I'm sorry Daniel, but you are absolutely wrong! Did you come up with the logic of this code, or did you take someone else's code and just massage it until it worked? If you took someone else's code, where are their Signed-off-by's? Just look at what I did recently with Lai's rt-mutex clean up patch. I ported it to the PREEMPT_RT (-rt) patch set. That port took me a week to do. I had to add a bunch of code to make it work. When it was finished, did I claim author of it? No! I left Lai as the author and simply added: [ Ported to the -rt patch set by Steven Rostedt ] Take a look at the changes. Here's the patch I started with: https://patchwork.kernel.org/patch/477841/ Here's what I ended with (all my changes) https://patchwork.kernel.org/patch/470171/ Do a diff of the two. There was a lot of work going into that. But I did not come up with change to the algorithm that Lai did, so I kept him as author. I just added my insert in the change log and added my own SoB (including his). If I repaint a Picasso, can I call it a Rostedt? -- Steve