Re: [RFC] ARM: sa1100: collie-battery.c: include platform_device.h directly
From: Andrea Adami
Date: Sun Feb 17 2013 - 19:14:06 EST
On Sun, Feb 17, 2013 at 12:00 AM, Anton Vorontsov <anton@xxxxxxxxxx> wrote:
> Hi Andrea,
> On Sat, Feb 16, 2013 at 07:16:00PM +0100, Andrea Adami wrote:
>> Initial try to fix compilation error
> Things are a bit more complex...
>> linux/include/linux/mfd/mcp.h:22:16: error: field 'attached_device' has incomplete type
> You have to add #include <linux/device.h> into mfd/mcp.h. This will add
> struct device and...
Bingo.Adding it solves all following errors, no need for further inclusions.
>> linux/include/linux/mfd/mcp.h:48:23: error: field 'drv' has incomplete type
> struct device_driver.
>> linux/include/linux/mfd/ucb1x00.h:137:17: error: field 'dev' has incomplete type
> Same here, the file ucb1x00.h needs #include <linux/device.h>.
Seems not needed, mcp.h is included by ucb1x00.h
>> linux/drivers/power/collie_battery.c:290:56: error: unknown type name 'pm_message_t'
> #include <linux/pm.h> into collie_battery.c
Seems not needed, is included in mcp-sa11x0.c (included by collie_battery.c)
The warning remains, though:
linux/drivers/power/collie_battery.c:372:2: warning: initialization
from incompatible pointer type [enabled by default]
Thank you for your attention
>> linux/drivers/power/collie_battery.c:371:13: error: 'collie_bat_suspend' undeclared here (not in a function)
>> | make: *** [drivers/power/collie_battery.o] Error 1
>> | make: *** [drivers/power] Error 2
>> As seen in Arnd Bergmann patches on LAKML
>> [PATCH 7/9] ARM defconfigs: add missing inclusions of linux/platform_device.h
>> [PATCH 1/9] ARM: sa1100/assabet: include platform_device.h directly
>> Compilation is still not clean, two warnings:
>> linux/drivers/power/collie_battery.c:372:2: warning: initialization from incompatible pointer type [enabled by default]
>> linux/drivers/power/collie_battery.c:372:2: warning: (near initialization for 'collie_bat_driver.suspend') [enabled by default]
>> Signed-off-by: Andrea Adami <andrea.adami@xxxxxxxxx>
>> drivers/power/collie_battery.c | 1 +
>> drivers/video/logo/logo_linux_clut224.ppm | 232006 ++++++++++++++++++++++++++-
>> 2 files changed, 230404 insertions(+), 1603 deletions(-)
>> diff --git a/drivers/power/collie_battery.c b/drivers/power/collie_battery.c
>> index c58d0e3..52cbab3 100644
>> --- a/drivers/power/collie_battery.c
>> +++ b/drivers/power/collie_battery.c
>> @@ -14,6 +14,7 @@
>> #include <linux/module.h>
>> #include <linux/power_supply.h>
>> #include <linux/delay.h>
>> +#include <linux/platform_device.h>
> I don't see anywhere in the driver which would need platform_device.h...
> But pm.h, yes.
Sorry, I'm not a kernel developer. It was a desperate try to unbrick build.
I'm just maintaining the layer needed to build Zaurus images using
Up to now, I could only build a working linux-as-bootloader kernel
(kexecboot) for collie with old 2.6.31+ sources:
userspace is not a problem now but the poor device is suffering some
serious bitrot :/
>> #include <linux/spinlock.h>
>> #include <linux/interrupt.h>
>> #include <linux/gpio.h>
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/