diff options
| author | Vincent Palatin <vpalatin@chromium.org> | 2015-01-06 16:04:43 -0800 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-01-08 00:38:17 +0000 |
| commit | cca794d2fec61133f09b6e1cc6816392f11e15cb (patch) | |
| tree | 06cff81c910ae796f2859b9d78f3b656ae1d846a /board/dingdong | |
| parent | dc1362ca82e72ac488bb25d308c97aaf733274fc (diff) | |
| download | chrome-ec-cca794d2fec61133f09b6e1cc6816392f11e15cb.tar.gz | |
pd: dingdong/hoho: Enable flash write-protection
These USB type-C accessories don't have a write-protect GPIO.
Add a configure flag (CONFIG_WP_ALWAYS) to force the flash
write-protection on the dongles.
Also set the read protection (by elevating RDP to level 1),
so trying to unprotect the flash will trigger a full erase.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
BRANCH=samus
BUG=chrome-os-partner:35088
TEST=boot Hoho,
check the flash OBR and WRPR registers:
"rw 0x4002201c" / "rw 0x40022020"
and the option bytes write-protect bits: "rw 0x1FFFF808"
dump the logical state with "flashinfo" command.
> rw 0x4002201c
read 0x40022020 = 0xffff0002
> rw 0x40022020
read 0x40022020 = 0xffff0000
> rw 0x1FFFF808
read 0x1ffff808 = 0xff00ff00
> flashinfo
Physical: 128 KB
Usable: 128 KB
Write: 2 B (ideal 2 B)
Erase: 2048 B (to 1-bits)
Protect: 4096 B
Flags: wp_gpio_asserted ro_at_boot ro_now
Protected now:
YYYYYYYY YYYYYYYY ........ ........
Change-Id: I45bbc0bce40ecc174b6b8a1ebacf4f53d2fd372d
Reviewed-on: https://chromium-review.googlesource.com/238893
Trybot-Ready: Vincent Palatin <vpalatin@chromium.org>
Commit-Queue: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
Reviewed-by: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Diffstat (limited to 'board/dingdong')
| -rw-r--r-- | board/dingdong/board.h | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/board/dingdong/board.h b/board/dingdong/board.h index 4ac7294c73..6247ae5fcc 100644 --- a/board/dingdong/board.h +++ b/board/dingdong/board.h @@ -28,7 +28,7 @@ #define CONFIG_USB_POWER_DELIVERY #define CONFIG_USB_PD_ALT_MODE #define CONFIG_USB_PD_HW_DEV_ID_BOARD_MAJOR USB_PD_HW_DEV_ID_DINGDONG -#define CONFIG_USB_PD_HW_DEV_ID_BOARD_MINOR 1 +#define CONFIG_USB_PD_HW_DEV_ID_BOARD_MINOR 2 #define CONFIG_USB_PD_DUAL_ROLE #define CONFIG_USB_PD_CUSTOM_VDM #define CONFIG_USB_PD_FLASH @@ -44,11 +44,8 @@ #define CONFIG_USB_PID 0x5011 #define CONFIG_USB_BCD_DEV 0x0001 /* v 0.01 */ -/* - * Allow dangerous commands all the time, since we don't have a write protect - * switch. - */ -#define CONFIG_SYSTEM_UNLOCKED +/* No Write-protect GPIO, force the write-protection */ +#define CONFIG_WP_ALWAYS #ifndef __ASSEMBLER__ |