[U-Boot] [PATCH] ARM: EXYNOS: fixed compiler warning message

[U-Boot] [PATCH] ARM: EXYNOS: fixed compiler warning message

[Jaehoon Chung]

Removed [-Wuninitialized] warning message.
The fout_sel is assigned to "-1" by default.
And start, gpio_func is initialized to 0.

Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
 arch/arm/cpu/armv7/exynos/clock.c  |    3 +++
 arch/arm/cpu/armv7/exynos/pinmux.c |    2 +-
 2 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/arch/arm/cpu/armv7/exynos/clock.c b/arch/arm/cpu/armv7/exynos/clock.c
index fc0ed5e..672598f 100644
--- a/arch/arm/cpu/armv7/exynos/clock.c
+++ b/arch/arm/cpu/armv7/exynos/clock.c
@@ -173,6 +173,9 @@ static unsigned long exynos5_get_pll_clk(int pllreg)
 			fout_sel = (pll_div2_sel >> BPLL_FOUT_SEL_SHIFT)
 					& BPLL_FOUT_SEL_MASK;
 			break;
+		default:
+			fout_sel = -1;
+			break;
 		}
 
 		if (fout_sel == 0)
diff --git a/arch/arm/cpu/armv7/exynos/pinmux.c b/arch/arm/cpu/armv7/exynos/pinmux.c
index 822410e..0e91a6c 100644
--- a/arch/arm/cpu/armv7/exynos/pinmux.c
+++ b/arch/arm/cpu/armv7/exynos/pinmux.c
@@ -66,7 +66,7 @@ static int exynos5_mmc_config(int peripheral, int flags)
 	struct exynos5_gpio_part1 *gpio1 =
 		(struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
 	struct s5p_gpio_bank *bank, *bank_ext;
-	int i, start, gpio_func;
+	int i, start = 0, gpio_func = 0;
 
 	switch (peripheral) {
 	case PERIPH_ID_SDMMC0:

[U-Boot] [PATCH] ARM: EXYNOS: fixed compiler warning message

[Rajeshwari Birje]

Hi Jaehoon Chung,

Acked-by: Rajeshwari Shinde <rajeshwari.s@samsung.com>

Thanks and Regards,
Rajeshwari Shinde.

On Tue, Jul 10, 2012 at 12:50 PM, Jaehoon Chung <jh80.chung@samsung.com> wrote:
> Removed [-Wuninitialized] warning message.
> The fout_sel is assigned to "-1" by default.
> And start, gpio_func is initialized to 0.
>
> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> ---
>  arch/arm/cpu/armv7/exynos/clock.c  |    3 +++
>  arch/arm/cpu/armv7/exynos/pinmux.c |    2 +-
>  2 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/exynos/clock.c b/arch/arm/cpu/armv7/exynos/clock.c
> index fc0ed5e..672598f 100644
> --- a/arch/arm/cpu/armv7/exynos/clock.c
> +++ b/arch/arm/cpu/armv7/exynos/clock.c
> @@ -173,6 +173,9 @@ static unsigned long exynos5_get_pll_clk(int pllreg)
>                         fout_sel = (pll_div2_sel >> BPLL_FOUT_SEL_SHIFT)
>                                         & BPLL_FOUT_SEL_MASK;
>                         break;
> +               default:
> +                       fout_sel = -1;
> +                       break;
>                 }
>
>                 if (fout_sel == 0)
> diff --git a/arch/arm/cpu/armv7/exynos/pinmux.c b/arch/arm/cpu/armv7/exynos/pinmux.c
> index 822410e..0e91a6c 100644
> --- a/arch/arm/cpu/armv7/exynos/pinmux.c
> +++ b/arch/arm/cpu/armv7/exynos/pinmux.c
> @@ -66,7 +66,7 @@ static int exynos5_mmc_config(int peripheral, int flags)
>         struct exynos5_gpio_part1 *gpio1 =
>                 (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>         struct s5p_gpio_bank *bank, *bank_ext;
> -       int i, start, gpio_func;
> +       int i, start = 0, gpio_func = 0;
>
>         switch (peripheral) {
>         case PERIPH_ID_SDMMC0:
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] ARM: EXYNOS: fixed compiler warning message

[Minkyu Kang]

On 11 July 2012 13:17, Rajeshwari Birje <rajeshwari.birje@gmail.com> wrote:
> Hi Jaehoon Chung,
>
> Acked-by: Rajeshwari Shinde <rajeshwari.s@samsung.com>
>
> Thanks and Regards,
> Rajeshwari Shinde.
>
> On Tue, Jul 10, 2012 at 12:50 PM, Jaehoon Chung <jh80.chung@samsung.com> wrote:
>> Removed [-Wuninitialized] warning message.
>> The fout_sel is assigned to "-1" by default.
>> And start, gpio_func is initialized to 0.
>>
>> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
>> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
>> ---
>>  arch/arm/cpu/armv7/exynos/clock.c  |    3 +++
>>  arch/arm/cpu/armv7/exynos/pinmux.c |    2 +-
>>  2 files changed, 4 insertions(+), 1 deletions(-)

applied to u-boot-samsung.

Thanks.
Minkyu Kang.
-- 
from. prom.
www.promsoft.net