Add chip Pm39LV512

[Kyösti Mälkki]

This flash is like PMC model Pm39LV010 but capacity is 64kB.
Model ID was already defined, PREW works for me.

Signed-off-by: Kyösti Mälkki

Index: flashrom-pmc64k/flashchips.c
===================================================================
--- flashrom-pmc64k/flashchips.c	(revision 1538)
+++ flashrom-pmc64k/flashchips.c	(working copy)
@@ -5267,6 +5267,36 @@

{
    	.vendor		= "PMC",
+		.name		= "Pm39LV512",
+		.bustype	= BUS_PARALLEL,
+		.manufacture_id	= PMC_ID_NOPREFIX,
+		.model_id	= PMC_PM39LV512,
+		.total_size	= 64,
+		.page_size	= 4096,
+		.feature_bits	= FEATURE_ADDR_2AA | FEATURE_EITHER_RESET,
+		.tested		= TEST_OK_PREW,
+		.probe		= probe_jedec,
+		.probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
+		.block_erasers	=
+		{
+			{
+				.eraseblocks = { {4 * 1024, 16} },
+				.block_erase = erase_sector_jedec,
+			}, {
+				.eraseblocks = { {64 * 1024, 1} },
+				.block_erase = erase_block_jedec,
+			}, {
+				.eraseblocks = { {64 * 1024, 1} },
+				.block_erase = erase_chip_block_jedec,
+			}
+		},
+		.write		= write_jedec_1,
+		.read		= read_memmapped,
+		.voltage	= {2700, 3600},
+	},
+
+	{
+		.vendor		= "PMC",
    	.name		= "Pm39LV010",
    	.bustype	= BUS_PARALLEL,
    	.manufacture_id	= PMC_ID_NOPREFIX,

[Paul Menzel]

Dear Kyösti,

Am Samstag, den 19.05.2012, 22:09 +0300 schrieb Kyösti Mälkki:
> This flash is like PMC model Pm39LV010 but capacity is 64kB.

could you add a link to the data sheet to the commit message?
> Model ID was already defined, PREW works for me.

Could you also sent the output of `flashrom -Vr` and the other commends
to the list for archival purposes.
> Signed-off-by: Kyösti Mälkki
> Index: flashrom-pmc64k/flashchips.c
>
> --- flashrom-pmc64k/flashchips.c	(revision 1538)
> +++ flashrom-pmc64k/flashchips.c	(working copy)
> @@ -5267,6 +5267,36 @@
>
> {
>    	.vendor		= "PMC",
> .name		= "Pm39LV512",

If this list is sorted, it should go after »Pm39LV010«.
> .bustype	= BUS_PARALLEL,
> .manufacture_id	= PMC_ID_NOPREFIX,
> .model_id	= PMC_PM39LV512,
> .total_size	= 64,
> .page_size	= 4096,
> .feature_bits	= FEATURE_ADDR_2AA | FEATURE_EITHER_RESET,
> .tested		= TEST_OK_PREW,
> .probe		= probe_jedec,
> .probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
> .block_erasers	=
> {
> 	{
> 		.eraseblocks = { {4 * 1024, 16} },
> 		.block_erase = erase_sector_jedec,
> 	}, {
> 		.eraseblocks = { {64 * 1024, 1} },
> 		.block_erase = erase_block_jedec,
> 	}, {
> 		.eraseblocks = { {64 * 1024, 1} },
> 		.block_erase = erase_chip_block_jedec,
> 	}
> },
> .write		= write_jedec_1,
> .read		= read_memmapped,
> .voltage	= {2700, 3600},
> },
> {
> .vendor		= "PMC",
> .name		= "Pm39LV010",
> .bustype	= BUS_PARALLEL,
> .manufacture_id	= PMC_ID_NOPREFIX,

With the above comments addressed, you can add

Acked-by: Paul Menzel

to the [PATCH v2].

Thanks,

Paul

[Kyösti Mälkki]

On Sat, 2012-05-19 at 21:50 +0200, Paul Menzel wrote:
> Dear Kyösti,
>
> Am Samstag, den 19.05.2012, 22:09 +0300 schrieb Kyösti Mälkki:
> > This flash is like PMC model Pm39LV010 but capacity is 64kB.
>
> could you add a link to the data sheet to the commit message?

Done. I found no official site, though.
> > Model ID was already defined, PREW works for me.
>
> Could you also sent the output of `flashrom -Vr` and the other commends
> to the list for archival purposes.

I will do so. I used IT8212F PCI PATA card as a programmer, need to
submit that patch too.
> > Signed-off-by: Kyösti Mälkki
> > Index: flashrom-pmc64k/flashchips.c
> >
> > --- flashrom-pmc64k/flashchips.c	(revision 1538)
> > +++ flashrom-pmc64k/flashchips.c	(working copy)
> > @@ -5267,6 +5267,36 @@
> >
> > {
> >    	.vendor		= "PMC",
> > .name		= "Pm39LV512",
>
> If this list is sorted, it should go after »Pm39LV010«.

Done. It's after Pm39LV040 now.
> > .bustype	= BUS_PARALLEL,
> > .manufacture_id	= PMC_ID_NOPREFIX,
> > .model_id	= PMC_PM39LV512,
> > .total_size	= 64,
> > .page_size	= 4096,
> > .feature_bits	= FEATURE_ADDR_2AA | FEATURE_EITHER_RESET,
> > .tested		= TEST_OK_PREW,
> > .probe		= probe_jedec,
> > .probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
> > .block_erasers	=
> > {
> > 	{
> > 		.eraseblocks = { {4 * 1024, 16} },
> > 		.block_erase = erase_sector_jedec,
> > 	}, {
> > 		.eraseblocks = { {64 * 1024, 1} },
> > 		.block_erase = erase_block_jedec,
> > 	}, {
> > 		.eraseblocks = { {64 * 1024, 1} },
> > 		.block_erase = erase_chip_block_jedec,
> > 	}
> > },
> > .write		= write_jedec_1,
> > .read		= read_memmapped,
> > .voltage	= {2700, 3600},
> > },
> > {
> > .vendor		= "PMC",
> > .name		= "Pm39LV010",
> > .bustype	= BUS_PARALLEL,
> > .manufacture_id	= PMC_ID_NOPREFIX,
>
> With the above comments addressed, you can add
> Acked-by: Paul Menzel <[e-mail]>
>
> to the [PATCH v2].

Done.
> Thanks,
>
> Paul
> _______________________________________________
> flashrom mailing list
> [e-mail]
> http://www.flashrom.org/mailman/listinfo/flashrom

[Kyösti Mälkki]

This flash is like PMC model Pm39LV010 but capacity is 64kB.
Model ID was already defined, PREW works for me.

http://www.alldatasheet.com/datasheet-pdf/pdf/PMC/PM39LV512-70JC.html

Signed-off-by: Kyösti Mälkki 
Acked-by: Paul Menzel

Index: flashrom-pmc64k/flashchips.c
===================================================================
--- flashrom-pmc64k/flashchips.c	(revision 1538)
+++ flashrom-pmc64k/flashchips.c	(working copy)
@@ -5354,9 +5354,39 @@
    	.read		= read_memmapped,
    	.voltage	= {2700, 3600},
    },
-	
+
    {
    	.vendor		= "PMC",
+		.name		= "Pm39LV512",
+		.bustype	= BUS_PARALLEL,
+		.manufacture_id	= PMC_ID_NOPREFIX,
+		.model_id	= PMC_PM39LV512,
+		.total_size	= 64,
+		.page_size	= 4096,
+		.feature_bits	= FEATURE_ADDR_2AA | FEATURE_EITHER_RESET,
+		.tested		= TEST_OK_PREW,
+		.probe		= probe_jedec,
+		.probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
+		.block_erasers	=
+		{
+			{
+				.eraseblocks = { {4 * 1024, 16} },
+				.block_erase = erase_sector_jedec,
+			}, {
+				.eraseblocks = { {64 * 1024, 1} },
+				.block_erase = erase_block_jedec,
+			}, {
+				.eraseblocks = { {64 * 1024, 1} },
+				.block_erase = erase_chip_block_jedec,
+			}
+		},
+		.write		= write_jedec_1,
+		.read		= read_memmapped,
+		.voltage	= {2700, 3600},
+	},
+
+	{
+		.vendor		= "PMC",
    	.name		= "Pm49FL002",
    	.bustype	= BUS_LPC | BUS_FWH, /* A/A Mux */
    	.manufacture_id	= PMC_ID_NOPREFIX,

[Stefan Tauner]

On Sun, 20 May 2012 13:58:09 +0300
Kyösti Mälkki  wrote:
> This flash is like PMC model Pm39LV010 but capacity is 64kB.
> Model ID was already defined, PREW works for me.
>
> http://www.alldatasheet.com/datasheet-pdf/pdf/PMC/PM39LV512-70JC.html
>
> Signed-off-by: Kyösti Mälkki 
> Acked-by: Paul Menzel

thanks for the patch!
applied with minor changes to the commit message in r1539.
the link to the datasheet got kicked because i dont think it makes a
lot of sense to link to sites that are easily replaceable with any
other hit by your favourite search engine.