[PATCH v4] arm: remove unused code in delay.S
Steve Chen
schen at mvista.com
Sat Sep 19 10:09:48 EDT 2009
On Sat, 2009-09-19 at 15:47 +0200, Uwe Kleine-König wrote:
> Hello Steve,
>
> On Sat, Sep 19, 2009 at 07:55:29AM -0500, Steve Chen wrote:
> > Document #if 0 code block in delay.S and make it selectable for compile.
> >
> > Signed-off-by: Steve Chen <schen at mvista.com>
> > Reviewed-by: Felipe Contreras <felipe.contreras at gmail.com>
> > Reviewed-by: Jamie Lokier <jamie at shareable.org>
> > Reviewed-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> > Reviewed-by: Krzysztof Halasa <khc at pm.waw.pl>
> > Reviewed-by: Marek Vasut <marek.vasut at gmail.com>
> > Reviewed-by: Pavel Machek <pavel at ucw.cz>
> > Reviewed-by: Sergei Shtylyov <sshtylyov at ru.mvista.com>
> > Reviewed-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> I'm very surprised you added these Reviewed-by: tags.
>
> IMO you should not add them without the explicit permission and request
> of the corresponding person. (That means that at least mine should not
> be there.)
>
> Moreover Reviewed-by: has a meaning (described in
> Documentation/SubmittingPatches).
>
My only intention was to give credit to all who contributed (at least try). I already
sent out a patch with all Reviewed-by tags removed. I can add back in anyone who wish
to be added.
Regards,
Steve
More information about the linux-arm-kernel
mailing list