Thread (23 messages) 23 messages, 6 authors, 2011-05-24
DORMANTno replies
Revisions (17)
  1. v1 [diff vs current]
  2. v2 [diff vs current]
  3. v3 [diff vs current]
  4. v4 [diff vs current]
  5. v4 [diff vs current]
  6. v4 [diff vs current]
  7. v4 [diff vs current]
  8. v5 [diff vs current]
  9. v5 [diff vs current]
  10. v5 [diff vs current]
  11. v5 [diff vs current]
  12. v6 [diff vs current]
  13. v6 [diff vs current]
  14. v5 current
  15. v6 [diff vs current]
  16. v6 [diff vs current]
  17. v6 [diff vs current]

[PATCH v5 1/8] MTD: Add integrator-flash feature to physmap

From: Mike Frysinger <hidden>
Date: 2011-05-24 04:43:21

On Mon, May 16, 2011 at 10:59, Marc Zyngier wrote:
quoted hunk ↗ jump to hunk
--- a/include/linux/mtd/physmap.h
+++ b/include/linux/mtd/physmap.h
@@ -22,6 +22,8 @@ struct map_info;
?struct physmap_flash_data {
? ? ? ?unsigned int ? ? ? ? ? ?width;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? (*init)(struct platform_device *);
+ ? ? ? void ? ? ? ? ? ? ? ? ? ?(*exit)(struct platform_device *);
? ? ? ?void ? ? ? ? ? ? ? ? ? ?(*set_vpp)(struct map_info *, int);
? ? ? ?unsigned int ? ? ? ? ? ?nr_parts;
? ? ? ?unsigned int ? ? ? ? ? ?pfow_base;
you use the platform_device structure but never include any header for
it.  please add "struct platform_device;" to the top of this file
after the #includes.

include/linux/mtd/physmap.h:25: warning: ?struct platform_device?
declared inside parameter list
include/linux/mtd/physmap.h:25: warning: its scope is only this
definition or declaration, which is probably not what you want
include/linux/mtd/physmap.h:26: warning: ?struct platform_device?
declared inside parameter list
include/linux/mtd/physmap.h:27: warning: ?struct platform_device?
declared inside parameter list
-mike
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help