Anastasia Klimchuk has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/83451?usp=email )
The change is no longer submittable: All-Comments-Resolved is unsatisfied now.
Change subject: doc: Add doc for in-system programming
......................................................................
Patch Set 1:
(2 comments)
Patchset:
PS1:
> Oh, I didn't even know we had such a page! […]
This would be fantastic, thank you so much!
PS1:
Wait until the tag v1.4.0 done and after that the patch can be submitted. ETA 26 July.
https://mail.coreboot.org/hyperkitty/list/flashrom@flashrom.org/thread/UUDQ…
I am leaving the comment unresolved, so that not to accidentally submit before the tag.
--
To view, visit https://review.coreboot.org/c/flashrom/+/83451?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: If4752f0f02ae973b3d832f42166de643d95c9f97
Gerrit-Change-Number: 83451
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Fri, 19 Jul 2024 02:23:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Alexander Goncharov <chat(a)joursoir.net>
Attention is currently required from: Anastasia Klimchuk.
Alexander Goncharov has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/83451?usp=email )
Change subject: doc: Add doc for in-system programming
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Patchset:
PS1:
Oh, I didn't even know we had such a page!
Just by accident, I am working with some ISP via a special header that is provided on our motherboard right now. This part is not described in the document. So I can write more about it in the following patch.
--
To view, visit https://review.coreboot.org/c/flashrom/+/83451?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: If4752f0f02ae973b3d832f42166de643d95c9f97
Gerrit-Change-Number: 83451
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Comment-Date: Thu, 18 Jul 2024 17:38:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: Anastasia Klimchuk, Bora Guvendik, Nikolai Artemiev, Stefan Reinauer.
Subrata Banik has posted comments on this change by Bora Guvendik. ( https://review.coreboot.org/c/flashrom/+/82626?usp=email )
Change subject: flashchips: add support for MX77U51250F chip
......................................................................
Patch Set 3:
(3 comments)
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/82626/comment/28adcac8_50bf3e51?us… :
PS3, Line 11618: 65536
I believe this is 64MB SPI flash. please confirm ?
https://review.coreboot.org/c/flashrom/+/82626/comment/67ced4a1_d3ba4065?us… :
PS3, Line 11620: .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_4BA,
it doesn't support FEATURE_WRSR2, FEATURE_WRSR3?
https://review.coreboot.org/c/flashrom/+/82626/comment/3aa290ac_431367c8?us… :
PS3, Line 11648: },
please add below details as well
```
.reg_bits =
{
...
}
.decode_range
```
--
To view, visit https://review.coreboot.org/c/flashrom/+/82626?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I2c2e94f01dc63f60cf636bc6afe1f033e2a6f83c
Gerrit-Change-Number: 82626
Gerrit-PatchSet: 3
Gerrit-Owner: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: DZ <danielzhang(a)mxic.com.cn>
Gerrit-Reviewer: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Attention: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Comment-Date: Thu, 18 Jul 2024 14:24:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Anastasia Klimchuk, Nikolai Artemiev, Peter Marheine, Stefan Reinauer.
Alexander Goncharov has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/83359?usp=email )
Change subject: doc: Release notes for version 1.4.0
......................................................................
Patch Set 4: Code-Review+1
--
To view, visit https://review.coreboot.org/c/flashrom/+/83359?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: Ie5597f1c3ae9289e424f54c2d313fef8efbdf1a0
Gerrit-Change-Number: 83359
Gerrit-PatchSet: 4
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Attention: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Comment-Date: Wed, 17 Jul 2024 19:06:22 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: Anastasia Klimchuk, Bora Guvendik, Nikolai Artemiev, Stefan Reinauer.
Subrata Banik has posted comments on this change by Bora Guvendik. ( https://review.coreboot.org/c/flashrom/+/82626?usp=email )
Change subject: flashchips: add support for MX77U51250F chip
......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS3:
> Yes, I have the datasheet and chip. We were able to flash with this patch.
I don't think all IHV's are mandatory required to upload the doc into the public domain to land the code in flashrom. Are we ? in that case there are many datasheet that are not available openly? Can we get the doc under NDA to get the code review going ?
--
To view, visit https://review.coreboot.org/c/flashrom/+/82626?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I2c2e94f01dc63f60cf636bc6afe1f033e2a6f83c
Gerrit-Change-Number: 82626
Gerrit-PatchSet: 3
Gerrit-Owner: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: DZ <danielzhang(a)mxic.com.cn>
Gerrit-Reviewer: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Attention: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Comment-Date: Wed, 17 Jul 2024 16:36:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Bora Guvendik <bora.guvendik(a)intel.com>
Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org>
Anastasia Klimchuk has uploaded a new patch set (#2). ( https://review.coreboot.org/c/flashrom/+/83471?usp=email )
Change subject: doc: Add doc for buspirate programmer
......................................................................
doc: Add doc for buspirate programmer
Doc migrated from the wiki page:
https://wiki.flashrom.org/Bus_Pirate
Change-Id: I5a57f08ea3fce0c78d73aa61b85ff7b0cff450b8
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
---
A doc/supported_hw/supported_prog/Buspirate_v3_back.jpg
A doc/supported_hw/supported_prog/Buspirate_v3_front.jpg
A doc/supported_hw/supported_prog/Lycom-pe115-flashrom-buspirate-2.jpg
A doc/supported_hw/supported_prog/buspirate.rst
M doc/supported_hw/supported_prog/index.rst
5 files changed, 79 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/71/83471/2
--
To view, visit https://review.coreboot.org/c/flashrom/+/83471?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I5a57f08ea3fce0c78d73aa61b85ff7b0cff450b8
Gerrit-Change-Number: 83471
Gerrit-PatchSet: 2
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/83471?usp=email )
Change subject: doc: Add doc for buspirate programmer
......................................................................
doc: Add doc for buspirate programmer
Doc migrated from the wiki page:
https://wiki.flashrom.org/Bus_Pirate
Change-Id: I5a57f08ea3fce0c78d73aa61b85ff7b0cff450b8
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
---
A doc/supported_hw/supported_prog/Buspirate_v3_back.jpg
A doc/supported_hw/supported_prog/Buspirate_v3_front.jpg
A doc/supported_hw/supported_prog/Lycom-pe115-flashrom-buspirate-2.jpg
A doc/supported_hw/supported_prog/buspirate.rst
M doc/supported_hw/supported_prog/index.rst
5 files changed, 62 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/71/83471/1
diff --git a/doc/supported_hw/supported_prog/Buspirate_v3_back.jpg b/doc/supported_hw/supported_prog/Buspirate_v3_back.jpg
new file mode 100644
index 0000000..3afb974
--- /dev/null
+++ b/doc/supported_hw/supported_prog/Buspirate_v3_back.jpg
Binary files differ
diff --git a/doc/supported_hw/supported_prog/Buspirate_v3_front.jpg b/doc/supported_hw/supported_prog/Buspirate_v3_front.jpg
new file mode 100644
index 0000000..3c4fa9c
--- /dev/null
+++ b/doc/supported_hw/supported_prog/Buspirate_v3_front.jpg
Binary files differ
diff --git a/doc/supported_hw/supported_prog/Lycom-pe115-flashrom-buspirate-2.jpg b/doc/supported_hw/supported_prog/Lycom-pe115-flashrom-buspirate-2.jpg
new file mode 100644
index 0000000..5962fa9
--- /dev/null
+++ b/doc/supported_hw/supported_prog/Lycom-pe115-flashrom-buspirate-2.jpg
Binary files differ
diff --git a/doc/supported_hw/supported_prog/buspirate.rst b/doc/supported_hw/supported_prog/buspirate.rst
new file mode 100644
index 0000000..23ccfaf
--- /dev/null
+++ b/doc/supported_hw/supported_prog/buspirate.rst
@@ -0,0 +1,61 @@
+==========
+Bus Pirate
+==========
+
+The `Bus Pirate <http://dangerousprototypes.com/docs/Bus_Pirate>`_ is an open source design
+for a multi-purpose chip-level serial protocol transceiver and debugger.
+flashrom supports the Bus Pirate for `SPI programming <http://dangerousprototypes.com/docs/SPI>`_.
+It also has `SPI sniffing <http://dangerousprototypes.com/docs/Bus_Pirate_binary_SPI_sniffer_utility>`_
+functionality, which may come in useful for analysing chip or programmer behaviour.
+
+They are available for around US$30 from various sources.
+
+Connections
+===========
+
+The table below shows how a typical SPI flash chip (sitting in the center of the table)
+needs to be connected (NB: not all flash chips feature all of the pins below, but in general
+you should always connect all input pins of ICs to some defined potential (usually GND or VCC),
+ideally with a pull-up/down resistor in between). Most SPI flash chips require a 3.3V supply voltage,
+but there exist some models that use e.g. 1.8V. Make sure the device in question is compatible
+before connecting any wires.
+
+*NB: Some rather rare SPI flash chips (e.g. Atmel AT45DB series) have a completely different layout, please beware.*
+
++----------------------+------------+------+---------------------------------+------+------------+-----------------------------+
+| Description | Bus Pirate | Dir. | Flash chip | Dir. | Bus Pirate | Description |
++======================+============+======+===+===========+=============+===+======+============+=============================+
+| (not) Chip Select | CS | → | 1 | /CS | VCC | 8 | ← | +3.3v | Supply |
++----------------------+------------+------+---+-----------+-------------+---+------+------------+-----------------------------+
+| Master In, Slave Out | MISO | ← | 2 | DO (IO1) | /HOLD (IO3) | 7 | ← | +3.3v | (not) hold (see datasheets) |
++----------------------+------------+------+---+-----------+-------------+---+------+------------+-----------------------------+
+| (not) Write Protect | +3.3v | → | 3 | /WP (IO2) | CLK | 6 | ← | CLK | The SPI clock |
++----------------------+------------+------+---+-----------+-------------+---+------+------------+-----------------------------+
+| Ground | GND | → | 4 | GND | DI (IO0) | 5 | ← | MOSI | Master Out, Slave In |
++----------------------+------------+------+---+-----------+-------------+---+------+------------+-----------------------------+
+
+Usage
+=========
+
+::
+
+ $ flashrom -p buspirate_spi:dev=/dev/device,spispeed=frequency
+
+Example::
+
+ $ flashrom -p buspirate_spi:dev=/dev/ttyUSB0,spispeed=1M
+
+Troubleshooting
+===============
+
+In case of problems probing the chip with flashrom - especially when connecting chips
+still soldered in a system - please take a look at the doc :doc:`/user_docs/in_system`. In-system programming is often possible
+**only as long as no other devices on the SPI bus are trying to access the device**.
+
+Speedup
+=========
+
+A beta firmware build exists, to speed up the buspirate.
+`See this post on dangerousprototypes.com <http://dangerousprototypes.com/forum/viewtopic.php?f=40&t=3864&start=15#p41…>`_
+
+See also: http://dangerousprototypes.com/docs/Bus_Pirate#Firmware_upgrades
diff --git a/doc/supported_hw/supported_prog/index.rst b/doc/supported_hw/supported_prog/index.rst
index 130ac20..ec96b9f 100644
--- a/doc/supported_hw/supported_prog/index.rst
+++ b/doc/supported_hw/supported_prog/index.rst
@@ -15,5 +15,6 @@
.. toctree::
:maxdepth: 1
+ buspirate
dummyflasher
serprog/index
--
To view, visit https://review.coreboot.org/c/flashrom/+/83471?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I5a57f08ea3fce0c78d73aa61b85ff7b0cff450b8
Gerrit-Change-Number: 83471
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Attention is currently required from: Stefan Reinauer.
Anastasia Klimchuk has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/83467?usp=email )
Change subject: doc: Add manpage entries for nic3com, gfxnvidia, satasii
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Wait until the tag v1.4.0 done and after that the patch can be submitted. ETA 26 July.
https://mail.coreboot.org/hyperkitty/list/flashrom@flashrom.org/thread/UUDQ…
I am leaving the comment unresolved, so that not to accidentally submit before the tag.
--
To view, visit https://review.coreboot.org/c/flashrom/+/83467?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: Id2a2aefc6a3c8348fdaa4498aaa704fd2da7602f
Gerrit-Change-Number: 83467
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Comment-Date: Mon, 15 Jul 2024 09:53:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/83467?usp=email )
Change subject: doc: Add manpage entries for nic3com, gfxnvidia, satasii
......................................................................
doc: Add manpage entries for nic3com, gfxnvidia, satasii
These programmers have a little information about them on wiki.
It's too little info to create a dedicated page for each
programmer, however info can go to the manpage.
For reference, wiki pages are the following:
https://wiki.flashrom.org/NIC3Comhttps://wiki.flashrom.org/Gfxnvidiahttps://wiki.flashrom.org/Supported_programmers (search for the
programmers in the table).
Change-Id: Id2a2aefc6a3c8348fdaa4498aaa704fd2da7602f
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
---
M doc/classic_cli_manpage.rst
1 file changed, 20 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/67/83467/1
diff --git a/doc/classic_cli_manpage.rst b/doc/classic_cli_manpage.rst
index 19c804a..d8380c7 100644
--- a/doc/classic_cli_manpage.rst
+++ b/doc/classic_cli_manpage.rst
@@ -636,6 +636,7 @@
flashrom -p nic3com:pci=05:04.0
+Some of these programmers have more info below.
atavia programmer
^^^^^^^^^^^^^^^^^
@@ -655,6 +656,13 @@
fact that, on the tested device (a Promise Ultra100), not all of the chip's address lines were actually connected.
You may use this programmer to flash firmware updates, since these are only 16 kB in size (padding to 32 kB is required).
+nic3com programmer
+^^^^^^^^^^^^^^^^^^
+
+flashrom supports some 3Com network cards to reflash the (parallel) flash attached to these cards,
+but it is also possible to use these cards to reflash other chips which fit in there electrically.
+Please note that the small number of address lines connected to the chip may make accessing large chips impossible.
+The maximum supported chip size is 128KB.
nicintel_eeprom programmer
^^^^^^^^^^^^^^^^^^^^^^^^^^
@@ -666,6 +674,18 @@
Intel specifies following EEPROMs to be compatible:
Atmel AT25128, AT25256, Micron (ST) M95128, M95256 and OnSemi (Catalyst) CAT25CS128.
+gfxnvidia programmer
+^^^^^^^^^^^^^^^^^^^^
+
+Flashrom supports some Nvidia graphic cards to reflash the (parallel) flash attached to these cards,
+but it is also possible to use these cards to reflash other chips which fit in there electrically.
+
+satasii programmer
+^^^^^^^^^^^^^^^^^^
+
+Flashrom supports some SiI ATA/SATA controllers to reflash the flash attached to these controller cards,
+but it is also possible to use these cards to reflash other chips which fit in there electrically.
+Please note that the small number of address lines connected to the chip may make accessing large chips impossible.
ft2232_spi programmer
^^^^^^^^^^^^^^^^^^^^^
--
To view, visit https://review.coreboot.org/c/flashrom/+/83467?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: Id2a2aefc6a3c8348fdaa4498aaa704fd2da7602f
Gerrit-Change-Number: 83467
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>