Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758279Ab3HIDxM (ORCPT ); Thu, 8 Aug 2013 23:53:12 -0400 Received: from mga02.intel.com ([134.134.136.20]:39843 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758253Ab3HIDxK (ORCPT ); Thu, 8 Aug 2013 23:53:10 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.89,843,1367996400"; d="scan'208";a="360015910" Message-ID: <1376020723.2219.105.camel@sauron.fi.intel.com> Subject: Re: [PATCH v6 00/10] mtd: add datasheet's ECC information to nand_chip{} From: Artem Bityutskiy Reply-To: dedekind1@gmail.com To: Brian Norris Cc: Huang Shijie , dwmw2@infradead.org, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, vikram186@gmail.com Date: Fri, 09 Aug 2013 06:58:43 +0300 In-Reply-To: <20130808230630.GA14434@norris.computersforpeace.net> References: <1368760654-28754-1-git-send-email-b32955@freescale.com> <520357DF.3030403@freescale.com> <20130808230630.GA14434@norris.computersforpeace.net> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.4 (3.6.4-3.fc18) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2214 Lines: 58 On Thu, 2013-08-08 at 16:06 -0700, Brian Norris wrote: > On Thu, Aug 08, 2013 at 04:33:35PM +0800, Huang Shijie wrote: > > Hi Artem & Brian: > > >Hi Huang and others, > > > > > >On Thu, May 16, 2013 at 8:17 PM, Huang Shijie wrote: > > >>1.) Why add the ECC information to the nand_chip{} ? > ... > > >Reviewed-by: Brian Norris > > >Tested-by: Brian Norris > > > > > >Thanks for the work Huang. > > > > > Could you please merge this patch set? > > Thanks for the reminder. I haven't sorted through all the piles of > backed up stuff yet! > > I made a few minor changes (for checkpatch.pl, regarding > __attribute__((packed)), and to make the style a little more > easily-read) and tested the generic stuff on my platform. I've pushed > the series to l2-mtd.git. Thanks for the patience! I guess you can try aiaiai, I run it for all the patches I take to l2-mtd.git. It uses also coccinelle/smatch/sparse and other tools to verify the patches, not only checkpatch. And this git tree contains scripts and various defconfigs for various MTD drivers - the scripts run aiaiai. For example, to check Huang's patches, I run something like: aiaiai-concat-mboxes ~/tmp/huang*.mbox | ./verify ../l2-mtd/ gpmi-nand where "huang*.mbox" is Huang's patch set. There are aliases for verious drivers. There is a 'gen' alias which I use for general changes like changes in nand_base.c. Read about the tool here: http://git.infradead.org/users/dedekind/aiaiai.git/blob/HEAD:/doc/README.announcement http://git.infradead.org/users/dedekind/aiaiai.git/blob/HEAD:/doc/README The mtd maintenance helpers: http://git.infradead.org/users/dedekind/maintaining.git There are few minor issues in aiaiai, which I can fix if you hit them and complain. Also, I did not invest much time in making the user interface and experience very nice. But this can also be improved. -- Best Regards, Artem Bityutskiy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/