Denis, Your approach would have been fine as well. Some mechanism to point out where the original lives like pasting a link saying 'Original code can be found in <url>" or "Based on the code as found in <url>" as you did would have also been enough as well. --Dima On Sat, Jan 22, 2011 at 10:07 AM, Denis 'GNUtoo' Carikli <GNUtoo@xxxxxxxxxx> wrote: > On Sat, 2011-01-22 at 18:28 +0100, Thomas Gleixner wrote: >> I really appreciate the well structured effort which Daniel is putting >> into this. > Me too. > > > I've an issue similar to the problem mentioned in this thread: > I own a board which came,by default with a 2.6.27 kernel. > A developer of the company that produced and sell the board ported the > board to the 2.6.30 kernel during his spare time(if I remember well) > The patch is publicly available in their svn tree in the form of a > standard unified diff patch(not a git patch) > > Is that the proper format for the commit message: > mx31: add support for the bugbase 1.3 from buglabs > > This work was based on bug-linux-2.6.30.patch that can be found > in buglabs's svn here: > svn://bugcamp.net/bug/branches/izzy/experimental > > Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@xxxxxxxxxx> > > The work I did was porting forward the board initialization code and the > serial port support from 2.6.30 to linux-next. > I debugged(with md in uboot to get the printk buffer) the fact that the > serial didn't work,and removed unneeded code to make it work. > > I've not submitted yet the work because of this thread, > thanks to Russell King's response I think I'll submit it. > > > Here's another commit I made and that went into mainline: > commit 9df86e2e702c6d5547aced7f241addd2d698bb11 > Author: Denis 'GNUtoo' Carikli <GNUtoo@xxxxxxxxxx> > Date: Fri Aug 27 23:48:19 2010 +0200 > > wl1251: Fix queue stopping/waking for TX path > > This patch was adapted from 06f7bc7db79fabe6b2ec16eff0f59e4acc21eb72 > (from linus's linux-2.6 tree of kernel.org) > > here's the original message: > The queue stopping/waking functionality was broken in a way that > could > cause huge latencies in TX transfers and even cause the TX to stall > in the > right circumstances. Correct these problems. > > Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@xxxxxxxxxx> > Acked-by: Kalle Valo <kvalo@xxxxxxxxxx> > Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx> > > In this commit I was told not to put the original sign-off, authors > etc... > This is because it would have been confusing and misleading: > It would have appeared like if the commit got Ack,Sign-off etc... by the > people involved in the original commit. > > Denis. > > > -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html