commit 25866fe89321ca57aae4dde062db4c88086e187b
author Felix Held <felix-coreboot@felixheld.de> Thu Sep 29 16:06:45 2022 +0200
committer Felix Held <felix-coreboot@felixheld.de> Fri Sep 30 14:49:25 2022 +0000
tree e357176b1ae2aa477fdc0a1b405e747ef7c14e9b
parent f69cb29c20e40750e7b0595852ea9030050ebe2a

soc/amd/cezanne,mendocino,picasso/uart: use write16p to avoid typecasts

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I6e743068dfcf9d393096f775759181af1a1c470d
Reviewed-on: https://review.coreboot.org/c/coreboot/+/67979
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>

src/soc/amd/cezanne/uart.c

diff --git a/src/soc/amd/cezanne/uart.c b/src/soc/amd/cezanne/uart.c
index 7917ebd..64252ca 100644
--- a/src/soc/amd/cezanne/uart.c
+++ b/src/soc/amd/cezanne/uart.c
@@ -37,7 +37,7 @@
 
 void clear_uart_legacy_config(void)
 {
-	write16((void *)FCH_LEGACY_UART_DECODE, 0);
+	write16p(FCH_LEGACY_UART_DECODE, 0);
 }
 
 void set_uart_config(unsigned int idx)

src/soc/amd/mendocino/uart.c

diff --git a/src/soc/amd/mendocino/uart.c b/src/soc/amd/mendocino/uart.c
index 66e0cf1..ac98a79 100644
--- a/src/soc/amd/mendocino/uart.c
+++ b/src/soc/amd/mendocino/uart.c
@@ -49,7 +49,7 @@
 
 void clear_uart_legacy_config(void)
 {
-	write16((void *)FCH_LEGACY_UART_DECODE, 0);
+	write16p(FCH_LEGACY_UART_DECODE, 0);
 }
 
 void set_uart_config(unsigned int idx)

src/soc/amd/picasso/uart.c

diff --git a/src/soc/amd/picasso/uart.c b/src/soc/amd/picasso/uart.c
index 5763a5b..4ae4c5c 100644
--- a/src/soc/amd/picasso/uart.c
+++ b/src/soc/amd/picasso/uart.c
@@ -45,7 +45,7 @@
 
 void clear_uart_legacy_config(void)
 {
-	write16((void *)FCH_LEGACY_UART_DECODE, 0);
+	write16p(FCH_LEGACY_UART_DECODE, 0);
 }
 
 void set_uart_config(unsigned int idx)