openwrt/flashrom
1
0
Fork 0
mirror of https://review.coreboot.org/flashrom.git synced 2025-04-27 23:22:37 +02:00
Code Issues Releases Wiki Activity

parade_lspcon.c: Clarify coincidentally JEDEC command usage

Browse Source 
Clarify that SWSPI_WDATA_* coincidentally uses the same
commands as specified by JEDEC.

A similar data sheet does not really shed light if these
literally are raw JEDEC command values or 'virtual' ones.
As to avoid confusion, comment but perhaps not use the JEDEC
literals from spi.h until it is certain.

Change-Id: I851319ad4c36baad1e280309a6df8c86d6c4ad3d
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/65557
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Reviewed-by: Thomas Heijligen <src@posteo.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
This commit is contained in:
Edward O'Callaghan 2022-06-30 20:00:11 +10:00 committed by Anastasia Klimchuk
parent f1ca19598c
commit dd1d07646f
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
parade_lspcon.c
View File

@ -35,10 +35,11 @@
#define ROMADDR_BYTE1 0x8e
#define ROMADDR_BYTE2 0x8f
#define SWSPI_WDATA 0x90
/* SWSPI_WDATA_* appear to be numerically the same as JEDEC commands. */
#define SWSPI_WDATA_CLEAR_STATUS 0x00
#define SWSPI_WDATA_WRITE_REGISTER 0x01
#define SWSPI_WDATA_READ_REGISTER 0x05
#define SWSPI_WDATA_ENABLE_REGISTER 0x06
#define SWSPI_WDATA_WRITE_REGISTER 0x01 /* JEDEC_WRSR */
#define SWSPI_WDATA_READ_REGISTER 0x05 /* JEDEC_RDSR */
#define SWSPI_WDATA_ENABLE_REGISTER 0x06 /* JEDEC_WREN */
#define SWSPI_WDATA_PROTECT_BP 0x8c
#define SWSPI_RDATA 0x91
#define SWSPI_LEN 0x92