summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/soc/amd/picasso/include/soc/uart.h4
-rw-r--r--src/soc/amd/picasso/uart.c8
2 files changed, 6 insertions, 6 deletions
diff --git a/src/soc/amd/picasso/include/soc/uart.h b/src/soc/amd/picasso/include/soc/uart.h
index 3aac936dff..4e5619fa86 100644
--- a/src/soc/amd/picasso/include/soc/uart.h
+++ b/src/soc/amd/picasso/include/soc/uart.h
@@ -5,9 +5,9 @@
#include <types.h>
-void set_uart_config(int idx); /* configure hardware of FCH UART selected by idx */
+void set_uart_config(unsigned int idx); /* configure hardware of FCH UART selected by idx */
void clear_uart_legacy_config(void); /* disable legacy I/O decode for FCH UART */
-uintptr_t get_uart_base(int idx); /* get MMIO base address of FCH UART */
+uintptr_t get_uart_base(unsigned int idx); /* get MMIO base address of FCH UART */
#endif /* __PICASSO_UART_H__ */
diff --git a/src/soc/amd/picasso/uart.c b/src/soc/amd/picasso/uart.c
index d892333ca1..0bf7eb88d7 100644
--- a/src/soc/amd/picasso/uart.c
+++ b/src/soc/amd/picasso/uart.c
@@ -32,9 +32,9 @@ static const struct _uart_info {
} },
};
-uintptr_t get_uart_base(int idx)
+uintptr_t get_uart_base(unsigned int idx)
{
- if (idx < 0 || idx >= ARRAY_SIZE(uart_info))
+ if (idx >= ARRAY_SIZE(uart_info))
return 0;
return uart_info[idx].base;
@@ -45,12 +45,12 @@ void clear_uart_legacy_config(void)
write16((void *)FCH_UART_LEGACY_DECODE, 0);
}
-void set_uart_config(int idx)
+void set_uart_config(unsigned int idx)
{
uint32_t uart_ctrl;
uint16_t uart_leg;
- if (idx < 0 || idx >= ARRAY_SIZE(uart_info))
+ if (idx >= ARRAY_SIZE(uart_info))
return;
program_gpios(uart_info[idx].mux, 2);