From ee10cce9c49be36e11d313487861f54e236e80b4 Mon Sep 17 00:00:00 2001 From: Lukasz Majewski Date: Wed, 5 Mar 2014 17:01:10 +0100 Subject: [PATCH] MAINLINE pinmux:exynos:uart:fix: Add default statement to remove warnings (gcc-4.8.2) For Exynos' pinmux UART implementation there was a possibility to run for() statement with uninitialized bank, start and count values. Those warnings appear when following toolchain is used (gcc-4.8.2-glibc-2.18-binutils-2.24): warning: 'count' may be used uninitialized in this function [-Wmaybe-uninitialized] Signed-off-by: Lukasz Majewski Cc: Minkyu Kang --- arch/arm/cpu/armv7/exynos/pinmux.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/arch/arm/cpu/armv7/exynos/pinmux.c b/arch/arm/cpu/armv7/exynos/pinmux.c index 645c497..494f608 100644 --- a/arch/arm/cpu/armv7/exynos/pinmux.c +++ b/arch/arm/cpu/armv7/exynos/pinmux.c @@ -39,6 +39,9 @@ static void exynos5_uart_config(int peripheral) start = 4; count = 2; break; + default: + error("UART device %d not implemented\n", peripheral); + return; } for (i = start; i < start + count; i++) { s5p_gpio_set_pull(bank, i, GPIO_PULL_NONE); @@ -74,6 +77,9 @@ static void exynos5420_uart_config(int peripheral) start = 4; count = 2; break; + default: + error("UART device %d not implemented\n", peripheral); + return; } for (i = start; i < start + count; i++) { @@ -683,6 +689,9 @@ static void exynos4_uart_config(int peripheral) start = 4; count = 2; break; + default: + error("UART device %d not implemented\n", peripheral); + return; } for (i = start; i < start + count; i++) { s5p_gpio_set_pull(bank, i, GPIO_PULL_NONE); -- 2.7.4