From mboxrd@z Thu Jan 1 00:00:00 1970 From: Artem Bityutskiy Subject: Re: [PATCH 1/3] Fixing compilation warning for 'nand/omap2.c' Date: Tue, 03 Nov 2009 10:42:08 +0200 Message-ID: <1257237728.21596.65.camel@localhost> References: <1257236323.21596.56.camel@localhost> Reply-To: dedekind1@gmail.com Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from smtp.nokia.com ([192.100.122.233]:37795 "EHLO mgw-mx06.nokia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751716AbZKCImc (ORCPT ); Tue, 3 Nov 2009 03:42:32 -0500 In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Vimal Singh Cc: Linux MTD , linux-omap@vger.kernel.org On Tue, 2009-11-03 at 14:08 +0530, Vimal Singh wrote: > On Tue, Nov 3, 2009 at 1:48 PM, Artem Bityutskiy wrote: > > On Fri, 2009-10-30 at 14:12 +0530, Vimal Singh wrote: > >> Looping Linux-omap mailing list. > >> > >> > >> From: Vimal Singh > >> Date: Fri, 30 Oct 2009 11:31:34 +0530 > >> Subject: [PATCH] Fixing compilation warning for 'nand/omap2.c' > >> > >> Fixing below warning in compilation: > >> drivers/mtd/nand/omap2.c: In function 'omap_write_buf_dma_pref': > >> drivers/mtd/nand/omap2.c:508: warning: passing argument 2 of > >> 'omap_nand_dma_transfer' discards qualifiers from pointer target t= ype > >> > >> Signed-off-by: Vimal Singh > >> --- > >> drivers/mtd/nand/omap2.c | 6 ++++-- > >> 1 files changed, 4 insertions(+), 2 deletions(-) > > > > [dedekind@eru l2-mtd-2.6.git]$ scripts/checkpatch.pl vimal > > ERROR: patch seems to be corrupt (line wrapped?) > > #94: FILE: drivers/mtd/nand/omap2.c:500: > > mtd_info *mtd, u_char *buf, int len) > > > > total: 1 errors, 0 warnings, 16 lines checked > > > > vimal has style problems, please review. If any of these errors > > are false positives report them to the maintainer, see > > CHECKPATCH in MAINTAINERS. > > [dedekind@eru l2-mtd-2.6.git]$ vim vimal > > [dedekind@eru l2-mtd-2.6.git]$ git am vimal > > Applying: Fixing compilation warning for 'nand/omap2.c' > > fatal: corrupt patch at line 10 > > Patch failed at 0001 Fixing compilation warning for 'nand/omap2.c' > > When you have resolved this problem run "git am --resolved". > > If you would prefer to skip this patch, instead run "git am --skip"= =2E > > To restore the original branch and stop patching run "git am --abor= t". > > [dedekind@eru l2-mtd-2.6.git]$ > > >=20 > Grr... its again mailer's issue. Fixed this in below patch: >=20 > -vimal >=20 > From eebbd81141b196af2dc3f7a2650ce16b6b0d76ae Mon Sep 17 00:00:00 200= 1 > From: Vimal Singh > Date: Fri, 30 Oct 2009 11:31:34 +0530 > Subject: [PATCH] Fixing compilation warning for 'nand/omap2.c' >=20 > Fixing below warning in compilation: > drivers/mtd/nand/omap2.c: In function 'omap_write_buf_dma_pref': > drivers/mtd/nand/omap2.c:508: warning: passing argument 2 of > 'omap_nand_dma_transfer' discards qualifiers from pointer target type >=20 > Signed-off-by: Vimal Singh > --- > drivers/mtd/nand/omap2.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) git-send-email ... > diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c > index 090ab87..92573d5 100644 > --- a/drivers/mtd/nand/omap2.c > +++ b/drivers/mtd/nand/omap2.c > @@ -501,11 +501,13 @@ static void omap_read_buf_dma_pref(struct > mtd_info *mtd, u_char :-) It is the same again. Try to send the e-mail to yourself, then save it and apply with git-am. If it works, send to the mailing list. > static void omap_write_buf_dma_pref(struct mtd_info *mtd, > const u_char *buf, int len) > { > + u_char *p =3D (u_char *)buf; > + > if (len <=3D mtd->oobsize) > - omap_write_buf_pref(mtd, buf, len); > + omap_write_buf_pref(mtd, p, len); > else > /* start transfer in DMA mode */ > - omap_nand_dma_transfer(mtd, buf, len, 0x1); > + omap_nand_dma_transfer(mtd, p, len, 0x1); > } >=20 > /** --=20 Best Regards, Artem Bityutskiy (=D0=90=D1=80=D1=82=D1=91=D0=BC =D0=91=D0=B8=D1=82=D1=8E= =D1=86=D0=BA=D0=B8=D0=B9) -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html