diff options
| author | Jim Pollock <jmpollock@chromium.org> | 2022-09-04 23:53:48 +0000 |
|---|---|---|
| committer | Jim Pollock <jmpollock@chromium.org> | 2022-09-04 23:58:58 +0000 |
| commit | 8cc24f9c7f076d25869a167efdc504b1e49947e1 (patch) | |
| tree | b7947bf3a795f3b97b74ccd4522394842a38e361 | |
| parent | fadf897cf685e29a5d6101ed735d70337380a458 (diff) | |
| download | chrome-ec-8cc24f9c7f076d25869a167efdc504b1e49947e1.tar.gz | |
Revert "ec: Fix DEBUG "variable" in usb_updater Makefile"
This reverts commit af5a3f99166f5c395c14add96af0bc13516946c9.
Reason for revert: Change has broken most builders
Original change's description:
> ec: Fix DEBUG "variable" in usb_updater Makefile
>
> DEBUG is not the way to refer to a variable in a Makefile expression;
> $(DEBUG) is.
>
> Also, imitate the =1 check from extra/rma_reset/Makefile.
>
> BUG=none
> TEST=build
>
> Change-Id: Ic71be497ca83041dfdedf63c10e7c74b38c05bc8
> Signed-off-by: Brian Norris <briannorris@chromium.org>
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3872547
> Commit-Queue: Mary Ruthven <mruthven@chromium.org>
> Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Bug: none
Change-Id: I4950c7f081f04f1dad0091dd2d7abe64426b3518
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3874108
Tested-by: Jim Pollock <jmpollock@chromium.org>
Owners-Override: Jim Pollock <jmpollock@chromium.org>
Reviewed-by: Dan Callaghan <dcallagh@chromium.org>
| -rw-r--r-- | extra/usb_updater/Makefile | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/extra/usb_updater/Makefile b/extra/usb_updater/Makefile index f1ec4f8b0e..04adfff4dd 100644 --- a/extra/usb_updater/Makefile +++ b/extra/usb_updater/Makefile @@ -19,10 +19,10 @@ CFLAGS := -std=gnu99 \ -Wredundant-decls \ -Wmissing-declarations -ifeq ($(DEBUG),1) -CFLAGS += -O0 -else +ifeq (DEBUG,) CFLAGS += -O3 +else +CFLAGS += -O0 endif # |