On Mon, May 30, 2011 at 11:44:01PM +0200, Janusz Krzysztofik wrote:
> Include <linux/platform_device.h> from 
> arch/arm/plat-omap/include/plat/flash.h, otherwise compilation may break 
> with:
> 
> In file included from arch/arm/mach-omap1/flash.c:15:
> arch/arm/plat-omap/include/plat/flash.h:14: warning: 'struct platform_device' 
> declared inside parameter list
> arch/arm/plat-omap/include/plat/flash.h:14: warning: its scope is only this 
> definition or declaration, which is probably not what you want
> arch/arm/mach-omap1/flash.c:16: warning: 'struct platform_device' declared 
> inside parameter list
> arch/arm/mach-omap1/flash.c:17: error: conflicting types for 'omap1_set_vpp'
> arch/arm/plat-omap/include/plat/flash.h:14: error: previous declaration of 
> 'omap1_set_vpp' was here

Just forward-declare the struct by adding:

struct platform_device;

in this header; there's no need to add to the complexities of header file
inclusion...
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to