diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-10 17:56:44 -0800 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-15 05:05:04 +0100 |
| commit | 36984d85e758bd7b280a835c5c6e5c68b10e82b6 (patch) | |
| tree | 62b7bdc244d5d3c25120fe37fc107de87468a92b | |
| parent | 491c5b60d01d5fa7d27c281335343e67519e799c (diff) | |
| download | coreboot-36984d85e758bd7b280a835c5c6e5c68b10e82b6.tar.xz | |
src/lib: Clean up general issues found by checkpatch.pl
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: Bad function definition - void init_timer() should probably be void init_timer(void)
ERROR: Prefixing 0x with decimal output is defective
WARNING: Comparisons should place the constant on the right side of the test
WARNING: char * array declaration might be better as static const
TEST=Build and run on Galileo Gen2
Change-Id: I9f618eea95e1f92fa34f4f89da27c0b16ae7f4ee
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18763
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
| -rw-r--r-- | src/lib/edid.c | 2 | ||||
| -rw-r--r-- | src/lib/gcov-glue.c | 2 | ||||
| -rw-r--r-- | src/lib/reg_script.c | 4 | ||||
| -rw-r--r-- | src/lib/timer.c | 2 | ||||
| -rw-r--r-- | src/lib/tlcl.c | 2 |
5 files changed, 6 insertions, 6 deletions
diff --git a/src/lib/edid.c b/src/lib/edid.c index 95ff92f22a..58dd813dd2 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -117,7 +117,7 @@ static int detailed_cvt_descriptor(unsigned char *x, int first) { const unsigned char empty[3] = { 0, 0, 0 }; - const char *names[] = { "50", "60", "75", "85" }; + static const char *names[] = { "50", "60", "75", "85" }; int width = 0, height = 0; int valid = 1; int fifty = 0, sixty = 0, seventyfive = 0, eightyfive = 0, reduced = 0; diff --git a/src/lib/gcov-glue.c b/src/lib/gcov-glue.c index 4e46367ab1..a002ea790f 100644 --- a/src/lib/gcov-glue.c +++ b/src/lib/gcov-glue.c @@ -114,7 +114,7 @@ static size_t fread(void *ptr, size_t size, size_t nmemb, FILE *stream) static size_t fwrite(const void *ptr, size_t size, size_t nmemb, FILE *stream) { #if CONFIG_DEBUG_COVERAGE - printk(BIOS_DEBUG, "fwrite: %zd * 0x%zd bytes to file %s\n", + printk(BIOS_DEBUG, "fwrite: %zd * %zd bytes to file %s\n", nmemb, size, stream->filename); #endif // TODO check if file is last opened file and fail otherwise. diff --git a/src/lib/reg_script.c b/src/lib/reg_script.c index e3b453d2d0..8d813b6f8d 100644 --- a/src/lib/reg_script.c +++ b/src/lib/reg_script.c @@ -476,7 +476,7 @@ static uint64_t reg_script_read(struct reg_script_context *ctx) /* Read from the platform specific bus */ bus = find_bus(step); - if (NULL != bus) { + if (bus != NULL) { value = bus->reg_script_read(ctx); break; } @@ -530,7 +530,7 @@ static void reg_script_write(struct reg_script_context *ctx) /* Write to the platform specific bus */ bus = find_bus(step); - if (NULL != bus) { + if (bus != NULL) { bus->reg_script_write(ctx); break; } diff --git a/src/lib/timer.c b/src/lib/timer.c index 0796d99ccf..58d32ee8e4 100644 --- a/src/lib/timer.c +++ b/src/lib/timer.c @@ -18,7 +18,7 @@ #include <delay.h> #include <thread.h> -__attribute__((weak)) void init_timer() { /* do nothing */ } +__attribute__((weak)) void init_timer(void) { /* do nothing */ } void udelay(unsigned int usec) { diff --git a/src/lib/tlcl.c b/src/lib/tlcl.c index 34da816adc..49854cbdcb 100644 --- a/src/lib/tlcl.c +++ b/src/lib/tlcl.c @@ -86,7 +86,7 @@ static uint32_t tlcl_send_receive_no_retry(const uint8_t *request, result = tpm_send_receive(request, tpm_command_size(request), response, &response_length); - if (0 != result) { + if (result != 0) { /* Communication with TPM failed, so response is garbage */ VBDEBUG("TPM: command 0x%x send/receive failed: 0x%x\n", tpm_command_code(request), result); |