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

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

[Macpaul Lin]

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>
---
 drivers/mtd/spi/macronix.c |   16 ++++++++++++++++
 1 files changed, 16 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
index ff66f2a..90aa657 100644
--- a/drivers/mtd/spi/macronix.c
+++ b/drivers/mtd/spi/macronix.c
@@ -71,6 +71,22 @@ static inline struct macronix_spi_flash *to_macronix_spi_flash(struct spi_flash
 
 static const struct macronix_spi_flash_params macronix_spi_flash_table[] = {
 	{
+		.idcode = 0x2013,
+		.page_size = 256,
+		.pages_per_sector = 16,
+		.sectors_per_block = 16,
+		.nr_blocks = 8,
+		.name = "MX25L4005",
+	},
+	{
+		.idcode = 0x2014,
+		.page_size = 256,
+		.pages_per_sector = 16,
+		.sectors_per_block = 16,
+		.nr_blocks = 16,
+		.name = "MX25L8005",
+	},
+	{
 		.idcode = 0x2015,
 		.page_size = 256,
 		.pages_per_sector = 16,
-- 
1.7.3.5

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

[Mike Frysinger]

On Wed, Apr 20, 2011 at 10:51 PM, Macpaul Lin wrote:
> 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

thanks, added to my sf branch
-mike

[U-Boot] [PATCH v2] sf: macronix: add MX25L4005 and MX25L8005

[Mike Frysinger]

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

 drivers/mtd/spi/macronix.c |   16 ++++++++++++++++
 1 files changed, 16 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
index ff66f2a..90aa657 100644
--- a/drivers/mtd/spi/macronix.c
+++ b/drivers/mtd/spi/macronix.c
@@ -71,6 +71,22 @@ static inline struct macronix_spi_flash *to_macronix_spi_flash(struct spi_flash
 
 static const struct macronix_spi_flash_params macronix_spi_flash_table[] = {
 	{
+		.idcode = 0x2013,
+		.page_size = 256,
+		.pages_per_sector = 16,
+		.sectors_per_block = 16,
+		.nr_blocks = 8,
+		.name = "MX25L4005",
+	},
+	{
+		.idcode = 0x2014,
+		.page_size = 256,
+		.pages_per_sector = 16,
+		.sectors_per_block = 16,
+		.nr_blocks = 16,
+		.name = "MX25L8005",
+	},
+	{
 		.idcode = 0x2015,
 		.page_size = 256,
 		.pages_per_sector = 16,
-- 
1.7.6

[U-Boot] [PATCH v2] sf: macronix: add MX25L4005 and MX25L8005

[Wolfgang Denk]

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?



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There's an old story about the person who wished his computer were as
easy to use as his telephone. That wish has come  true,  since  I  no
longer know how to use my telephone.

[U-Boot] [PATCH v2] sf: macronix: add MX25L4005 and MX25L8005

[Graeme Russ]

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

[U-Boot] [PATCH v2] sf: macronix: add MX25L4005 and MX25L8005

[Mike Frysinger]

On Sat, Jul 9, 2011 at 01:16, Wolfgang Denk wrote:
> 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

you already made it clear that you NAK-ed all of the patches posted on
June 28th.  and since you refused to take any pull requests of the
original patches (again, having nothing to do with the patches posted
on June 28th), my only recourse is to send the few that had minor
modifications to the changelog in response to the original patch that
they originated from (previous to June 28th) with the "reply-to-id"
set and the changelog of the update.

i really have nfc what your logic is here.  i cant send pull requests
of the original patches, you NAK updated patches lacking
reply-to-id/history (i.e. the patch series on June 28th), and when i
send patches with reply-to-id/history to the original patches, you
start rejecting those too.  at this point, you've blocked every avenue
possible for me to post patches.  ive shown logic behind my actions
and rather than point out exactly what you have a problem with, you
vaguely refer to "the rules" which really doesnt help at all.
-mike

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

[Wolfgang Denk]

Dear Macpaul Lin,

In message <1303354299-26484-1-git-send-email-macpaul@andestech.com> you wrote:
> 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>
> ---
>  drivers/mtd/spi/macronix.c |   16 ++++++++++++++++
>  1 files changed, 16 insertions(+), 0 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The universe does not have laws - it has habits, and  habits  can  be
broken.