linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: marek.vasut@gmail.com (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/6] AT91 Reset cleanup
Date: Tue, 29 Nov 2011 20:23:19 +0100	[thread overview]
Message-ID: <201111292023.19202.marek.vasut@gmail.com> (raw)
In-Reply-To: <20111129190726.GM9581@n2100.arm.linux.org.uk>

> On Tue, Nov 29, 2011 at 07:46:13PM +0100, Marek Vasut wrote:
> > > On Tue, Nov 29, 2011 at 06:35:14PM +0100, Jean-Christophe
> > > PLAGNIOL-VILLARD
> > 
> > wrote:
> > > > Hi,
> > > > 
> > > > 	the following patch series will merge the ddr controler header 
file
> > > > 	for sam9 and cap9 and cleanup the reset by
> > > > 	
> > > > 	swtich to arm_pm_restart
> > > > 	fix the sam9g45 and cap9 reset
> > > > 	make rstc soc independent
> > > > 
> > > > The following changes since commit 
15327570807209b5c088382af23ed3fc47558d87:
> > > >   ARM: at91/gpio: fix display of number of irq setuped (2011-11-28
> > > >   22:53:09 +0800)
> > > > 
> > > > are available in the git repository at:
> > > >   git://github.com/at91linux/linux-at91.git for-arnd-3.3-reset
> > > > 
> > > > Jean-Christophe PLAGNIOL-VILLARD (6):
> > > >       ARM: at91: fix cap9 ddrsdr register
> > > >       ARM: at91: merge at91cap9_ddrsdr.h in at91sam9_ddrsdr.h
> > > >       ARM: at91: introduce AT91_SAM9_ALT_RESET to select the at91sam9
> > > >       alternative reset ARM: restart: at91: use new restart hook
> > > 
> > > Well, it looks like you've taken my work here and committed it as
> > > yourself. That's plagerism.  Don't do it - and fix it before someone
> > > pulls your crap.
> > 
> > Hey,
> > 
> > looking at the patch in question, it looks so trivial the clash is just
> > possible. Why not just put two SoB lines there and maybe From: and be
> > done with it?
> 
> Look closer.  The patch which was posted has my sign-off on it, it says
> it's from me, but according to the commit summary, it's author is Jean-
> Christophe.
> 
> So, it _is_ my patch which Jean has decided to _recommit_ into his tree
> taking it from my tree _without_ _asking_ _me_ whether it was either a
> good idea to do that, whether it was a finished patch, etc.

Ah ok, I see.
> 
> So all in all I'm pretty disgusted with Jean-Christophe over this.

Oh come on, calm down. Just keep an eye on him so he won't do it again and be 
done with it ;-) And Jean should certainly drop the patch if it's unfinished 
stuff.
> 
> I would like to see my patch updated at some point not to use
> arm_pm_restart, but to use the .restart method directly, like I've been
> doing for everyone else.  In other words, this patch is NOT STABLE at
> the present time.

OK

  reply	other threads:[~2011-11-29 19:23 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-29 17:35 [PATCH 0/6] AT91 Reset cleanup Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:42 ` Russell King - ARM Linux
2011-11-29 17:49   ` Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 18:46   ` Marek Vasut
2011-11-29 19:07     ` Russell King - ARM Linux
2011-11-29 19:23       ` Marek Vasut [this message]
2011-11-29 22:07         ` Russell King - ARM Linux
2011-11-29 22:43           ` Marek Vasut
2011-11-29 22:48             ` Russell King - ARM Linux
2011-11-29 23:03               ` Marek Vasut
2011-11-30  4:46               ` Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:52 ` [PATCH 1/6] ARM: at91: fix cap9 ddrsdr register Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:52 ` [PATCH 2/6] ARM: at91: merge at91cap9_ddrsdr.h in at91sam9_ddrsdr.h Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:52 ` [PATCH 3/6] ARM: at91: introduce AT91_SAM9_ALT_RESET to select the at91sam9 alternative reset Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:52 ` [PATCH 4/6] ARM: restart: at91: use new restart hook Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:52 ` [PATCH 5/6] ARM: at91: Fix at91sam9g45 and at91cap9 reset Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 22:12   ` Russell King - ARM Linux
2011-11-30  4:55     ` Jean-Christophe PLAGNIOL-VILLARD
2011-11-29 17:52 ` [PATCH 6/6] ARM: at91: make rstc soc independent Jean-Christophe PLAGNIOL-VILLARD

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=201111292023.19202.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).