MAINLINE pinmux:exynos:uart:fix: Add default statement to remove warnings (gcc-4...
authorLukasz Majewski <l.majewski@samsung.com>
Wed, 5 Mar 2014 16:01:10 +0000 (17:01 +0100)
committerLukasz Majewski <l.majewski@samsung.com>
Tue, 18 Mar 2014 15:48:59 +0000 (16:48 +0100)
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 <l.majewski@samsung.com>
Cc: Minkyu Kang <mk7.kang@samsung.com>
arch/arm/cpu/armv7/exynos/pinmux.c

index 645c497..494f608 100644 (file)
@@ -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);