diff options
author | Patrick Georgi <pgeorgi@google.com> | 2018-05-02 19:03:38 +0200 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-05-18 10:05:13 -0700 |
commit | 85ddb2ce533cb0276aab7780238ec98e1abec2fe (patch) | |
tree | b434418120801569b00af876c36b90080ca5f1f0 /chip | |
parent | e5f3ee270a0e6d7ef8bc9965dad27b11cffda367 (diff) | |
download | chrome-ec-85ddb2ce533cb0276aab7780238ec98e1abec2fe.tar.gz |
Shuffle const around
gcc 8.1 complains about duplicate const, and while some of these really
are duplicate, others look like they were supposed to tighten the API
contract so that variables are "const pointer to const data", but didn't
have that effect.
BUG=b:65441143
BRANCH=none
TEST=building Chrome EC as part of upstream coreboot's build with a
gcc 8.1 compiler now works (better. there are other issues left)
Change-Id: I6016c5f282516471746f08d5714ea07ebdd10331
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1039812
Commit-Ready: Patrick Georgi <pgeorgi@chromium.org>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Stefan Reinauer <reinauer@google.com>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'chip')
-rw-r--r-- | chip/g/idle.c | 2 | ||||
-rw-r--r-- | chip/npcx/flash.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/chip/g/idle.c b/chip/g/idle.c index 0bff45da84..dd745ec766 100644 --- a/chip/g/idle.c +++ b/chip/g/idle.c @@ -33,7 +33,7 @@ static enum { static int idle_default; -static const char const *idle_name[] = { +static const char *const idle_name[] = { "invalid", "wfi", "sleep", diff --git a/chip/npcx/flash.c b/chip/npcx/flash.c index 219347e17e..ccb5dd2470 100644 --- a/chip/npcx/flash.c +++ b/chip/npcx/flash.c @@ -399,7 +399,7 @@ static void flash_burst_write(unsigned int dest_addr, unsigned int bytes, } static int flash_program_bytes(uint32_t offset, uint32_t bytes, - const uint8_t const *data) + const uint8_t *data) { int write_size; int rv; |