From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754664AbcDFXfF (ORCPT ); Wed, 6 Apr 2016 19:35:05 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:48701 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752691AbcDFXfB (ORCPT ); Wed, 6 Apr 2016 19:35:01 -0400 Subject: Re: [PATCH 00/12] mtd: get rid of of_mtd.{c,h} and of_get_nand_xx() To: Boris Brezillon , Roger Quadros , Tony Lindgren , Wenyou Yang , Josh Wu , Richard Weinberger , , David Woodhouse , Brian Norris , Kamal Dasu , Han Xu , Ezequiel Garcia , Stefan Agner References: <1459515272-31011-1-git-send-email-boris.brezillon@free-electrons.com> CC: , Nicolas Ferre , Jean-Christophe Plagniol-Villard , Alexandre Belloni , Thomas Petazzoni , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Daniel Mack , Haojian Zhuang , Robert Jarzmik , , , , Maxime Ripard , Chen-Yu Tsai , , Zhou Wang , Alex Smith , Harvey Hunt From: "Franklin S Cooper Jr." Message-ID: <57059CAF.3030002@ti.com> Date: Wed, 6 Apr 2016 18:33:03 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1459515272-31011-1-git-send-email-boris.brezillon@free-electrons.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/01/2016 07:54 AM, Boris Brezillon wrote: > Hello, > > of_mtd.{h,c} are providing the of_get_nand_xxx() helpers to help NAND > controller drivers parse some generic NAND DT properties. > An infrastructure has recently been added to NAND core to automatically > parse those properties when nand_scan_ident() is called, thus removing > the need for NAND controller drivers to manually parse them. > > This series modifies drivers still making use of those helpers to rely > on NAND core initialization, and get rid of the of_mtd.{c,h} files by > moving the of_get_nand_xx() helpers into nand_base.c. > Nothing broke when I ran some NAND tests on my AM335x GP EVM (OMAP SOC). Tested-by: Franklin S Cooper Jr. > Best Regards, > > Boris > > Boris Brezillon (12): > mtd: nand: remove unneeded of_mtd.h inclusions > mtd: nand: atmel: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: omap2: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: brcm: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: davinci: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: gpmi: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: hisi504: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: lpc32xx: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: mxc: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: pxa3xx: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: sh_flctl: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: move of_get_nand_xxx() helpers into nand_base.c > > drivers/memory/omap-gpmc.c | 7 -- > drivers/mtd/nand/atmel_nand.c | 133 +++++++++++++++------------- > drivers/mtd/nand/brcmnand/brcmnand.c | 5 +- > drivers/mtd/nand/davinci_nand.c | 85 +++++++++--------- > drivers/mtd/nand/gpmi-nand/gpmi-nand.c | 21 +++-- > drivers/mtd/nand/hisi504_nand.c | 14 +-- > drivers/mtd/nand/jz4780_nand.c | 1 - > drivers/mtd/nand/lpc32xx_mlc.c | 1 - > drivers/mtd/nand/lpc32xx_slc.c | 24 ++--- > drivers/mtd/nand/mxc_nand.c | 50 +++++------ > drivers/mtd/nand/nand_base.c | 99 ++++++++++++++++++++- > drivers/mtd/nand/omap2.c | 9 +- > drivers/mtd/nand/pxa3xx_nand.c | 28 +++--- > drivers/mtd/nand/qcom_nandc.c | 1 - > drivers/mtd/nand/sh_flctl.c | 31 +++---- > drivers/mtd/nand/sunxi_nand.c | 1 - > drivers/mtd/nand/vf610_nfc.c | 1 - > drivers/of/Makefile | 1 - > drivers/of/of_mtd.c | 155 --------------------------------- > include/linux/of_mtd.h | 56 ------------ > 20 files changed, 291 insertions(+), 432 deletions(-) > delete mode 100644 drivers/of/of_mtd.c > delete mode 100644 include/linux/of_mtd.h > From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Franklin S Cooper Jr." Subject: Re: [PATCH 00/12] mtd: get rid of of_mtd.{c,h} and of_get_nand_xx() Date: Wed, 6 Apr 2016 18:33:03 -0500 Message-ID: <57059CAF.3030002@ti.com> References: <1459515272-31011-1-git-send-email-boris.brezillon@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1459515272-31011-1-git-send-email-boris.brezillon@free-electrons.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Boris Brezillon , Roger Quadros , Tony Lindgren , Wenyou Yang , Josh Wu , Richard Weinberger , linux-mtd@lists.infradead.org, David Woodhouse , Brian Norris , Kamal Dasu , Han Xu , Ezequiel Garcia , Stefan Agner Cc: Thomas Petazzoni , Andrew Lunn , Jason Cooper , Alex Smith , Harvey Hunt , Robert Jarzmik , linux-sunxi@googlegroups.com, Nicolas Ferre , linux-kernel@vger.kernel.org, Haojian Zhuang , Chen-Yu Tsai , Alexandre Belloni , linux-arm-kernel@lists.infradead.org, Zhou Wang , Gregory CLEMENT , bcm-kernel-feedback-list@broadcom.com, Maxime Ripard , linux-omap@vger.kernel.org, Jean-Christophe Plagniol-Villard , Daniel Mack , Sebastian Hesselbarth List-Id: linux-omap@vger.kernel.org On 04/01/2016 07:54 AM, Boris Brezillon wrote: > Hello, > > of_mtd.{h,c} are providing the of_get_nand_xxx() helpers to help NAND > controller drivers parse some generic NAND DT properties. > An infrastructure has recently been added to NAND core to automatically > parse those properties when nand_scan_ident() is called, thus removing > the need for NAND controller drivers to manually parse them. > > This series modifies drivers still making use of those helpers to rely > on NAND core initialization, and get rid of the of_mtd.{c,h} files by > moving the of_get_nand_xx() helpers into nand_base.c. > Nothing broke when I ran some NAND tests on my AM335x GP EVM (OMAP SOC). Tested-by: Franklin S Cooper Jr. > Best Regards, > > Boris > > Boris Brezillon (12): > mtd: nand: remove unneeded of_mtd.h inclusions > mtd: nand: atmel: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: omap2: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: brcm: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: davinci: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: gpmi: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: hisi504: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: lpc32xx: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: mxc: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: pxa3xx: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: sh_flctl: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: move of_get_nand_xxx() helpers into nand_base.c > > drivers/memory/omap-gpmc.c | 7 -- > drivers/mtd/nand/atmel_nand.c | 133 +++++++++++++++------------- > drivers/mtd/nand/brcmnand/brcmnand.c | 5 +- > drivers/mtd/nand/davinci_nand.c | 85 +++++++++--------- > drivers/mtd/nand/gpmi-nand/gpmi-nand.c | 21 +++-- > drivers/mtd/nand/hisi504_nand.c | 14 +-- > drivers/mtd/nand/jz4780_nand.c | 1 - > drivers/mtd/nand/lpc32xx_mlc.c | 1 - > drivers/mtd/nand/lpc32xx_slc.c | 24 ++--- > drivers/mtd/nand/mxc_nand.c | 50 +++++------ > drivers/mtd/nand/nand_base.c | 99 ++++++++++++++++++++- > drivers/mtd/nand/omap2.c | 9 +- > drivers/mtd/nand/pxa3xx_nand.c | 28 +++--- > drivers/mtd/nand/qcom_nandc.c | 1 - > drivers/mtd/nand/sh_flctl.c | 31 +++---- > drivers/mtd/nand/sunxi_nand.c | 1 - > drivers/mtd/nand/vf610_nfc.c | 1 - > drivers/of/Makefile | 1 - > drivers/of/of_mtd.c | 155 --------------------------------- > include/linux/of_mtd.h | 56 ------------ > 20 files changed, 291 insertions(+), 432 deletions(-) > delete mode 100644 drivers/of/of_mtd.c > delete mode 100644 include/linux/of_mtd.h > From mboxrd@z Thu Jan 1 00:00:00 1970 From: fcooper@ti.com (Franklin S Cooper Jr.) Date: Wed, 6 Apr 2016 18:33:03 -0500 Subject: [PATCH 00/12] mtd: get rid of of_mtd.{c,h} and of_get_nand_xx() In-Reply-To: <1459515272-31011-1-git-send-email-boris.brezillon@free-electrons.com> References: <1459515272-31011-1-git-send-email-boris.brezillon@free-electrons.com> Message-ID: <57059CAF.3030002@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 04/01/2016 07:54 AM, Boris Brezillon wrote: > Hello, > > of_mtd.{h,c} are providing the of_get_nand_xxx() helpers to help NAND > controller drivers parse some generic NAND DT properties. > An infrastructure has recently been added to NAND core to automatically > parse those properties when nand_scan_ident() is called, thus removing > the need for NAND controller drivers to manually parse them. > > This series modifies drivers still making use of those helpers to rely > on NAND core initialization, and get rid of the of_mtd.{c,h} files by > moving the of_get_nand_xx() helpers into nand_base.c. > Nothing broke when I ran some NAND tests on my AM335x GP EVM (OMAP SOC). Tested-by: Franklin S Cooper Jr. > Best Regards, > > Boris > > Boris Brezillon (12): > mtd: nand: remove unneeded of_mtd.h inclusions > mtd: nand: atmel: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: omap2: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: brcm: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: davinci: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: gpmi: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: hisi504: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: lpc32xx: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: mxc: rely on generic DT parsing done in nand_scan_ident() > mtd: nand: pxa3xx: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: sh_flctl: rely on generic DT parsing done in > nand_scan_ident() > mtd: nand: move of_get_nand_xxx() helpers into nand_base.c > > drivers/memory/omap-gpmc.c | 7 -- > drivers/mtd/nand/atmel_nand.c | 133 +++++++++++++++------------- > drivers/mtd/nand/brcmnand/brcmnand.c | 5 +- > drivers/mtd/nand/davinci_nand.c | 85 +++++++++--------- > drivers/mtd/nand/gpmi-nand/gpmi-nand.c | 21 +++-- > drivers/mtd/nand/hisi504_nand.c | 14 +-- > drivers/mtd/nand/jz4780_nand.c | 1 - > drivers/mtd/nand/lpc32xx_mlc.c | 1 - > drivers/mtd/nand/lpc32xx_slc.c | 24 ++--- > drivers/mtd/nand/mxc_nand.c | 50 +++++------ > drivers/mtd/nand/nand_base.c | 99 ++++++++++++++++++++- > drivers/mtd/nand/omap2.c | 9 +- > drivers/mtd/nand/pxa3xx_nand.c | 28 +++--- > drivers/mtd/nand/qcom_nandc.c | 1 - > drivers/mtd/nand/sh_flctl.c | 31 +++---- > drivers/mtd/nand/sunxi_nand.c | 1 - > drivers/mtd/nand/vf610_nfc.c | 1 - > drivers/of/Makefile | 1 - > drivers/of/of_mtd.c | 155 --------------------------------- > include/linux/of_mtd.h | 56 ------------ > 20 files changed, 291 insertions(+), 432 deletions(-) > delete mode 100644 drivers/of/of_mtd.c > delete mode 100644 include/linux/of_mtd.h >