From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757156AbbLAWRx (ORCPT ); Tue, 1 Dec 2015 17:17:53 -0500 Received: from mail-pa0-f48.google.com ([209.85.220.48]:36841 "EHLO mail-pa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757046AbbLAWRv (ORCPT ); Tue, 1 Dec 2015 17:17:51 -0500 Date: Tue, 1 Dec 2015 14:17:47 -0800 From: Brian Norris To: Boris Brezillon Cc: David Woodhouse , linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Jonathan Corbet , linux-doc@vger.kernel.org, Hartley Sweeten , Ryan Mallon , Shawn Guo , Sascha Hauer , Imre Kaloz , Krzysztof Halasa , Tony Lindgren , linux-omap@vger.kernel.org, Alexander Clouter , Thomas Petazzoni , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Daniel Mack , Haojian Zhuang , Robert Jarzmik , Marek Vasut , Steven Miao , adi-buildroot-devel@lists.sourceforge.net, Mikael Starvik , Jesper Nilsson , linux-cris-kernel@axis.com, Josh Wu , Wan ZongShun , Ezequiel Garcia , Maxim Levitsky , Kukjin Kim , Krzysztof Kozlowski , linux-samsung-soc@vger.kernel.org, Maxime Ripard , Chen-Yu Tsai , linux-sunxi@googlegroups.com, Stefan Agner , Greg Kroah-Hartman , devel@driverdev.osuosl.org Subject: Re: [PATCH v2 17/25] mtd: nand: remove useless mtd->priv = chip assignments Message-ID: <20151201221747.GB64635@google.com> References: <1448967802-25796-1-git-send-email-boris.brezillon@free-electrons.com> <1448967802-25796-18-git-send-email-boris.brezillon@free-electrons.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1448967802-25796-18-git-send-email-boris.brezillon@free-electrons.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 01, 2015 at 12:03:14PM +0100, Boris Brezillon wrote: > mtd_to_nand() now uses the container_of() approach to transform an > mtd_info pointer into a nand_chip one. Drop useless mtd->priv > assignments from NAND controller drivers. > > Signed-off-by: Boris Brezillon > --- > Patch generated with the following coccinelle script: > > ---8<---- > virtual patch > > @@ > struct mtd_info m; > struct mtd_info *mp; > struct nand_chip *c; > @@ > ( > -(m).priv = c; > | > -(mp)->priv = c; > | > -(mp)->priv = (void *)c; > ) > ---8<---- > --- ... > diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c > index 3f29734..ed4184c 100644 > --- a/drivers/mtd/nand/s3c2410.c > +++ b/drivers/mtd/nand/s3c2410.c > @@ -834,7 +834,6 @@ static void s3c2410_nand_init_chip(struct s3c2410_nand_info *info, > chip->IO_ADDR_R = chip->IO_ADDR_W; > > nmtd->info = info; > - mtd->priv = chip; After this one, we have: drivers/mtd/nand/s3c2410.c: In function ‘s3c2410_nand_init_chip’: drivers/mtd/nand/s3c2410.c:791:19: warning: unused variable ‘mtd’ [-Wunused-variable] > nmtd->set = set; > > #ifdef CONFIG_MTD_NAND_S3C2410_HWECC Brian From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brian Norris Subject: Re: [PATCH v2 17/25] mtd: nand: remove useless mtd->priv = chip assignments Date: Tue, 1 Dec 2015 14:17:47 -0800 Message-ID: <20151201221747.GB64635@google.com> References: <1448967802-25796-1-git-send-email-boris.brezillon@free-electrons.com> <1448967802-25796-18-git-send-email-boris.brezillon@free-electrons.com> Reply-To: computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <1448967802-25796-18-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Boris Brezillon Cc: David Woodhouse , linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jonathan Corbet , linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Hartley Sweeten , Ryan Mallon , Shawn Guo , Sascha Hauer , Imre Kaloz , Krzysztof Halasa , Tony Lindgren , linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Alexander Clouter , Thomas Petazzoni , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Daniel Mack , Haojian Zhuang , Robert Jarzmik , Marek List-Id: linux-omap@vger.kernel.org On Tue, Dec 01, 2015 at 12:03:14PM +0100, Boris Brezillon wrote: > mtd_to_nand() now uses the container_of() approach to transform an > mtd_info pointer into a nand_chip one. Drop useless mtd->priv > assignments from NAND controller drivers. >=20 > Signed-off-by: Boris Brezillon > --- > Patch generated with the following coccinelle script: >=20 > ---8<---- > virtual patch >=20 > @@ > struct mtd_info m; > struct mtd_info *mp; > struct nand_chip *c; > @@ > ( > -(m).priv =3D c; > | > -(mp)->priv =3D c; > | > -(mp)->priv =3D (void *)c; > ) > ---8<---- > --- ... > diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c > index 3f29734..ed4184c 100644 > --- a/drivers/mtd/nand/s3c2410.c > +++ b/drivers/mtd/nand/s3c2410.c > @@ -834,7 +834,6 @@ static void s3c2410_nand_init_chip(struct s3c2410_nan= d_info *info, > chip->IO_ADDR_R =3D chip->IO_ADDR_W; > =20 > nmtd->info =3D info; > - mtd->priv =3D chip; After this one, we have: drivers/mtd/nand/s3c2410.c: In function =E2=80=98s3c2410_nand_init_chip=E2= =80=99: drivers/mtd/nand/s3c2410.c:791:19: warning: unused variable =E2=80=98mtd=E2= =80=99 [-Wunused-variable] > nmtd->set =3D set; > =20 > #ifdef CONFIG_MTD_NAND_S3C2410_HWECC =20 Brian --=20 You received this message because you are subscribed to the Google Groups "= linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an e= mail to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout. From mboxrd@z Thu Jan 1 00:00:00 1970 From: computersforpeace@gmail.com (Brian Norris) Date: Tue, 1 Dec 2015 14:17:47 -0800 Subject: [PATCH v2 17/25] mtd: nand: remove useless mtd->priv = chip assignments In-Reply-To: <1448967802-25796-18-git-send-email-boris.brezillon@free-electrons.com> References: <1448967802-25796-1-git-send-email-boris.brezillon@free-electrons.com> <1448967802-25796-18-git-send-email-boris.brezillon@free-electrons.com> Message-ID: <20151201221747.GB64635@google.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Dec 01, 2015 at 12:03:14PM +0100, Boris Brezillon wrote: > mtd_to_nand() now uses the container_of() approach to transform an > mtd_info pointer into a nand_chip one. Drop useless mtd->priv > assignments from NAND controller drivers. > > Signed-off-by: Boris Brezillon > --- > Patch generated with the following coccinelle script: > > ---8<---- > virtual patch > > @@ > struct mtd_info m; > struct mtd_info *mp; > struct nand_chip *c; > @@ > ( > -(m).priv = c; > | > -(mp)->priv = c; > | > -(mp)->priv = (void *)c; > ) > ---8<---- > --- ... > diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c > index 3f29734..ed4184c 100644 > --- a/drivers/mtd/nand/s3c2410.c > +++ b/drivers/mtd/nand/s3c2410.c > @@ -834,7 +834,6 @@ static void s3c2410_nand_init_chip(struct s3c2410_nand_info *info, > chip->IO_ADDR_R = chip->IO_ADDR_W; > > nmtd->info = info; > - mtd->priv = chip; After this one, we have: drivers/mtd/nand/s3c2410.c: In function ?s3c2410_nand_init_chip?: drivers/mtd/nand/s3c2410.c:791:19: warning: unused variable ?mtd? [-Wunused-variable] > nmtd->set = set; > > #ifdef CONFIG_MTD_NAND_S3C2410_HWECC Brian