diff mbox series

[v3,6/9] serial: msm-geni: fix a compile time warning from msm_serial_setbrg()

Message ID 20230421175041.275566-7-vladimir.zapolskiy@linaro.org
State Accepted
Commit babdadc8a51ec54157d9e050eaf3f745052e1583
Headers show
Series serial: msm-geni: fix UART baudrate on modern platforms | expand

Commit Message

Vladimir Zapolskiy April 21, 2023, 5:50 p.m. UTC 
A compiler warns about a missing function prototype, which is valid
and fixed by converting the function into static one, also fix
interleaved local variable declarations and assignments.

Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Fixes: 324df15a292e ("serial: qcom: add support for GENI serial driver")
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 drivers/serial/serial_msm_geni.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/drivers/serial/serial_msm_geni.c b/drivers/serial/serial_msm_geni.c
index 55dd9188a56c..3a200f45a6ce 100644
--- a/drivers/serial/serial_msm_geni.c
+++ b/drivers/serial/serial_msm_geni.c
@@ -238,13 +238,13 @@  static inline void geni_serial_baud(phys_addr_t base_address, u32 clk_div,
 	writel(s_clk_cfg, base_address + GENI_SER_S_CLK_CFG);
 }
 
-int msm_serial_setbrg(struct udevice *dev, int baud)
+static int msm_serial_setbrg(struct udevice *dev, int baud)
 {
 	struct msm_serial_data *priv = dev_get_priv(dev);
+	u64 clk_rate;
+	u32 clk_div;
 
 	priv->baud = baud;
-	u32 clk_div;
-	u64 clk_rate;
 
 	clk_rate = get_clk_div_rate(baud, UART_OVERSAMPLING, &clk_div);
 	geni_serial_set_clock_rate(dev, clk_rate);