diff options
| author | Vincent Palatin <vpalatin@chromium.org> | 2015-01-16 09:59:39 -0800 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-01-30 03:28:00 +0000 |
| commit | 862145dd758f1d45465232e39e6b00c111444e28 (patch) | |
| tree | 2d818f36dc5e7a9a6b8cdb04c31bf8f6d46d7359 | |
| parent | 0775b2a51bb1a6bb57a8384a16cdff5ab92a3065 (diff) | |
| download | chrome-ec-862145dd758f1d45465232e39e6b00c111444e28.tar.gz | |
samus_pd: remove asserts
Remove assertions to save flash space :
free up 1728 bytes.
Removing the string in assertions saves 1.1kB.
(aka CONFIG_DEBUG_ASSERT_BRIEF)
Fully removing assertion checks saves an additional 576 bytes.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
BRANCH=samus
BUG=chrome-os-partner:34489
TEST=make buildall and check firmware size.
BASELINE 93bb7195
=== build/samus_pd/ ===
FLASH 59.5k / 60.0k [ text 48.7k rodat 10.7k data 0.1k ]
RAM 11.8k / 16.0k [ data 0.1k bss 11.7k ]
CONFIG_DEBUG_ASSERT_BRIEF
=== build/samus_pd/ ===
FLASH 58.4k / 60.0k [ text 48.5k rodat 9.8k data 0.1k ]
RAM 11.8k / 16.0k [ data 0.1k bss 11.7k ]
undef CONFIG_DEBUG_ASSERT
=== build/samus_pd/ ===
FLASH 57.8k / 60.0k [ text 48.0k rodat 9.7k data 0.1k ]
RAM 11.8k / 16.0k [ data 0.1k bss 11.7k ]
Reviewed-on: https://chromium-review.googlesource.com/241451
Reviewed-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Alec Berg <alecaberg@chromium.org>
Commit-Queue: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
(cherry picked from commit cd90837bb7009fa87e79e56f5497853ad4d2ea2e)
Signed-off-by: Duncan Laurie <dlaurie@chromium.org>
Original-Change-Id: Iebcb54ffe2849d1ad911abe1a5e6e21f3120af86
Reviewed-on: https://chromium-review.googlesource.com/241848
Tested-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Commit-Queue: Duncan Laurie <dlaurie@chromium.org>
(cherry picked from commit 3c26c7b4b6de17af6266d887e1287610bfa533c9)
Signed-off-by: Todd Broch <tbroch@chromium.org>
Change-Id: Ice680175297907497d60788af15a64c01eee6073
Reviewed-on: https://chromium-review.googlesource.com/244168
Reviewed-by: Alec Berg <alecaberg@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Commit-Queue: Todd Broch <tbroch@chromium.org>
| -rw-r--r-- | board/samus_pd/board.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/board/samus_pd/board.h b/board/samus_pd/board.h index 82785aa574..58e917d269 100644 --- a/board/samus_pd/board.h +++ b/board/samus_pd/board.h @@ -25,6 +25,7 @@ #define CONFIG_COMMON_GPIO_SHORTNAMES #undef CONFIG_CONSOLE_CMDHELP #undef CONFIG_CONSOLE_HISTORY +#undef CONFIG_DEBUG_ASSERT #define CONFIG_FORCE_CONSOLE_RESUME #define CONFIG_HIBERNATE_WAKEUP_PINS (STM32_PWR_CSR_EWUP3|STM32_PWR_CSR_EWUP8) #undef CONFIG_HOSTCMD_EVENTS |