[PATCH/RFC] MTD: Striping layer core
Alexander Belyakov
alexander.belyakov at intel.com
Fri Mar 31 01:52:59 EST 2006
Jörn Engel wrote:
> On Thu, 30 March 2006 11:57:37 +0400, Belyakov, Alexander wrote:
>> diff -uNr a/drivers/mtd/maps/mphysmap.c b/drivers/mtd/maps/mphysmap.c
>> --- a/drivers/mtd/maps/mphysmap.c 2006-03-28 12:08:28.000000000
>> +0400
>> +++ b/drivers/mtd/maps/mphysmap.c 2006-03-28 12:10:48.000000000
>> +0400
>> @@ -12,6 +12,9 @@
>> #ifdef CONFIG_MTD_PARTITIONS
>> #include <linux/mtd/partitions.h>
>> #endif
>> +#ifdef CONFIG_MTD_CMDLINE_STRIPE
>> +#include <linux/mtd/stripe.h>
>> +#endif
>
> Move the #ifdef into the header.
May I ask what is the reason for that (taking into account that
partitions.h is already included here the same way for the same purpose)?
> Your code suffers from a lot of very basic things that make it hard to
> review.
What exactly you wish to see here? Hope I can bring clarifications by
changing code/comments or via this mailing list.
Thanks,
Alexander Belyakov
More information about the linux-mtd
mailing list