by dpcimanager＠users.sourceforge.net
flashrom v0.9.6.1-r1646 on Darwin 12.2.0 (x86_64)
flashrom is free software, get the source code at http://www.flashrom.org
flashrom was built with libpci 3.1.10, LLVM Clang 4.1 ((tags/Apple/clang-421.11.66)), little endian
Command line (5 args): /var/folders/xj/73b1fsj16m71h44gl8gsh1f00000gn/T/wzQL.HV75QG/DPCIManager.app/Contents/MacOS/flashrom -VV -r /Volumes/Seagate Backup Plus Drive/BIOS/BIOSTAR H61MH 4.6.4.rom -p internal
Calibrating delay loop... OS timer resolution is 1 usecs, 437M loops per second, 10 myus = 9 us, 100 myus = 110 us, 1000 myus = 1071 us, 10000 myus = 10738 us, 4 myus = 4 us, OK.
Initializing internal programmer
Mapping low megabyte at 0x00000400, unaligned size 0xffc00.
Mapping low megabyte, 0xffc00 bytes at unaligned 0x00000400.
No coreboot table found.
DMI string system-manufacturer: "BIOSTAR Group"
DMI string system-product-name: "H61MH"
DMI string system-version: "6.0"
DMI string baseboard-manufacturer: "BIOSTAR Group"
DMI string baseboard-product-name: "H61MH"
DMI string baseboard-version: "6.0"
DMI string chassis-type: "Desktop"
Found ITE Super I/O, ID 0x8728 on port 0x2e
Found chipset "Intel H61" with PCI ID 8086:1c5c. Enabling flash write...
0xfff80000/0xffb80000 FWH IDSEL: 0x0
0xfff00000/0xffb00000 FWH IDSEL: 0x0
0xffe80000/0xffa80000 FWH IDSEL: 0x1
0xffe00000/0xffa00000 FWH IDSEL: 0x1
0xffd80000/0xff980000 FWH IDSEL: 0x2
0xffd00000/0xff900000 FWH IDSEL: 0x2
0xffc80000/0xff880000 FWH IDSEL: 0x3
0xffc00000/0xff800000 FWH IDSEL: 0x3
0xff700000/0xff300000 FWH IDSEL: 0x4
0xff600000/0xff200000 FWH IDSEL: 0x5
0xff500000/0xff100000 FWH IDSEL: 0x6
0xff400000/0xff000000 FWH IDSEL: 0x7
0xfff80000/0xffb80000 FWH decode enabled
0xfff00000/0xffb00000 FWH decode enabled
0xffe80000/0xffa80000 FWH decode enabled
0xffe00000/0xffa00000 FWH decode enabled
0xffd80000/0xff980000 FWH decode enabled
0xffd00000/0xff900000 FWH decode enabled
0xffc80000/0xff880000 FWH decode enabled
0xffc00000/0xff800000 FWH decode enabled
0xff700000/0xff300000 FWH decode disabled
0xff600000/0xff200000 FWH decode disabled
0xff500000/0xff100000 FWH decode disabled
0xff400000/0xff000000 FWH decode disabled
Maximum FWH chip size: 0x100000 bytes
BIOS_CNTL = 0x01: BIOS Lock Enable: disabled, BIOS Write Enable: enabled
Root Complex Register Block address = 0xfed1c000
GCS = 0xc04: BIOS Interface Lock-Down: disabled, Boot BIOS Straps: 0x3 (SPI)
Top Swap : not enabled
SPIBAR = 0xfed1c000 + 0x3800
0x04: 0x6008 (HSFS)
HSFS: FDONE=0, FCERR=0, AEL=0, BERASE=1, SCIP=0, FDOPSS=1, FDV=1, FLOCKDN=0
Programming OPCODES...
program_opcodes: preop=5006 optype=463b opmenu=05d80302c79f0190
done
OP Type Pre-OP
op[0]: 0x02, write w/ addr, none
op[1]: 0x03, read w/ addr, none
op[2]: 0xd8, write w/ addr, none
op[3]: 0x05, read w/o addr, none
op[4]: 0x90, read w/ addr, none
op[5]: 0x01, write w/o addr, none
op[6]: 0x9f, read w/o addr, none
op[7]: 0xc7, write w/o addr, none
Pre-OP 0: 0x06, Pre-OP 1: 0x50
0x06: 0x0000 (HSFC)
HSFC: FGO=0, FCYCLE=0, FDBC=0, SME=0
0x08: 0x0021a77a (FADDR)
0x50: 0x00000a0b (FRAP)
BMWAG 0x00, BMRAG 0x00, BRWA 0x0a, BRRA 0x0b
0x54: 0x00000000 FREG0: Warning: Flash Descriptor region (0x00000000-0x00000fff) is read-only.
0x58: 0x03ff0200 FREG1: BIOS region (0x00200000-0x003fffff) is read-write.
0x5C: 0x01ff0001 FREG2: Warning: Management Engine region (0x00001000-0x001fffff) is locked.
0x60: 0x00000fff FREG3: Gigabit Ethernet region is unused.
0x64: 0x00000fff FREG4: Platform Data region is unused.
0x74: 0x00000000 (PR0 is unused)
0x78: 0x00000000 (PR1 is unused)
0x7C: 0x00000000 (PR2 is unused)
0x80: 0x00000000 (PR3 is unused)
0x84: 0x00000000 (PR4 is unused)
Please send a verbose log to flashrom(a)flashrom.org if this board is not listed on
http://flashrom.org/Supported_hardware#Supported_mainboards yet.
Writes have been disabled for safety reasons. You can enforce write
support with the ich_spi_force programmer option, but you will most likely
harm your hardware! If you force flashrom you will get no support if
something breaks. On a few mainboards it is possible to enable write
access by setting a jumper (see its documentation or the board itself).
0x90: 0x80 (SSFS)
SSFS: SCIP=0, FDONE=0, FCERR=0, AEL=0
0x91: 0xf94010 (SSFC)
SSFC: SCGO=0, ACS=0, SPOP=0, COP=1, DBC=0, SME=0, SCF=1
0x94: 0x5006 (PREOP)
0x96: 0x463b (OPTYPE)
0x98: 0x05d80302 (OPMENU)
0x9C: 0xc79f0190 (OPMENU+4)
0xA0: 0x00000000 (BBAR)
0xC4: 0x00002005 (LVSCC)
LVSCC: BES=0x1, WG=1, WSR=0, WEWS=0, EO=0x20, VCL=0
0xC8: 0x00002005 (UVSCC)
UVSCC: BES=0x1, WG=1, WSR=0, WEWS=0, EO=0x20, VCL=0
0xD0: 0x00000000 (FPB)
Reading flash descriptors mapped by the chipset via FDOC/FDOD... done.
=== Content Section ===
FLVALSIG 0x0ff0a55a
FLMAP0 0x02040003
FLMAP1 0x12100206
FLMAP2 0x00210120
--- Details ---
NR (Number of Regions): 3
FRBA (Flash Region Base Address): 0x040
NC (Number of Components): 1
FCBA (Flash Component Base Address): 0x030
ISL (ICH/PCH Strap Length): 18
FISBA/FPSBA (Flash ICH/PCH Strap Base Address): 0x100
NM (Number of Masters): 3
FMBA (Flash Master Base Address): 0x060
MSL/PSL (MCH/PROC Strap Length): 1
FMSBA (Flash MCH/PROC Strap Base Address): 0x200
=== Component Section ===
FLCOMP 0x09300023
FLILL 0x00000000
--- Details ---
Component 1 density: 4 MB
Component 2 is not used.
Read Clock Frequency: 20 MHz
Read ID and Status Clock Freq.: 33 MHz
Write and Erase Clock Freq.: 33 MHz
Fast Read is supported.
Fast Read Clock Frequency: 33 MHz
No forbidden opcodes.
=== Region Section ===
FLREG0 0x00000000
FLREG1 0x03ff0200
FLREG2 0x01ff0001
FLREG3 0x00000fff
--- Details ---
Region 0 (Descr.) 0x00000000 - 0x00000fff
Region 1 (BIOS ) 0x00200000 - 0x003fffff
Region 2 (ME ) 0x00001000 - 0x001fffff
Region 3 (GbE ) is unused.
=== Master Section ===
FLMSTR1 0x0a0b0000
FLMSTR2 0x0c0d0000
FLMSTR3 0x08080118
--- Details ---
Descr. BIOS ME GbE Platf.
BIOS r rw rw
ME r rw rw
GbE rw
SPI Read Configuration: prefetching disabled, caching enabled, OK.
Super I/O ID 0x8728 is not on the list of flash capable controllers.
The following protocols are supported: FWH, SPI.
Probing for AMIC A25L05PT, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L05PU, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L10PT, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L10PU, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L20PT, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L20PU, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L40PT, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L40PU, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L80P, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L16PT, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L16PU, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L512, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L010, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L020, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L040, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L080, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L016, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25L032, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for AMIC A25LQ032, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF021, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF041A, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF081, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF081A, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF161, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF321, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF321A, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DF641(A), 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25DQ161, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25F512, 64 kB:
program_opcodes: preop=5006 optype=460b opmenu=05150302c79f0190
on-the-fly OPCODE (0x15) re-programmed, op-pos=2
probe_spi_at25f: id1 0xff, id2 0xff
Probing for Atmel AT25F512A, 64 kB: probe_spi_at25f: id1 0xff, id2 0xff
Probing for Atmel AT25F512B, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25F1024(A), 128 kB: probe_spi_at25f: id1 0xff, id2 0xff
Probing for Atmel AT25F2048, 256 kB: probe_spi_at25f: id1 0xff, id2 0xff
Probing for Atmel AT25F4096, 512 kB: probe_spi_at25f: id1 0xff, id2 0xff
Probing for Atmel AT25FS010, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT25FS040, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT26DF041, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT26DF081A, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT26DF161, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT26DF161A, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT26F004, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45CS1282, 16896 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB011D, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB021D, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB041D, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB081D, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB161D, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB321C, 4224 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB321D, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel AT45DB642D, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for EMST F25L008A, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B05, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B05T, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B10, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B10T, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B20, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B20T, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B40T, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B80T, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B16T, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B32T, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25B64T, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F05, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F10, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F20, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25F64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25Q40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25Q80(A), 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25Q16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25Q32(A/B), 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Found Eon flash chip "EN25Q32(A/B)" (4096 kB, SPI) at physical address 0xffc00000.
Chip status register is 0x00.
Probing for Eon EN25Q64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25Q128, 16384 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25QH16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon EN25QH32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q20, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25Q128, 16384 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for GigaDevice GD25LQ32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Intel 25F160S33B8, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Intel 25F160S33T8, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Intel 25F320S33B8, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Intel 25F320S33T8, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Intel 25F640S33B8, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Intel 25F640S33T8, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L512, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L1005, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L2005, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L4005, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L8005, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L1605, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L1635D, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L1635E, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L3205, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L3235D, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L6405, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix MX25L12805, 16384 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx M25PE10, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx M25PE20, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx M25PE40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx M25PE80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx M25PE16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx N25Q016, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx N25Q032..1E, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx N25Q032..3E, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx N25Q064..1E, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Numonyx N25Q064..3E, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC Pm25LV010, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC Pm25LV016B, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC Pm25LV020, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC Pm25LV040, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC Pm25LV080B, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC Pm25LV512, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Sanyo LF25FW203A, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Spansion S25FL004A, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Spansion S25FL008A, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Spansion S25FL016A, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Spansion S25FL032A/P, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Spansion S25FL064A/P, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25LF040A, 512 kB:
program_opcodes: preop=5006 optype=462b opmenu=05ab0302c79f0190
on-the-fly OPCODE (0xAB) re-programmed, op-pos=2
probe_spi_res2: id1 0x15, id2 0x15
Probing for SST SST25LF080(A), 1024 kB: probe_spi_res2: id1 0x15, id2 0x15
Probing for SST SST25VF010, 128 kB: probe_spi_rems: id1 0x1c, id2 0x15
Probing for SST SST25VF016B, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25VF032B, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25VF064C, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25VF040, 512 kB: probe_spi_rems: id1 0x1c, id2 0x15
Probing for SST SST25VF040B, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25VF040B.REMS, 512 kB: probe_spi_rems: id1 0x1c, id2 0x15
Probing for SST SST25VF080B, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25WF512, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25WF010, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25WF020, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST SST25WF040, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P05-A, 64 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P05, 64 kB: Ignoring RES in favour of RDID.
Probing for ST M25P10-A, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P10, 128 kB: Ignoring RES in favour of RDID.
Probing for ST M25P20, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P40-old, 512 kB: Ignoring RES in favour of RDID.
Probing for ST M25P80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25P128, 16384 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25PX16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25PX32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST M25PX64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25Q80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25Q16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25Q32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25Q64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25Q128, 16384 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X10, 128 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X20, 256 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X40, 512 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X80, 1024 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X16, 2048 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X32, 4096 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond W25X64, 8192 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Unknown SFDP-capable chip, 0 kB:
program_opcodes: preop=5006 optype=462b opmenu=055a0302c79f0190
on-the-fly OPCODE (0x5A) re-programmed, op-pos=2
Signature = 0xffffffff (should be 0x50444653)
No SFDP signature found.
Probing for AMIC unknown AMIC SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Atmel unknown Atmel SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Eon unknown Eon SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Macronix unknown Macronix SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for PMC unknown PMC SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for SST unknown SST SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for ST unknown ST SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Sanyo unknown Sanyo SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Winbond unknown Winbond (ex Nexcom) SPI chip, 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Generic unknown SPI chip (RDID), 0 kB: probe_spi_rdid_generic: id1 0x1c, id2 0x3016
Probing for Generic unknown SPI chip (REMS), 0 kB: probe_spi_rems: id1 0x1c, id2 0x15
Probing for Atmel AT49LH002, 256 kB: probe_82802ab: id1 0x19, id2 0xf7, id1 is normal flash content, id2 is normal flash content
Probing for Intel 82802AB, 512 kB: probe_82802ab: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for Intel 82802AC, 1024 kB: probe_82802ab: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for PMC Pm49FL002, 256 kB: probe_jedec_common: id1 0x19, id2 0xf7, id1 is normal flash content, id2 is normal flash content
Probing for PMC Pm49FL004, 512 kB: probe_jedec_common: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for Sharp LHF00L04, 1024 kB: probe_82802ab: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF002A/B, 256 kB: probe_jedec_common: id1 0x19, id2 0xf7, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF003A/B, 384 kB: probe_jedec_common: id1 0x00, id2 0x00, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF004A/B, 512 kB: probe_jedec_common: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF004C, 512 kB: probe_82802ab: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF008A, 1024 kB: probe_jedec_common: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF008C, 1024 kB: probe_82802ab: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for SST SST49LF016C, 2048 kB: Chip size 2048 kB is bigger than supported size 1024 kB of chipset/board/programmer for FWH interface, probe/read/erase/write may fail. probe_82802ab: id1 0x00, id2 0x00, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FLW040A, 512 kB: probe_82802ab: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FLW040B, 512 kB: probe_82802ab: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FLW080A, 1024 kB: probe_82802ab: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FLW080B, 1024 kB: probe_82802ab: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FW002, 256 kB: probe_82802ab: id1 0x19, id2 0xf7, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FW016, 2048 kB: Chip size 2048 kB is bigger than supported size 1024 kB of chipset/board/programmer for FWH interface, probe/read/erase/write may fail. probe_82802ab: id1 0x00, id2 0x00, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FW040, 512 kB: probe_82802ab: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for ST M50FW080, 1024 kB: probe_82802ab: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for Winbond W39V040FA, 512 kB: probe_jedec_common: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for Winbond W39V040FB, 512 kB: probe_jedec_common: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for Winbond W39V040FC, 512 kB: probe_jedec_common: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Probing for Winbond W49V002FA, 256 kB: probe_jedec_common: id1 0x19, id2 0xf7, id1 is normal flash content, id2 is normal flash content
Probing for Winbond W39V080FA, 1024 kB: probe_jedec_common: id1 0x56, id2 0x08, id1 parity violation, id1 is normal flash content, id2 is normal flash content
Probing for Winbond W39V080FA (dual mode), 512 kB: probe_jedec_common: id1 0xea, id2 0xd0, id1 is normal flash content, id2 is normal flash content
Found Eon flash chip "EN25Q32(A/B)" (4096 kB, SPI).
This chip may contain one-time programmable memory. flashrom cannot read
and may never be able to write it, hence it may not be able to completely
clone the contents of this chip (see man page for details).
===
This flash part has status UNTESTED for operations: READ ERASE WRITE
The test status of this chip may have been updated in the latest development
version of flashrom. If you are running the latest development version,
please email a report to flashrom(a)flashrom.org if any of the above operations
work correctly for you with this flash part. Please include the flashrom
output with the additional -V option for all operations you tested (-V, -Vr,
-VE, -Vw), and mention which mainboard or programmer you tested.
Please mention your board in the subject line. Thanks for your help!
Block protection is disabled.
Reading flash... Transaction error!
SSFS: SCIP=0, FDONE=1, FCERR=1, AEL=0
SSFC: SCGO=0, ACS=0, SPOP=0, COP=1, DBC=63, SME=0, SCF=0
Running OPCODE 0x03 failed at address 0x001000 (payload length was 64).
Read operation failed!
FAILED.
Restoring MMIO space at 0x10913b8a0
Restoring MMIO space at 0x10913b89c
Restoring MMIO space at 0x10913b898
Restoring MMIO space at 0x10913b896
Restoring MMIO space at 0x10913b894
Restoring PCI config space for 00:1f:0 reg 0xdc
by Andrew Tarabaras
Hi,
Just a heads up that the rayer_spi module wont work with a buffered xilinx
DLC cable. There is a pull down buffer in the MISO ( Data In ) line which
is enabled when pin 6 of the parallel port is pulled low.
I dont have time to be submitting patches, fix it or not I dont mind.
These are the changes I made.
Global var
static int rayer_pulldown_bit = 4;
in rayer_spi_init()
after rget_io_perms();
val = 1;
lpt_outbyte &= ~(1 << rayer_pullup_bit);
lpt_outbyte |= (val << rayer_pullup_bit);
OUTB(lpt_outbyte, lpt_iobase);
Cheers, hope it will help stop someone else spending hours debugging
something that doesn't work.
by Justin Chevrier
Hey Guys,
Another report; this one successful going from BIOS 0205 to 0305.
Executed an erase before writing the new BIOS. Original MAC address of
the onboard LAN adapter was lost after upgrade.
Justin
by Stefan Tauner
Automatically round address ranges requested from rphysmap() to page
boundaries. Other physmap() variants were not changed so this is
safe regarding unmapping.
I had also to shorten the readbility defines for the physmap_common
parameters to make the calls fit a single line.
Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
---
physmap.c | 61 ++++++++++++++++++++++++++++++++++++++++---------------------
1 file changed, 40 insertions(+), 21 deletions(-)
diff --git a/physmap.c b/physmap.c
index 3811824..4f82d0f 100644
--- a/physmap.c
+++ b/physmap.c
@@ -218,12 +218,35 @@ void physunmap(void *virt_addr, size_t len)
}
#endif
-#define PHYSMAP_NOFAIL 0
-#define PHYSMAP_MAYFAIL 1
-#define PHYSMAP_RW 0
-#define PHYSMAP_RO 1
-#define PHYSMAP_NOCLEANUP 0
-#define PHYSMAP_CLEANUP 1
+#define PHYSM_NOFAIL 0
+#define PHYSM_MAYFAIL 1
+#define PHYSM_RW 0
+#define PHYSM_RO 1
+#define PHYSM_NOCLEANUP 0
+#define PHYSM_CLEANUP 1
+#define PHYSM_EXACT 0
+#define PHYSM_ROUND 1
+
+/* Round start to nearest page boundary below and set len so that the resulting
+ * address range ends at the lowest possible page boundary where the original
+ * address range is still entirely contained. It returns the difference between
+ * the rounded start address and the original start address. */
+static uintptr_t round_to_page_boundaries(uintptr_t *start, size_t *len)
+{
+ uintptr_t page_size = getpagesize();
+ uintptr_t page_mask = ~(page_size-1);
+ uintptr_t end = *start + *len;
+ uintptr_t old_start = *start;
+ msg_gspew("page_size=%" PRIxPTR "\n", page_size);
+ msg_gspew("pre-rounding: start=0x%0*" PRIxPTR ", len=0x%0zx, end=0x%0*" PRIxPTR "\n",
+ PRIxPTR_WIDTH, *start, *len, PRIxPTR_WIDTH, end);
+ *start = *start & page_mask;
+ end = (end + page_size - 1) & page_mask;
+ *len = end - *start;
+ msg_gspew("post-rounding: start=0x%0*" PRIxPTR ", len=0x%0zx, end=0x%0*" PRIxPTR "\n",
+ PRIxPTR_WIDTH, *start, *len, PRIxPTR_WIDTH, *start + *len);
+ return old_start - *start;
+}
struct undo_physmap_data {
void *virt_addr;
@@ -243,24 +266,18 @@ static int undo_physmap(void *data)
}
static void *physmap_common(const char *descr, uintptr_t phys_addr, size_t len, bool mayfail,
- bool readonly, bool autocleanup)
+ bool readonly, bool autocleanup, bool round)
{
void *virt_addr;
+ uintptr_t offset = 0;
if (len == 0) {
msg_pspew("Not mapping %s, zero size at 0x%0*" PRIxPTR ".\n", descr, PRIxPTR_WIDTH, phys_addr);
return ERROR_PTR;
}
- if ((getpagesize() - 1) & len) {
- msg_perr("Mapping %s at 0x%08lx, unaligned size 0x%lx.\n",
- descr, phys_addr, (unsigned long)len);
- }
-
- if ((getpagesize() - 1) & phys_addr) {
- msg_perr("Mapping %s, 0x%lx bytes at unaligned 0x%08lx.\n",
- descr, (unsigned long)len, phys_addr);
- }
+ if (round)
+ offset = round_to_page_boundaries(&phys_addr, &len);
if (readonly)
virt_addr = sys_physmap_ro_cached(phys_addr, len);
@@ -286,7 +303,9 @@ static void *physmap_common(const char *descr, uintptr_t phys_addr, size_t len,
"and reboot, or reboot into\n"
"single user mode.\n");
#endif
- if (!mayfail)
+ if (mayfail)
+ return ERROR_PTR;
+ else
exit(3);
}
@@ -305,7 +324,7 @@ static void *physmap_common(const char *descr, uintptr_t phys_addr, size_t len,
}
}
- return virt_addr;
+ return virt_addr + offset;
unmap_out:
physunmap(virt_addr, len);
if (!mayfail)
@@ -315,17 +334,17 @@ unmap_out:
void *physmap(const char *descr, uintptr_t phys_addr, size_t len)
{
- return physmap_common(descr, phys_addr, len, PHYSMAP_NOFAIL, PHYSMAP_RW, PHYSMAP_NOCLEANUP);
+ return physmap_common(descr, phys_addr, len, PHYSM_NOFAIL, PHYSM_RW, PHYSM_NOCLEANUP, PHYSM_EXACT);
}
void *rphysmap(const char *descr, uintptr_t phys_addr, size_t len)
{
- return physmap_common(descr, phys_addr, len, PHYSMAP_NOFAIL, PHYSMAP_RW, PHYSMAP_CLEANUP);
+ return physmap_common(descr, phys_addr, len, PHYSM_NOFAIL, PHYSM_RW, PHYSM_CLEANUP, PHYSM_ROUND);
}
void *physmap_try_ro(const char *descr, uintptr_t phys_addr, size_t len)
{
- return physmap_common(descr, phys_addr, len, PHYSMAP_MAYFAIL, PHYSMAP_RO, PHYSMAP_NOCLEANUP);
+ return physmap_common(descr, phys_addr, len, PHYSM_MAYFAIL, PHYSM_RO, PHYSM_NOCLEANUP, PHYSM_EXACT);
}
#if defined(__i386__) || defined(__x86_64__)
--
Kind regards, Stefan Tauner
by Stefan Tauner
unsigned long is not the right type for manipulating pointer values.
Since C99 there are suitable unsigned and signed types available, namely
uintptr_t and intptr_t respectively.
Use them in physmap.c where applicable. This should be done in a lot of
other places in flashrom too, but that's a huge pile of work and we better
start it with a small step than never (which is the forseeable alternative ;)
This patch also changes the display width of all address values in
physmap.c to 16/8 hex characters depending on the architecture.
Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
---
physmap.c | 30 ++++++++++++++++--------------
1 file changed, 16 insertions(+), 14 deletions(-)
diff --git a/physmap.c b/physmap.c
index 40dfa50..3811824 100644
--- a/physmap.c
+++ b/physmap.c
@@ -20,6 +20,7 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
+#include <inttypes.h>
#include <unistd.h>
#include <stdbool.h>
#include <stdio.h>
@@ -35,6 +36,8 @@
#include <errno.h>
#endif
+#define PRIxPTR_WIDTH ((int)(sizeof(uintptr_t)*2))
+
#ifdef __DJGPP__
#include <dpmi.h>
#include <sys/nearptr.h>
@@ -43,7 +46,7 @@
static void *realmem_map;
-static void *map_first_meg(unsigned long phys_addr, size_t len)
+static void *map_first_meg(uintptr_t phys_addr, size_t len)
{
if (realmem_map)
return realmem_map + phys_addr;
@@ -62,7 +65,7 @@ static void *map_first_meg(unsigned long phys_addr, size_t len)
return realmem_map + phys_addr;
}
-static void *sys_physmap(unsigned long phys_addr, size_t len)
+static void *sys_physmap(uintptr_t phys_addr, size_t len)
{
int ret;
__dpmi_meminfo mi;
@@ -110,7 +113,7 @@ void physunmap(void *virt_addr, size_t len)
#define MEM_DEV ""
-void *sys_physmap(unsigned long phys_addr, size_t len)
+void *sys_physmap(uintptr_t phys_addr, size_t len)
{
return (void *)phys_to_virt(phys_addr);
}
@@ -134,7 +137,7 @@ void cleanup_cpu_msr(void)
#define MEM_DEV "DirectHW"
-static void *sys_physmap(unsigned long phys_addr, size_t len)
+static void *sys_physmap(uintptr_t phys_addr, size_t len)
{
/* The short form of ?: is a GNU extension.
* FIXME: map_physical returns NULL both for errors and for success
@@ -166,7 +169,7 @@ static int fd_mem = -1;
static int fd_mem_cached = -1;
/* For MMIO access. Must be uncached, doesn't make sense to restrict to ro. */
-static void *sys_physmap_rw_uncached(unsigned long phys_addr, size_t len)
+static void *sys_physmap_rw_uncached(uintptr_t phys_addr, size_t len)
{
void *virt_addr;
@@ -186,7 +189,7 @@ static void *sys_physmap_rw_uncached(unsigned long phys_addr, size_t len)
/* For reading DMI/coreboot/whatever tables. We should never write, and we
* do not care about caching.
*/
-static void *sys_physmap_ro_cached(unsigned long phys_addr, size_t len)
+static void *sys_physmap_ro_cached(uintptr_t phys_addr, size_t len)
{
void *virt_addr;
@@ -239,14 +242,13 @@ static int undo_physmap(void *data)
return 0;
}
-static void *physmap_common(const char *descr, unsigned long phys_addr, size_t len, bool mayfail,
+static void *physmap_common(const char *descr, uintptr_t phys_addr, size_t len, bool mayfail,
bool readonly, bool autocleanup)
{
void *virt_addr;
if (len == 0) {
- msg_pspew("Not mapping %s, zero size at 0x%08lx.\n",
- descr, phys_addr);
+ msg_pspew("Not mapping %s, zero size at 0x%0*" PRIxPTR ".\n", descr, PRIxPTR_WIDTH, phys_addr);
return ERROR_PTR;
}
@@ -268,8 +270,8 @@ static void *physmap_common(const char *descr, unsigned long phys_addr, size_t l
if (ERROR_PTR == virt_addr) {
if (NULL == descr)
descr = "memory";
- msg_perr("Error accessing %s, 0x%lx bytes at 0x%08lx\n", descr,
- (unsigned long)len, phys_addr);
+ msg_perr("Error accessing %s, 0x%0zx bytes at 0x%0*" PRIxPTR "\n",
+ descr, len, PRIxPTR_WIDTH, phys_addr);
perror(MEM_DEV " mmap failed");
#ifdef __linux__
if (EINVAL == errno) {
@@ -311,17 +313,17 @@ unmap_out:
return ERROR_PTR;
}
-void *physmap(const char *descr, unsigned long phys_addr, size_t len)
+void *physmap(const char *descr, uintptr_t phys_addr, size_t len)
{
return physmap_common(descr, phys_addr, len, PHYSMAP_NOFAIL, PHYSMAP_RW, PHYSMAP_NOCLEANUP);
}
-void *rphysmap(const char *descr, unsigned long phys_addr, size_t len)
+void *rphysmap(const char *descr, uintptr_t phys_addr, size_t len)
{
return physmap_common(descr, phys_addr, len, PHYSMAP_NOFAIL, PHYSMAP_RW, PHYSMAP_CLEANUP);
}
-void *physmap_try_ro(const char *descr, unsigned long phys_addr, size_t len)
+void *physmap_try_ro(const char *descr, uintptr_t phys_addr, size_t len)
{
return physmap_common(descr, phys_addr, len, PHYSMAP_MAYFAIL, PHYSMAP_RO, PHYSMAP_NOCLEANUP);
}
--
Kind regards, Stefan Tauner
by Carl-Daniel Hailfinger
Am 01.01.2013 18:29 schrieb Stefan Tauner:
> Similarly to the previous PCI self-clean up patch this one allows to get rid
> of a huge number of programmer shutdown functions and makes introducing
> bugs harder. It adds a new function physmap_autocleanup() that takes
> care of unmapping at shutdown. Callers are changed where it makes sense.
>
> Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
Good work! The individual driver conversions were checked thoroughly and
seem to be OK.
With the comments addressed, this is
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Can you please change the name of physmap_autocleanup to rphysmap? That
would be consistent with rmmio_write* and rpci_write* and be shorter as
well (fixing some 112 columns limit issues).
> diff --git a/it85spi.c b/it85spi.c
> index 0b074eb..b778436 100644
> --- a/it85spi.c
> +++ b/it85spi.c
> @@ -262,6 +262,9 @@ static int it85xx_spi_common_init(struct superio s)
> * Major TODO here, and it will be a lot of work.
> */
> base = (chipaddr)physmap("it85 communication", 0xFFFFF000, 0x1000);
> + if (base == ERROR_PTR)
I somehow doubt that this really compiles. AFAICS you might not have
noticed the type mismatch problem because of #if guards.
> + return 1;
> +
> msg_pdbg("%s():%d base=0x%08x\n", __func__, __LINE__,
> (unsigned int)base);
> ce_high = (unsigned char *)(base + 0xE00); /* 0xFFFFFE00 */
> diff --git a/nicintel.c b/nicintel.c
> index 8481915..5463af2 100644
> --- a/nicintel.c
> +++ b/nicintel.c
> @@ -81,19 +74,15 @@ int nicintel_init(void)
> */
> addr = pcidev_init(PCI_BASE_ADDRESS_2, nics_intel);
>
> - nicintel_bar = physmap("Intel NIC flash", addr, NICINTEL_MEMMAP_SIZE);
> + nicintel_bar = physmap_autocleanup("Intel NIC flash", addr, NICINTEL_MEMMAP_SIZE);
> if (nicintel_bar == ERROR_PTR)
> - goto error_out_unmap;
> + return 1;
>
> /* FIXME: Using pcidev_dev _will_ cause pretty explosions in the future. */
> addr = pcidev_readbar(pcidev_dev, PCI_BASE_ADDRESS_0);
> /* FIXME: This is not an aligned mapping. Use 4k? */
> - nicintel_control_bar = physmap("Intel NIC control/status reg",
> - addr, NICINTEL_CONTROL_MEMMAP_SIZE);
> + nicintel_control_bar = physmap_autocleanup("Intel NIC control/status reg", addr, NICINTEL_CONTROL_MEMMAP_SIZE);
112 column limit... not a problem with rphysmap.
> if (nicintel_control_bar == ERROR_PTR)
> - goto error_out;
> -
> - if (register_shutdown(nicintel_shutdown, NULL))
> return 1;
>
> /* FIXME: This register is pretty undocumented in all publicly available
> diff --git a/physmap.c b/physmap.c
> index 5aa9874..a9445d5 100644
> --- a/physmap.c
> +++ b/physmap.c
> @@ -21,6 +21,7 @@
> */
>
> #include <unistd.h>
> +#include <stdbool.h>
Note: stdbool.h is only available in C99 compilers, which means this
won't ever compile with MSVC. I don't really have a strong preference
either way (and we use mingw/cygwin gcc for win* targets anyway).
> #include <stdio.h>
> #include <stdlib.h>
> #include <string.h>
> @@ -214,13 +215,25 @@ void physunmap(void *virt_addr, size_t len)
> }
> #endif
>
> -#define PHYSMAP_NOFAIL 0
> -#define PHYSMAP_MAYFAIL 1
> -#define PHYSMAP_RW 0
> -#define PHYSMAP_RO 1
IMHO those #defines really helped readability. You can replace 0 and 1
by false/true if you want, but please keep the #defines.
> +struct physmap_stutdown_data {
Maybe rename to undo_physmap_data for consistency with undo_pci_write_data.
> + void *addr;
Use virt_addr to make it obvious that this is not a physical address,
but the address where this was mapped into the address space of the
flashrom process?
> + size_t len;
> +};
>
> -static void *physmap_common(const char *descr, unsigned long phys_addr,
> - size_t len, int mayfail, int readonly)
> +static int physmap_shutdown(void *data)
Rename to undo_physmap for consistency with undo_pci_write?
> +{
> + if (data == NULL) {
> + msg_perr("%s: tried to shutdown without valid data!\n", __func__);
> + return 1;
> + }
> + struct physmap_stutdown_data *d = data;
Not all compilers support defining a new variable after code. Please
move the line above to the head of the function.
> + physunmap(d->addr, d->len);
> + free(data);
> + return 0;
> +}
> +
> +static void *physmap_common(const char *descr, unsigned long phys_addr, size_t len, bool mayfail,
> + bool readonly, bool autocleanup)
> {
> void *virt_addr;
>
> @@ -268,19 +281,40 @@ static void *physmap_common(const char *descr, unsigned long phys_addr,
> exit(3);
> }
>
> + if (autocleanup) {
> + struct physmap_stutdown_data *d = malloc(sizeof(struct physmap_stutdown_data));
I wonder if that malloc should be moved to the start of physmap_common.
It would detect OOM earlier, before we do any mapping.
> + if (d == NULL) {
> + msg_perr("%s: Out of memory!\n", __func__);
> + goto unmap_out;
> + }
> +
> + d->addr = virt_addr;
> + d->len = len;
> + if (register_shutdown(physmap_shutdown, d) != 0) {
> + msg_perr("%s: Could not register shutdown function!\n", __func__);
> + goto unmap_out;
> + }
> + }
> +
> return virt_addr;
> +unmap_out:
> + physunmap(virt_addr, len);
> + return ERROR_PTR;
> }
>
> void *physmap(const char *descr, unsigned long phys_addr, size_t len)
> {
> - return physmap_common(descr, phys_addr, len, PHYSMAP_NOFAIL,
> - PHYSMAP_RW);
> + return physmap_common(descr, phys_addr, len, false, false, false);
I really liked the #defines here because they increased readability.
> +}
> +
> +void *physmap_autocleanup(const char *descr, unsigned long phys_addr, size_t len)
> +{
> + return physmap_common(descr, phys_addr, len, false, false, true);
> }
>
> void *physmap_try_ro(const char *descr, unsigned long phys_addr, size_t len)
> {
> - return physmap_common(descr, phys_addr, len, PHYSMAP_MAYFAIL,
> - PHYSMAP_RO);
> + return physmap_common(descr, phys_addr, len, true, true, false);
> }
>
> #if defined(__i386__) || defined(__x86_64__)
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
by Justin Chevrier
Hi Guys,
Attached are logs from a failed attempted erase of the flash rom on
the system listed in the subject. I was able to verify the flash
against the backup I had taken and the system was able to successfully
reboot, so it appears no changes were made. It appears this system is
using a nonstandard chassis type ("Space-saving") and the:
"laptop=this_is_not_a_laptop" is required for Flashrom to interact
with the system. HP doesn't specify any particular process (jumper,
setting in BIOS) required to update the BIOS on this system (that I
could find).
Direct link to BIOS: ftp://ftp.hp.com/pub/softpaq/sp23001-23500/sp23350.exe
Downloaded file can be extracted via 7zip, then the resulting ROM.CAB
file can be extracted as well resulting in 'Rom.bin' that matches the
size of the chip.
7z x sp23350.exe
7z x ROM.CAB
Thanks,
Justin
by Anderson Vulczak
Hello,
Just tested this EPROM and it's writing, reading, erasing great.
Thanks, and Best Regards,
root@server:~# flashrom -p rayer_spi -w binary.bin
flashrom v0.9.5.2-r1517 on Linux 3.2.0-37-generic (x86_64), built with
libpci 3.1.8, GCC 4.6.3, little endian
flashrom is free software, get the source code at http://www.flashrom.org
Calibrating delay loop... OK.
Found Eon flash chip "EN25Q64" (8192 kB, SPI) on rayer_spi.
===
This flash part has status UNTESTED for operations: PROBE READ ERASE WRITE
The test status of this chip may have been updated in the latest development
version of flashrom. If you are running the latest development version,
please email a report to flashrom(a)flashrom.org if any of the above
operations
work correctly for you with this flash part. Please include the flashrom
output with the additional -V option for all operations you tested (-V, -Vr,
-VE, -Vw), and mention which mainboard or programmer you tested.
Please mention your board in the subject line. Thanks for your help!
Reading old flash chip contents... done.
Erasing and writing flash chip... Erase/write done.
Verifying flash... VERIFIED.
by Stefan Tauner
This allows to list yet unsupported chips easily.
First it tries to find the directory containing the files, then it uses sed to
extract the macro names of chips from flashchips.h, greps for them in
flashchips.c and prints it if it is not found.
If verbose mode is activated by giving at least one additional parameter
it prints the comment following the macro definition too.
Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
---
util/list_yet_unsupported_chips.sh | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
create mode 100755 util/list_yet_unsupported_chips.sh
diff --git a/util/list_yet_unsupported_chips.sh b/util/list_yet_unsupported_chips.sh
new file mode 100755
index 0000000..2ed71e6
--- /dev/null
+++ b/util/list_yet_unsupported_chips.sh
@@ -0,0 +1,22 @@
+#!/bin/sh
+
+path="$(dirname $(readlink -f $0))/.."
+if [ ! -e "$path/flashchips.c" -o ! -e "$path/flashchips.h" ]; then
+ echo "Warning: could not calculate flashchips.[ch]'s directory. Trying current..."
+ path="."
+ if [ ! -e "$path/flashchips.c" -o ! -e "$path/flashchips.h" ]; then
+ echo "Nope, sorry!"
+ exit 1
+ fi
+fi
+
+chips=`sed -re '/#define [A-Z]/ !d' -e '/_ID\s/d' -e 's/\s*#define\s+([[:alnum:]_]+)\s+.*/\1/' $path/flashchips.h`
+for c in $chips ; do
+ if ! grep "$c" "$path/flashchips.c" >/dev/null ; then
+ if [ -n "$1" ]; then
+ grep -o "$c.*" "$path/flashchips.h"
+ else
+ echo "$c"
+ fi
+ fi
+done
--
Kind regards, Stefan Tauner
