[PATCH 1/2] [MTD] nand/denali.c: fixed all checkpatch errors in denali.c

Dong, Chuanxiao chuanxiao.dong at intel.com
Mon Jul 26 23:58:03 EDT 2010


Thanks for your review and comments.
I split this patch to be 6 smaller patches. I will send it out after this mail.
Below are the subjects for these smaller patches:
[PATCH 1/6] mtd: denali.c: clean up all the whitespace in the end of some lines
[PATCH 2/6] mtd: denali.c: clean up whitespace in code indent
[PATCH 3/6] mtd: denali.c: fixed all open brace { check-patch errors
[PATCH 4/6] mtd: denali.c: add and update KERN_ facility level in printk()
[PATCH 5/6] mtd: denali.c: fix all "line over 80 characters" warnings
[PATCH 6/6] mtd: denali.c: fix reserved all check-patch errors

> -----Original Message-----
> From: Artem Bityutskiy [mailto:dedekind1 at gmail.com]
> Sent: Monday, July 26, 2010 12:30 PM
> To: Dong, Chuanxiao
> Cc: linux-mtd at lists.infradead.org; Woodhouse, David; Gao, Yunpeng; Yuan, Hang
> Subject: Re: [PATCH 1/2] [MTD] nand/denali.c: fixed all checkpatch errors in denali.c
> 
> On Thu, 2010-07-22 at 01:32 +0800, Dong, Chuanxiao wrote:
> > Hi all
> > There are 2 patches to fix checkpatch errors in nand/denali.c and nand/denali.h.
> Here is the first one.
> > So can this patch be accepted or somebody give me some comments? I will be
> very glad to fix it.
> > Thanks for your time to review. Looking forward any response.
> >
> > From 9dff702725437433f6412760a9df492be66be6e1 Mon Sep 17 00:00:00 2001
> > From: Chuanxiao Dong <chuanxiao.dong at intel.com>
> > Date: Thu, 22 Jul 2010 00:53:35 +0800
> > Subject: [PATCH 1/2] [MTD] nand/denali.c: fixed checkpatch errors in denali.c
> >
> > Signed-off-by: Chuanxiao Dong <chuanxiao.dong at intel.com>
> 
> This patch is too huge and not reviewable. Please, split it on several
> smaller patches.
> 
> --
> Best Regards,
> Artem Bityutskiy (Артём Битюцкий)



More information about the linux-mtd mailing list