diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2017-10-21 10:10:27 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-11-03 15:22:06 +0000 |
| commit | 11015342b4fb7a799bfdf7529da2f190ad2c447c (patch) | |
| tree | 036113593e82ffab6e53e7e02a299774f677c07f /src/soc/intel/quark | |
| parent | c649fbbf29e5396570e3d6277af9cb086040f2e8 (diff) | |
| download | coreboot-11015342b4fb7a799bfdf7529da2f190ad2c447c.tar.xz | |
soc/intel/quark/spi: Correct conversion specifier
Use the correct conversion specifier for `size_t` to fix the error
below.
```
from src/soc/intel/quark/spi.c:18:
src/soc/intel/quark/spi.c: In function 'xfer':
src/soc/intel/quark/spi.c:107:20: error: format '%ld' expects argument \
of type 'long int', but argument 3 has type 'unsigned int' \
[-Werror=format=]
printk(BIOS_ERR, "bytesin > %ld\n", sizeof(ctrlr->data));
^
```
Found-by: gcc (Debian 7.2.0-8) 7.2.0
Change-Id: I3974d116e85715086a2bd5533a80a20c4cc43303
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: https://review.coreboot.org/22130
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/intel/quark')
| -rw-r--r-- | src/soc/intel/quark/spi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/intel/quark/spi.c b/src/soc/intel/quark/spi.c index c943567e1d..a6647fb71a 100644 --- a/src/soc/intel/quark/spi.c +++ b/src/soc/intel/quark/spi.c @@ -104,7 +104,7 @@ static int xfer(const struct spi_slave *slave, const void *dout, /* Validate the buffer sizes */ if (bytesin > sizeof(ctrlr->data)) { - printk(BIOS_ERR, "bytesin > %ld\n", sizeof(ctrlr->data)); + printk(BIOS_ERR, "bytesin > %zu\n", sizeof(ctrlr->data)); goto error; } |