diff options
| author | Werner Zeh <werner.zeh@siemens.com> | 2022-01-25 07:02:49 +0100 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2022-02-16 10:25:09 +0000 |
| commit | ff4c624d94d77543313b1ff8247c6430bae14bcd (patch) | |
| tree | b5f546b80edea5c8940304089357945a978c6002 | |
| parent | 6080d488dd195869015a2d76df035d82e6c63da1 (diff) | |
| download | flashrom-ff4c624d94d77543313b1ff8247c6430bae14bcd.tar.gz flashrom-ff4c624d94d77543313b1ff8247c6430bae14bcd.tar.bz2 flashrom-ff4c624d94d77543313b1ff8247c6430bae14bcd.zip | |
ich_descriptors.c Invert the meaning of 'dual_output' bit
In the Flash Component description register (FLCOMP) bit 30 reports the
capability of using dual output for fast read operation on the flash
component. According to various SPI Programming Guides (checked for
Panther Point, Lewisburg C620, Apollo Lake and Elkhart Lake) the dual
output is enabled when this bit is set and disabled if not. Currently the
logic displays it the other way around when parsing the descriptor.
This patch changes this so now if bit 30 in FLCOMP is not set, dual read
support for fast read operation is shown as disabled.
Change-Id: If6282ac8326ab0b92e9c70c09dba0299bf0deb6f
Signed-off-by: Werner Zeh <werner.zeh@siemens.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/61362
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
| -rw-r--r-- | ich_descriptors.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ich_descriptors.c b/ich_descriptors.c index d716d1d7..0ce57205 100644 --- a/ich_descriptors.c +++ b/ich_descriptors.c @@ -380,7 +380,7 @@ void prettyprint_ich_descriptor_component(enum ich_chipset cs, const struct ich_ pprint_freq(cs, desc->component.modes.freq_fastread)); if (cs > CHIPSET_6_SERIES_COUGAR_POINT) msg_pdbg2("Dual Output Fast Read Support: %sabled\n", - desc->component.modes.dual_output ? "dis" : "en"); + desc->component.modes.dual_output ? "en" : "dis"); int has_forbidden_opcode = 0; if (desc->component.FLILL != 0) { |