Re: Latest build results - errors/warnings - lots of them

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Arnd, Tony,

On 02-05-2013 11:38, Tony Lindgren wrote:
> * Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> [130502 01:27]:
>> On Tue, Apr 30, 2013 at 09:17:39AM +0100, Russell King - ARM Linux wrote:
>>> Latest nightly build of 3.9+my for-next+arm-soc's for-next results in a
>>> great load of new warnings and errors.  arch/arm/common/mcpm_head.S,
>>> arch/arm/common/mcpm_platsmp.c, arch/arm/common/vlock.S are the biggest
>>> source of errors.
>>>
>>> OMAP stuff needs a serious look at too - much Kconfig madness there
>>> caused by over-use of select, which then goes on to cause build errors
>>> because it assumes some stuff is always enabled.
>>>
>>> There's also warnings about of_device_id from include/linux/of_platform.h
>>> via from arch/arm/kernel/setup.c which feature in all the non-OF builds
>>> too which need addressing.
>>>
>>> See todays http://www.arm.linux.org.uk/developer/build/ results for all
>>> the details and configs.  Not pushing my tree until some of this stuff
>>> gets fixed.
>>
>> And now we have a new bunch of warnings from OMAP stuff which weren't
>> previously there...
>>
>> arch/arm/mach-omap2/omap_device.c: In function 'omap_device_get_by_hwmod_name':
>> arch/arm/mach-omap2/omap_device.c:821:3: warning: return makes pointer from integer without a cast
>> arch/arm/mach-omap2/omap_device.c:826:3: warning: return makes pointer from integer without a cast
> 
> Hmm these I already fixed earlier along with a merge resolution, and
> I'm not seeing them in next/master or arm-soc/for-next. What do you
> have merged into your current tree?
> 
>> drivers/power/twl4030_charger.c: In function 'twl4030_charger_enable_usb':
>> drivers/power/twl4030_charger.c:192:20: warning: ignoring return value of 'regulator_enable', declared with attribute warn_unused_result
> 
> Neil, care to provide a fix for this? It's from your commit ab37813
> (twl4030_charger: Allow charger to control the regulator that feeds it).
> 
>> include/linux/thermal.h:254:12: warning: 'thermal_generate_netlink_event' defined but not used
> 
> Eduardo, can you fix that one? It seems to be from your commit 8ab3e6a
> (thermal: Use thermal zone device id in netlink messages).

Yeah sure I can fix it. As simple as the following:
From c04244c87312f5bfc61d9e12ba3fbaa0fdd81adb Mon Sep 17 00:00:00 2001
From: Eduardo Valentin <eduardo.valentin@xxxxxx>
Date: Thu, 2 May 2013 12:58:20 -0400
Subject: [PATCH 1/1] thermal: remove stub for thermal_generate_netlink_event

This patch removes the stub for thermal_generate_netlink_event
because this function is not used anywhere inside the kernel.

In case CONFIG_NET is not set we get:
include/linux/thermal.h:254:12: warning:
'thermal_generate_netlink_event' defined but not used

Thus removing it.

Signed-off-by: Eduardo Valentin <eduardo.valentin@xxxxxx>
---
 include/linux/thermal.h | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/include/linux/thermal.h b/include/linux/thermal.h
index e3c0ae9..e3f3cba 100644
--- a/include/linux/thermal.h
+++ b/include/linux/thermal.h
@@ -250,12 +250,6 @@ void thermal_unregister_governor(struct
thermal_governor *);
 #ifdef CONFIG_NET
 extern int thermal_generate_netlink_event(struct thermal_zone_device *tz,
 						enum events event);
-#else
-static int thermal_generate_netlink_event(struct thermal_zone_device *tz,
-						enum events event)
-{
-	return 0;
-}
 #endif

 #endif /* __THERMAL_H__ */
-- 
1.8.2.1.342.gfa7285d

However,...

Rui, Durga,

 That function was used in the previous thermal  layer design. Now the
notification is done via sysfs, not netlink. The netlink interface needs
to be flagged as DEPRECATED and then we would remove it.

In any case, the above function is used by no one within the kernel:
$ git grep thermal_generate_netlink_event
Documentation/thermal/sysfs-api.txt:just need to call
thermal_generate_netlink_event() with two arguments viz
drivers/thermal/thermal_core.c:int thermal_generate_netlink_event(struct
thermal_zone_device *tz,
drivers/thermal/thermal_core.c:EXPORT_SYMBOL_GPL(thermal_generate_netlink_event);
include/linux/thermal.h:extern int thermal_generate_netlink_event(struct
thermal_zone_device *tz,
include/linux/thermal.h:static inline int
thermal_generate_netlink_event(struct thermal_zone_device *tz,


For that reason, I d propose to simply remove it, as I do not have any
user space application that uses that net link interface. But that needs
a confirmation from the Intel folks.

Rui, Durga? Last merge window we discussed to remove this API, do you
guys have dependency on this one still?


Cheers,

> 
> Regards,
> 
> Tony
> 
> 


Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux