[PATCH v9 2/3] MTD : add helper functions library and header files for GPMI NAND driver

Huang Shijie shijie8 at gmail.com
Mon Aug 22 09:41:36 EDT 2011


Hi,

On Mon, Aug 22, 2011 at 7:20 AM, Marek Vasut <marek.vasut at gmail.com> wrote:
> On Monday, August 22, 2011 07:03:56 AM Huang Shijie wrote:
>> Hi,
>>
>> > On Wednesday, August 17, 2011 01:50:27 PM Huang Shijie wrote:
>> >> bch-regs.h : registers file for BCH module
>> >> gpmi-regs.h: registers file for GPMI module
>> >> gpmi-lib.c: helper functions library.
>> >>
>> >> Signed-off-by: Huang Shijie<b32955 at freescale.com>
>> >> ---
>
> [...]
>
>> >> +err_out:
>> >> +  return ret;
>> >> +}
>> >> +
>> >> +/* This is very useful! */
>> >
>> > Really? Cool, what for ?
>> > btw. this should really be enclosed in some #ifdef debug or whatnot.
>>
>> Yes, it shows the register contents when DMA timeout occurs.
>> this routine is called only when error occurs.
>>
> Hi,
>
> I'd expect exactly this explanation to be in the comment ;-)
>
ok, i will do it.

thanks
Huang Shijie

> Cheers
>



More information about the linux-arm-kernel mailing list