Date: Tue, 28 Jul 2015 09:07:43 +0800
From: Shawn Guo <shawnguo@kernel.org>
To: Stefan Agner <stefan@agner.ch>
Cc: dwmw2@infradead.org, computersforpeace@gmail.com, mark.rutland@arm.com,
        boris.brezillon@free-electrons.com, aaron@tastycactus.com,
        marb@ixxat.de, pawel.moll@arm.com, ijc+devicetree@hellion.org.uk,
        bpringlemeir@gmail.com, linux-kernel@vger.kernel.org,
        sebastian@breakpoint.cc, robh+dt@kernel.org,
        linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org,
        kernel@pengutronix.de, galak@codeaurora.org, shawn.guo@linaro.org,
        devicetree@vger.kernel.org
Subject: Re: [PATCH v8 0/5] mtd: nand: vf610_nfc: Freescale NFC for VF610
Message-ID: <20150728010743.GW12927@tiger>
References: <1438015365-15685-1-git-send-email-stefan@agner.ch>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1438015365-15685-1-git-send-email-stefan@agner.ch>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 4535
Lines: 105

On Mon, Jul 27, 2015 at 06:42:40PM +0200, Stefan Agner wrote:
> The eigth revision fixes some rough spots in the device trees.
> 
> More information and the full test log of earlier patchset version
> can be found in the cover letter of the last revision v6:
> http://thread.gmane.org/gmane.linux.kernel/1979868
> 
> Changes since v7:
> - vf610-twr.dts: Moved NFC pinmux into the existing iomuxc node
>   and sort new nfc node behind the existing iomuxc node as well.
> - vf610-twr.dts/vf-colibri.dtsi: Dropped _1 suffixes

Now I'm happy with the dts part and waiting for mtd part to be accepted
to actually apply them.

Shawn

> 
> Changes since v6:
> - Rebased ontop of l2-mtd/master (v4.2-rc1 based)
> - Removed HAVE_NAND_VF610_NFC and use depends on. This made
>   "[PATCH v6 4/6] ARM: vf610: enable NAND Flash Controller" unnecessary
> 
> Changes since v5:
> - Removed fsl,mpc5125-nfc compatible string
> - Removed readl/writel_relaxed
> - Change interface of vf610_nfc_transfer_size to match other accessors
> 
> Changes since v4:
> - Rebased ontop of l2-mtd/master (v4.1-rc4 based)
> - Eliminate unnecessary page read (NAND_CMD_SEQIN) since the driver does
>   not support sub-page writes anyway (improves write performance)
> - Support ONFI by enabling READID command with offset and parameter page
>   reads (CMD_PARAM)
> - Change to dedicated read_page/write_page function, enables raw writes
> - Use __LITTLE_ENDIAN to distingush between LE/BE relevant statements
> - Eliminated vf610_nfc_probe_dt in favor of common DT init code
> - Use wait_for_completion_timeout
> - Some style fixes (spaces, etc.)
> 
> Changes since v3:
> - Make the driver selectable when COMPILE_TEST is set
> - Fix compile error due to superfluous ECC_STATUS configuration in initial
>   patch (without ECC correction ECC_STATUS does not need to be configured)
> - Remove custom BBT pattern and switch to in-band BBT in the initial patch
> - Include two bug fixes, for details see the corresponding U-Boot patches:
>   http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/215802
> 
> Changes since v2:
> - Updated binding documentation
> 
> Changes since v1:
> - Nest nfc_config struct within the main nfc struct
> - Use assigned clock binding to specify NFC clock
> - Rebased ontop of MSCM IR patchset (driver parts have been merged)
> - Split out arch Kconfig in a separate config
> - Fix module license
> - Updated MAINTAINERS
> 
> Changes since RFC (Bill Pringlemeir):
> - Renamed driver from fsl_nfc to vf610_nfc
> - Use readl/writel for all register in accessor functions
> - Optimized field accessor functions
> - Implemented PM (suspend/resume) functions
> - Implemented basic support for ECC strength/ECC step size from dt
> - Improved performance of count_written_bits by using hweight32
> - Support ECC with 60-bytes to correct up to 32 bit errors
> - Changed to in-band BBT (NAND_BBT_NO_OOB) which also allows ECC modes
>   which uses up to 60 bytes on 64 byte OOB
> - Removed custom (downstream) BBT pattern since BBT table won't be
>   compatible anyway (due to the change above)
> 
> Stefan Agner (5):
>   mtd: nand: vf610_nfc: Freescale NFC for VF610, MPC5125 and others
>   mtd: nand: vf610_nfc: add hardware BCH-ECC support
>   mtd: nand: vf610_nfc: add device tree bindings
>   ARM: dts: vf610twr: add NAND flash controller peripherial
>   ARM: dts: vf-colibri: enable NAND flash controller
> 
>  .../devicetree/bindings/mtd/vf610-nfc.txt          |  45 ++
>  MAINTAINERS                                        |   6 +
>  arch/arm/boot/dts/vf-colibri.dtsi                  |  32 +
>  arch/arm/boot/dts/vf610-twr.dts                    |  40 +
>  arch/arm/boot/dts/vfxxx.dtsi                       |   8 +
>  drivers/mtd/nand/Kconfig                           |  11 +
>  drivers/mtd/nand/Makefile                          |   1 +
>  drivers/mtd/nand/vf610_nfc.c                       | 839 +++++++++++++++++++++
>  8 files changed, 982 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mtd/vf610-nfc.txt
>  create mode 100644 drivers/mtd/nand/vf610_nfc.c
> 
> -- 
> 2.4.5
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 
--
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/