diff options
author | Hsuan Ting Chen <roccochen@chromium.org> | 2021-07-05 18:49:20 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-07-16 09:35:45 +0000 |
commit | 71aa26394ac18ec35ea9cbb08392faf8dca4fbcb (patch) | |
tree | 4a6c8449dee713fe654ec48786a9a13c18263e3b /Makefile | |
parent | 22777bb5d531fac82da05c1fca6f02111b93e286 (diff) | |
download | vboot-71aa26394ac18ec35ea9cbb08392faf8dca4fbcb.tar.gz |
Makefile: deprecate config DIAGNOSTIC_UI
DIAGNOSTIC_UI is indirectly decided by the diag_payload and minidiag use
flag from depthcharge.
But with introducing the diagnostic boot mode, coreboot also needs to
build minidiag utility functions and enable this building flag.
Therefore we consider to deprecate DIAGNOSTIC_UI and always build them.
For diag_payload:
It is for a diagnostic tool which based on legacy UI and deprecated
For minidiag:
Mini-diag is set default enabled for newer devices, and we could still
use kernel secdata in runtime if we decide to disable it on certain
platforms.
In conclusion, deprecating DIAGNOSTIC_UI will not affect newer devices.
BUG=b:190796342, b:181931817
BRANCH=none
TEST=emerge-volteer depthcharge
TEST=emerge-volteer coreboot
TEST=CC=x86_64-pc-linux-gnu-clang;
make clean && make runtests
Cq-Depend: chromium:3004223
Signed-off-by: Hsuan Ting Chen <roccochen@chromium.org>
Change-Id: I0d804bcd9d31d3952c744a1926ac59cde7b7f841
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3006114
Reviewed-by: Yu-Ping Wu <yupingso@chromium.org>
Diffstat (limited to 'Makefile')
-rw-r--r-- | Makefile | 7 |
1 files changed, 0 insertions, 7 deletions
@@ -235,13 +235,6 @@ else CFLAGS += -DDETACHABLE=0 endif -# pass DIAGNOSTIC_UI= (or =0) to make to disable feature -ifneq ($(filter-out 0,${DIAGNOSTIC_UI}),) -CFLAGS += -DDIAGNOSTIC_UI=1 -else -CFLAGS += -DDIAGNOSTIC_UI=0 -endif - # Confirm physical presence using keyboard ifneq ($(filter-out 0,${PHYSICAL_PRESENCE_KEYBOARD}),) CFLAGS += -DPHYSICAL_PRESENCE_KEYBOARD=1 |