Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69764 )
Change subject: libpayload: Fix compiler warning ......................................................................
libpayload: Fix compiler warning
drivers/serial/8250.c:75: [-Werror=unused-variable] Move variable declaration inside the `#if !CONFIG(LP_PL011_SERIAL_CONSOLE)` block
drivers/udc/dwc2.c:505: [-Werror=format=] use `%zd` to match type `size_t`
Change-Id: Id285c24cba790f181fa203f3117e5df35bed27c4 Signed-off-by: Thomas Heijligen src@posteo.de --- M payloads/libpayload/drivers/serial/8250.c M payloads/libpayload/drivers/udc/dwc2.c 2 files changed, 19 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/69764/1
diff --git a/payloads/libpayload/drivers/serial/8250.c b/payloads/libpayload/drivers/serial/8250.c index 1df34c4..f3d98a9 100644 --- a/payloads/libpayload/drivers/serial/8250.c +++ b/payloads/libpayload/drivers/serial/8250.c @@ -72,9 +72,9 @@ static void serial_hardware_init(int speed, int word_bits, int parity, int stop_bits) { +#if !CONFIG(LP_PL011_SERIAL_CONSOLE) unsigned char reg;
-#if !CONFIG(LP_PL011_SERIAL_CONSOLE) /* Disable interrupts. */ serial_write_reg(0, 0x01);
diff --git a/payloads/libpayload/drivers/udc/dwc2.c b/payloads/libpayload/drivers/udc/dwc2.c index 6541308..6c72fba 100644 --- a/payloads/libpayload/drivers/udc/dwc2.c +++ b/payloads/libpayload/drivers/udc/dwc2.c @@ -502,7 +502,7 @@ } SIMPLEQ_REMOVE_HEAD(&p->eps[endpoint][in_dir].job_queue, queue);
- usb_debug("%d-%d: scheduled %zd, now %d bytes\n", endpoint, in_dir, + usb_debug("%d-%d: scheduled %zd, now %zd bytes\n", endpoint, in_dir, job->length, job->xfered_length);
if (this->current_config &&