New compiler warnings with v3.5-rc

New compiler warnings with v3.5-rc

[Paul Walmsley]

Hi Tero,

there are some new compiler warnings added in the v3.5-rc kernels:

arch/arm/mach-omap2/twl-common.c:51:12: warning: 'twl_set_voltage' defined but not used
arch/arm/mach-omap2/twl-common.c:57:12: warning: 'twl_get_voltage' defined but not used

They only seem to appear on my N800 builds:

http://www.pwsan.com/omap/testlogs/am35xx_devel_3.6/20120628014213/build/build_20120628011813_log.txt

but it looks like I somehow am not doing a 2430-specific build yet, so 
they may be 24xx-common.

Anyway they were probably added by:

commit 49c008ecce1f9a549c12e8957668d60008ab0d79
Author: Tero Kristo <t-kristo@ti.com>
Date:   Mon Feb 20 12:26:08 2012 +0200

    arm: omap3: twl: add external controllers for core voltage regulators
    

Maybe you can put together a quick patch to fix these?


- Paul

Re: New compiler warnings with v3.5-rc

[Tero Kristo]

On Thu, 2012-06-28 at 02:47 -0600, Paul Walmsley wrote:
> Hi Tero,
> 
> there are some new compiler warnings added in the v3.5-rc kernels:
> 
> arch/arm/mach-omap2/twl-common.c:51:12: warning: 'twl_set_voltage' defined but not used
> arch/arm/mach-omap2/twl-common.c:57:12: warning: 'twl_get_voltage' defined but not used
> 
> They only seem to appear on my N800 builds:
> 
> http://www.pwsan.com/omap/testlogs/am35xx_devel_3.6/20120628014213/build/build_20120628011813_log.txt
> 
> but it looks like I somehow am not doing a 2430-specific build yet, so 
> they may be 24xx-common.
> 
> Anyway they were probably added by:
> 
> commit 49c008ecce1f9a549c12e8957668d60008ab0d79
> Author: Tero Kristo <t-kristo@ti.com>
> Date:   Mon Feb 20 12:26:08 2012 +0200
> 
>     arm: omap3: twl: add external controllers for core voltage regulators
>     
> 
> Maybe you can put together a quick patch to fix these?

Yea, I can do that. Can you provide your .config as reference?

-Tero

Re: New compiler warnings with v3.5-rc

[S, Venkatraman]

On Thu, Jun 28, 2012 at 2:35 PM, Tero Kristo <t-kristo@ti.com> wrote:
> On Thu, 2012-06-28 at 02:47 -0600, Paul Walmsley wrote:
>> Hi Tero,
>>
>> there are some new compiler warnings added in the v3.5-rc kernels:
>>
>> arch/arm/mach-omap2/twl-common.c:51:12: warning: 'twl_set_voltage' defined but not used
>> arch/arm/mach-omap2/twl-common.c:57:12: warning: 'twl_get_voltage' defined but not used
>>
>> They only seem to appear on my N800 builds:
>>
>> http://www.pwsan.com/omap/testlogs/am35xx_devel_3.6/20120628014213/build/build_20120628011813_log.txt
>>
>> but it looks like I somehow am not doing a 2430-specific build yet, so
>> they may be 24xx-common.
>>
>> Anyway they were probably added by:
>>
>> commit 49c008ecce1f9a549c12e8957668d60008ab0d79
>> Author: Tero Kristo <t-kristo@ti.com>
>> Date:   Mon Feb 20 12:26:08 2012 +0200
>>
>>     arm: omap3: twl: add external controllers for core voltage regulators
>>
>>
>> Maybe you can put together a quick patch to fix these?
>
> Yea, I can do that. Can you provide your .config as reference?
>
You can generate one quickly using these steps..
make omap2plus_defconfig
scripts/config --disable ARCH_OMAP4
scripts/config --disable ARCH_OMAP3
make oldnoconfig
make uImage
... and it should expose the warning.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html