Hi Simon,

On 2015年11月06日 20:06, Simon Glass wrote:
diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h
index e3d3fc7..0ab6128 100644
--- a/include/linux/mtd/mtd.h
+++ b/include/linux/mtd/mtd.h
@@ -18,6 +18,7 @@

  #include <asm/div64.h>
  #else
+#include <dm.h>

I'm not keen on adding this header here. Why is it needed? Can we
instead include <m.h> in the C files that need it?


It is needed for the udevice in mtd_info. Some drivers use mtd.h but are not converted to driver model. Maybe I should remove the dm here, and add an #elif for the udevice below?

  #include <linux/compat.h>
  #include <mtd/mtd-abi.h>
  #include <asm/errno.h>
@@ -272,6 +273,8 @@ struct mtd_info {
         struct module *owner;
  #ifndef __UBOOT__
         struct device dev;
+#else

#elif CONFIG_IS_ENABLED(MTD)

+       struct udevice *dev;
  #endif
         int usecount;
  };

Thanks a lot for your review.

Best regards,
Thomas
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to