[U-Boot] [PATCH v4 1/3] dm: implement a MTD uclass

[Thomas Chou]

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