[PATCH v4] MTD: nand: add support for diskonchip G4 nand flash device

Robert Jarzmik robert.jarzmik at free.fr
Sun Jan 15 06:06:44 EST 2012


Mike Dunn <mikedunn at newsguy.com> writes:

> On 01/10/2012 12:25 AM, Artem Bityutskiy wrote:
>>
>> Would be nice if DoC guys could also fix huge amount of GCC warnings -
>> there are DoC I/O macro definitions which produce a warning every time
>> they are used - very annoying noise. I guess those macros can be turned
>> into static inline functions? Would someone take the initiative?
>
>
> Yeah, this thought crossed my mind every time I recompiled while working on the
> aborted api change patches.  Gives DoC a bad name.  I'll get to it shortly.
>
> Ping Robert.  Care to take a look at the G4 driver?

I looked through. It looks good to me.
I have one warning remaining (on top of Linus tree):
drivers/mtd/nand/docg4.c: In function '__check_ignore_badblocks':
drivers/mtd/nand/docg4.c:52: warning: return from incompatible pointer type

My feeling is that the driver is good to be sent upstream, with my:
Reviewed-by: Robert Jarzmik <robert.jarzmik at free.fr>

If anything comes up, it can be fixed later, and I think we should go forward.

Cheers.

--
Robert



More information about the linux-mtd mailing list