[PATCH V3 02/19] OMAP3+: voltage: fix build warning
Kevin Hilman
khilman at ti.com
Thu Mar 17 10:49:22 EDT 2011
Nishanth Menon <nm at ti.com> writes:
> Handle the case for a future SoC where sys_ck_name might be
> uninitialized. Fixes the build warning:
> arch/arm/mach-omap2/voltage.c: In function 'omap_voltage_late_init':
> arch/arm/mach-omap2/voltage.c:86:8: warning: 'sys_ck_name' may be used uninitialized in this function
>
> Signed-off-by: Nishanth Menon <nm at ti.com>
This one seems to be already fixed in l-o commit:
commit c0718df4d666cc5fd8837ac93c82995a17bfdbf5
Author: Paul Walmsley <paul at pwsan.com>
Date: Thu Mar 10 22:17:45 2011 -0700
OMAP2+: voltage: reorganize, split code from data
Kevin
> ---
> arch/arm/mach-omap2/voltage.c | 8 ++++++--
> 1 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/voltage.c b/arch/arm/mach-omap2/voltage.c
> index e0cbd93..56961df 100644
> --- a/arch/arm/mach-omap2/voltage.c
> +++ b/arch/arm/mach-omap2/voltage.c
> @@ -91,10 +91,14 @@ static int __init _config_common_vdd_data(struct omap_vdd_info *vdd)
> * XXX Clockfw should handle this, or this should be in a
> * struct record
> */
> - if (cpu_is_omap24xx() || cpu_is_omap34xx())
> + if (cpu_is_omap24xx() || cpu_is_omap34xx()) {
> sys_ck_name = "sys_ck";
> - else if (cpu_is_omap44xx())
> + } else if (cpu_is_omap44xx()) {
> sys_ck_name = "sys_clkin_ck";
> + } else {
> + pr_err("%s: Unsupported OMAP\n", __func__);
> + return -EINVAL;
> + }
>
> /*
> * Sys clk rate is require to calculate vp timeout value and
More information about the linux-arm-kernel
mailing list