public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Graeme Russ <graeme.russ@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] sf: macronix: add MX25L4005 and MX25L8005
Date: Sat, 09 Jul 2011 16:10:45 +1000	[thread overview]
Message-ID: <4E17F0E5.2020106@gmail.com> (raw)
In-Reply-To: <20110709051652.DF00316BA44B@gemini.denx.de>

Hi Wolfgang,

On 09/07/11 15:16, Wolfgang Denk wrote:
> Dear Mike Frysinger,
> 
> In message <1310157072-27512-1-git-send-email-vapier@gentoo.org> you wrote:
>> From: Macpaul Lin <macpaul@andestech.com>
>>
>> Add support of MX25L4005 and MX25L8005 according to the datasheet
>> http://www.mct.net/download/macronix/mx25l8005.pdf
>>
>> This patch has been tested with MX25L4005 and MX25L8005
>>
>> Signed-off-by: Macpaul Lin <macpaul@andestech.com>
>> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
>> ---
>> v2
>> 	- tweak summary
> 
> This is an identical repost of 
> 06/28 Mike Frysinger     [U-Boot] [PATCH 1/9] sf: spansion: add support for S25FL129P_64K
>              http://article.gmane.org/gmane.comp.boot-loaders.u-boot/102129
> 
> 
> Your "changelog":
> 
> 	- tweak summary
> 
> is a lie, as NOTHING changes (except for the "[PATCH 1/9]" versus
> "[PATCH v2]" part).
> 
> 
> I have explained to you MANY times that YOU ARE NOT SUPPOSED to resend
> unchanged patches.
> 
> I feel you are intentionally provoking me.
> 
> Or am I missing somethign?

The 'Subject' (Summary??) has changed:

Old:
[U-Boot] [PATCH] mtd/spi/macronix.c: add MX25L4005 and MX25L8005

New (and imho improved):
[U-Boot] [PATCH v2] sf: macronix: add MX25L4005 and MX25L8005

While the actual patch itself is identical, there is a change to what would
ultimately end up in git - How should we treat such trivial changes? It
seems to me that the maintainers will tweak this kind of detail at their
discretion and simply post an 'Applied with updated summary' to the ML - Is
this your preferred option?

Regards,

Graeme

  reply	other threads:[~2011-07-09  6:10 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-21  2:51 [U-Boot] [PATCH] mtd/spi/macronix.c: add MX25L4005 and MX25L8005 Macpaul Lin
2011-04-21 16:54 ` Mike Frysinger
2011-07-08 20:31 ` [U-Boot] [PATCH v2] sf: macronix: " Mike Frysinger
2011-07-09  5:16   ` Wolfgang Denk
2011-07-09  6:10     ` Graeme Russ [this message]
2011-07-09  6:43     ` Mike Frysinger
2011-08-02 19:56 ` [U-Boot] [PATCH] mtd/spi/macronix.c: " Wolfgang Denk

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=4E17F0E5.2020106@gmail.com \
    --to=graeme.russ@gmail.com \
    --cc=u-boot@lists.denx.de \
    /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